This repository was archived by the owner on Dec 10, 2020. It is now read-only.
Closed
Conversation
- adopt to changes in emscripten-core/emscripten#5892 - set the version to C++11 - update the interface itself
This is also a better fix for uncrustify#1041 Thanks to @aneesh1127
Adopt the UT tests 18439 and 19894
Misc refactor
Thanks to KalyanaChakravarthi
that is the place where it belongs and is actually used
frm.level is not used anywhere in indent.cpp
update emscripten interface
Thanks to KalyanaChakravarthi. Thanks to Aneesh Gandavarapu.
Adopt UT tests 60027 and 60028
Adopt UT test 60039
misc improvements
Adopt UT tests 60008, 60012, 60019, 60024
fix template for nl_before options
update ChangeLog Thanks to KalyanaChakravarthi
Adopt UT test 10060
Adopt UT tests 60006-60038
Thanks to KalyanaChakravarthi
Thanks to satish puppala
…y anything inside not to break the things.
Objective-C boxed array @[] and @catch formatting
Add basic processing of @available() syntax in objc
Author
|
One of the CI parameters is wrong so the results are not valid. Waiting for another round (and likely style related PRs) before putting this back in review... |
Author
|
Waiting for a regression to be addressed upstream before we can use the latest version: see uncrustify#1646. |
This was referenced Feb 28, 2018
Merged
Merged
Merged
Merged
Merged
Merged
Author
|
With this configuration change I was able to work around the remaining differences: |
Author
|
Even though all CI builds running the uncrustify tests pass I will close this in favor of an upcoming PR (#21) targeting the latest release of
|
Merged
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
8 participants
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.
Pulling in half a year of bug fixes and improvements.
CI builds only running the
uncrustifylinter tests:Linux
Linux-aarch64
macOS
Windows
Ready for "review".