Skip to content

Standardize option descriptions to "override"#1362

Merged
gmaurel merged 1 commit intouncrustify:masterfrom
brmqk3:standardize_option_descriptions
Oct 13, 2017
Merged

Standardize option descriptions to "override"#1362
gmaurel merged 1 commit intouncrustify:masterfrom
brmqk3:standardize_option_descriptions

Conversation

@brmqk3
Copy link
Copy Markdown
Contributor

@brmqk3 brmqk3 commented Sep 29, 2017

These options have similar descriptions:
*sp_after_operator_sym_empty - Have precedence of...
*nl_typedef_blk_start - takes preference over...
*nl_func_paren_empty - Overrides...

This change modifies these descriptions to use the
"override" wording. This doesn't make any changes to
the options like sp_before_unnamed_ptr_star, where if it's
set to ignore, another options is used, and isn't really
overridden.

@gmaurel
Copy link
Copy Markdown
Collaborator

gmaurel commented Sep 30, 2017

You have to modify the file scripts/Output/mini_nd_ucwd.txt to reflect your changes.

These options have similar descriptions:
*sp_after_operator_sym_empty - Have precedence of...
*nl_typedef_blk_start - takes preference over...
*nl_func_paren_empty - Overrides...

This change modifies these descriptions to use the
"override" wording. This doesn't make any changes to
the options like sp_before_unnamed_ptr_star, where if it's
set to ignore, another options is used, and isn't really
overridden.

Change-Id: I47ecd54c4ed3f3950a15052ef86f4caabe8ffbdf
Type: Code Improvement
@brmqk3 brmqk3 force-pushed the standardize_option_descriptions branch from 8e5749d to e5ce3cc Compare October 12, 2017 22:32
@brmqk3
Copy link
Copy Markdown
Contributor Author

brmqk3 commented Oct 12, 2017

Modified.

@gmaurel gmaurel merged commit 49715c3 into uncrustify:master Oct 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants