diff options
| author | Fabrice Tiercelin | 2021-04-20 16:30:04 +0000 |
|---|---|---|
| committer | Jeff Johnston | 2021-04-21 22:31:27 +0000 |
| commit | 186b204fb4ef60b465039b1ff18ec2b7df68f995 (patch) | |
| tree | fa008faec37697df77bf8c9f6d6462c72b4d9c94 | |
| parent | 91f439fd840c3e99b6f9b6a672bcf0794ef32aa0 (diff) | |
| download | eclipse.jdt.ui-186b204fb4ef60b465039b1ff18ec2b7df68f995.tar.gz eclipse.jdt.ui-186b204fb4ef60b465039b1ff18ec2b7df68f995.tar.xz eclipse.jdt.ui-186b204fb4ef60b465039b1ff18ec2b7df68f995.zip | |
Bug 530122 - Add new Clean Up profile which is more helpful and modern
Change-Id: I024ed1749598fd536360d80c13676cacd8bb1f0a
Signed-off-by: Fabrice Tiercelin <fabrice.tiercelin@yahoo.fr>
Reviewed-on: https://git.eclipse.org/r/c/jdt/eclipse.jdt.ui/+/178732
Tested-by: JDT Bot <jdt-bot@eclipse.org>
Tested-by: Jeff Johnston <jjohnstn@redhat.com>
Reviewed-by: Jeff Johnston <jjohnstn@redhat.com>
| -rw-r--r-- | org.eclipse.jdt.ui/core extension/org/eclipse/jdt/internal/corext/fix/CleanUpConstantsOptions.java | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/org.eclipse.jdt.ui/core extension/org/eclipse/jdt/internal/corext/fix/CleanUpConstantsOptions.java b/org.eclipse.jdt.ui/core extension/org/eclipse/jdt/internal/corext/fix/CleanUpConstantsOptions.java index 6e9a86d873..daf13d9f11 100644 --- a/org.eclipse.jdt.ui/core extension/org/eclipse/jdt/internal/corext/fix/CleanUpConstantsOptions.java +++ b/org.eclipse.jdt.ui/core extension/org/eclipse/jdt/internal/corext/fix/CleanUpConstantsOptions.java @@ -70,11 +70,11 @@ public class CleanUpConstantsOptions extends CleanUpConstants { //Functional Interfaces options.setOption(CONVERT_FUNCTIONAL_INTERFACES, CleanUpOptions.FALSE); options.setOption(SIMPLIFY_LAMBDA_EXPRESSION_AND_METHOD_REF, CleanUpOptions.FALSE); + + // Optimization options.setOption(PRECOMPILE_REGEX, CleanUpOptions.FALSE); options.setOption(NO_STRING_CREATION, CleanUpOptions.FALSE); options.setOption(PREFER_BOOLEAN_LITERAL, CleanUpOptions.FALSE); - - // Optimization options.setOption(SINGLE_USED_FIELD, CleanUpOptions.FALSE); options.setOption(BREAK_LOOP, CleanUpOptions.FALSE); options.setOption(STATIC_INNER_CLASS, CleanUpOptions.FALSE); @@ -117,7 +117,7 @@ public class CleanUpConstantsOptions extends CleanUpConstants { options.setOption(OVERRIDDEN_ASSIGNMENT, CleanUpOptions.FALSE); options.setOption(REMOVE_REDUNDANT_MODIFIERS, CleanUpOptions.FALSE); options.setOption(RAISE_EMBEDDED_IF, CleanUpOptions.FALSE); - options.setOption(REMOVE_REDUNDANT_SEMICOLONS, CleanUpOptions.FALSE); + options.setOption(REMOVE_REDUNDANT_SEMICOLONS, CleanUpOptions.TRUE); options.setOption(REDUNDANT_COMPARATOR, CleanUpOptions.FALSE); options.setOption(REMOVE_UNNECESSARY_ARRAY_CREATION, CleanUpOptions.FALSE); options.setOption(ARRAY_WITH_CURLY, CleanUpOptions.FALSE); @@ -145,13 +145,13 @@ public class CleanUpConstantsOptions extends CleanUpConstants { options.setOption(FORMAT_SOURCE_CODE, CleanUpOptions.FALSE); options.setOption(FORMAT_SOURCE_CODE_CHANGES_ONLY, CleanUpOptions.FALSE); - options.setOption(FORMAT_REMOVE_TRAILING_WHITESPACES, CleanUpOptions.FALSE); + options.setOption(FORMAT_REMOVE_TRAILING_WHITESPACES, CleanUpOptions.TRUE); options.setOption(FORMAT_REMOVE_TRAILING_WHITESPACES_ALL, CleanUpOptions.TRUE); options.setOption(FORMAT_REMOVE_TRAILING_WHITESPACES_IGNORE_EMPTY, CleanUpOptions.FALSE); options.setOption(FORMAT_CORRECT_INDENTATION, CleanUpOptions.FALSE); - options.setOption(ORGANIZE_IMPORTS, CleanUpOptions.FALSE); + options.setOption(ORGANIZE_IMPORTS, CleanUpOptions.TRUE); options.setOption(SORT_MEMBERS, CleanUpOptions.FALSE); options.setOption(SORT_MEMBERS_ALL, CleanUpOptions.FALSE); @@ -181,7 +181,7 @@ public class CleanUpConstantsOptions extends CleanUpConstants { options.setOption(JOIN, CleanUpOptions.FALSE); options.setOption(TRY_WITH_RESOURCE, CleanUpOptions.FALSE); options.setOption(MULTI_CATCH, CleanUpOptions.FALSE); - options.setOption(REMOVE_REDUNDANT_TYPE_ARGUMENTS, CleanUpOptions.FALSE); + options.setOption(REMOVE_REDUNDANT_TYPE_ARGUMENTS, CleanUpOptions.TRUE); options.setOption(USE_AUTOBOXING, CleanUpOptions.FALSE); options.setOption(USE_UNBOXING, CleanUpOptions.FALSE); options.setOption(CONSTANTS_FOR_SYSTEM_PROPERTY, CleanUpOptions.FALSE); @@ -191,8 +191,8 @@ public class CleanUpConstantsOptions extends CleanUpConstants { options.setOption(CONSTANTS_FOR_SYSTEM_PROPERTY_FILE_ENCODING, CleanUpOptions.FALSE); options.setOption(CONSTANTS_FOR_SYSTEM_PROPERTY_BOOLEAN, CleanUpOptions.FALSE); - options.setOption(CONTROL_STATEMENTS_CONVERT_FOR_LOOP_TO_ENHANCED, CleanUpOptions.FALSE); - options.setOption(CONTROL_STATEMENTS_CONVERT_FOR_LOOP_ONLY_IF_LOOP_VAR_USED, CleanUpOptions.FALSE); + options.setOption(CONTROL_STATEMENTS_CONVERT_FOR_LOOP_TO_ENHANCED, CleanUpOptions.TRUE); + options.setOption(CONTROL_STATEMENTS_CONVERT_FOR_LOOP_ONLY_IF_LOOP_VAR_USED, CleanUpOptions.TRUE); options.setOption(CONTROL_STATEMENTS_USE_ADD_ALL, CleanUpOptions.FALSE); } @@ -242,11 +242,11 @@ public class CleanUpConstantsOptions extends CleanUpConstants { //Functional Interfaces options.setOption(CONVERT_FUNCTIONAL_INTERFACES, CleanUpOptions.FALSE); options.setOption(SIMPLIFY_LAMBDA_EXPRESSION_AND_METHOD_REF, CleanUpOptions.FALSE); + + // Optimization options.setOption(PRECOMPILE_REGEX, CleanUpOptions.FALSE); options.setOption(NO_STRING_CREATION, CleanUpOptions.FALSE); options.setOption(PREFER_BOOLEAN_LITERAL, CleanUpOptions.FALSE); - - // Optimization options.setOption(SINGLE_USED_FIELD, CleanUpOptions.FALSE); options.setOption(BREAK_LOOP, CleanUpOptions.FALSE); options.setOption(STATIC_INNER_CLASS, CleanUpOptions.FALSE); |