Reviewed-by: Michael D Kinney <michael.d.kin...@intel.com> Mike
> -----Original Message----- > From: mikub...@linux.microsoft.com <mikub...@linux.microsoft.com> > Sent: Thursday, February 10, 2022 8:19 AM > To: devel@edk2.groups.io > Cc: Kinney, Michael D <michael.d.kin...@intel.com>; Gao, Liming > <gaolim...@byosoft.com.cn>; Sean Brogan > <sean.bro...@microsoft.com>; Bret Barkelew <bret.barke...@microsoft.com> > Subject: [PATCH v1 1/1] .pytool/Plugin/UncrustifyCheck: Output file diffs by > default > > From: Michael Kubacki <michael.kuba...@microsoft.com> > > REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3808 > > Changes the default for the "OutputFileDiffs" configuration option > to "True" so the formatting suggestions detected by Uncrustify > will be output in the test case log. The diff is printed in unified > diff format. > > This was disabled by default during the initial enabling of > Uncrustify to reduce overall execution time of the plugin against > the codebase due to the large number of changes detected. > > Cc: Michael D Kinney <michael.d.kin...@intel.com> > Cc: Liming Gao <gaolim...@byosoft.com.cn> > Cc: Sean Brogan <sean.bro...@microsoft.com> > Cc: Bret Barkelew <bret.barke...@microsoft.com> > Signed-off-by: Michael Kubacki <michael.kuba...@microsoft.com> > --- > .pytool/Plugin/UncrustifyCheck/Readme.md | 4 ++-- > .pytool/Plugin/UncrustifyCheck/UncrustifyCheck.py | 6 +++--- > 2 files changed, 5 insertions(+), 5 deletions(-) > > diff --git a/.pytool/Plugin/UncrustifyCheck/Readme.md > b/.pytool/Plugin/UncrustifyCheck/Readme.md > index bb263bcc87d7..0c46fd241a7a 100644 > --- a/.pytool/Plugin/UncrustifyCheck/Readme.md > +++ b/.pytool/Plugin/UncrustifyCheck/Readme.md > @@ -42,7 +42,7 @@ The plugin can be configured with a few optional > configuration options. > "AuditOnly": False, # Don't fail the build if there are > errors. Just log them. > "ConfigFilePath": "", # Custom path to an Uncrustify config > file. > "IgnoreStandardPaths": [], # Standard Plugin defined paths that > should be ignored. > - "OutputFileDiffs": False, # Output chunks of formatting diffs in > the test case log. > + "OutputFileDiffs": True, # Output chunks of formatting diffs in > the test case log. > # This can significantly slow down the > plugin on very large packages. > "SkipGitExclusions": False # Don't exclude git ignored files and > files in git submodules. > } > @@ -82,7 +82,7 @@ to be ignored. > > ### `OutputFileDiffs` > > -`Boolean` - Default is `False`. > +`Boolean` - Default is `True`. > > If `True`, output diffs of formatting changes into the test case log. This > is helpful to exactly understand what changes > need to be made to the source code in order to fix a coding standard > compliance issue. > diff --git a/.pytool/Plugin/UncrustifyCheck/UncrustifyCheck.py > b/.pytool/Plugin/UncrustifyCheck/UncrustifyCheck.py > index 6db8d1739a80..ea8396942b16 100644 > --- a/.pytool/Plugin/UncrustifyCheck/UncrustifyCheck.py > +++ b/.pytool/Plugin/UncrustifyCheck/UncrustifyCheck.py > @@ -494,13 +494,13 @@ class UncrustifyCheck(ICiBuildPlugin): > Initializes options that influence test case output. > """ > self._audit_only_mode = False > - self._output_file_diffs = False > + self._output_file_diffs = True > > if "AuditOnly" in self._package_config and > self._package_config["AuditOnly"]: > self._audit_only_mode = True > > - if "OutputFileDiffs" in self._package_config and > self._package_config["OutputFileDiffs"]: > - self._output_file_diffs = True > + if "OutputFileDiffs" in self._package_config and not > self._package_config["OutputFileDiffs"]: > + self._output_file_diffs = False > > def _log_uncrustify_app_info(self) -> None: > """ > -- > 2.28.0.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#86566): https://edk2.groups.io/g/devel/message/86566 Mute This Topic: https://groups.io/mt/89049118/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-