Release v0.6.5#39
Closed
likivisw wants to merge 1 commit intoqualcomm:mainfrom
Closed
Conversation
Missing headers now produce reports instead of halting: - file1 missing: report from file2, BACKWARD_COMPATIBLE - file2 missing: report from file1, BACKWARD_COMPATIBLE - Both missing: unchanged, no report Adds --lang/-l for C/C++ mode; fixes parse failures on C headers using C++ reserved keywords. Default remains cpp. Signed-off-by: likith viswanath <likivisw@qti.qualcomm.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Release v0.6.5 — ARMOR
🚀 New Features
Missing Header Handling — Report Generation on Absent Files
Previously, when a header file was missing from either the older (
projectroot1) or newer(
projectroot2) version of a project, ARMOR would only log an error and halt processing forthat header. No report artefact was produced, leaving callers with nothing to inspect.
Report generation is now performed in both processing paths — the explicit
headerslistand the
--header-dirdirectory scan — for the two single-file-missing cases described below.file1missing (header absent in the older version): A report is now generatedusing the filename derived from
file2, with compatibility verdictBACKWARD_COMPATIBLE,parsed diff status
SUPPORTED_UPDATES, unparsed diff statusUN_CHANGED, and note"Missing header in older version"file2missing (header absent in the newer version): A report is now generatedusing the filename derived from
file1, with compatibility verdictBACKWARD_COMPATIBLE,parsed diff status
SUPPORTED_UPDATES, unparsed diff statusUN_CHANGED, and note"Missing header in newer version"file1andfile2missing: No change in behaviour. An error is loggedand no report is generated, as there is no file from which to derive a meaningful header
name or report content
C Language Mode —
--lang/-lOptionAdds a
--lang/-lCLI option to the ARMOR AST diff tool, enabling correct parsing ofC headers that use C++ reserved keywords (e.g.
class,template,namespace) asidentifiers. Without this flag such headers would previously fail to parse entirely.
Default:
cpp— existing invocations are unaffected.Example:
🐛 Bug Fixes
Parser Accuracy Improvements
keywords (
class,template,namespace, etc.) as valid C identifiers — these headersnow parse correctly when
--lang cis supplied📊 Technical Improvements
resolved language mode
LANG_OPTIONSenum (C = 0,CPP = 1) and string constantsLANG_C/LANG_CPPadded tocomm_def.hppfor consistent language-mode handling across the codebase--lang/-loption wired throughoptions_handler.cpp; resolved value forwardedto both the alpha and beta header processors
LANG_OPTIONSvalue