diff options
| author | Robin Stocker | 2012-04-27 20:56:03 +0000 |
|---|---|---|
| committer | Robin Stocker | 2012-04-27 21:07:46 +0000 |
| commit | e77998338de9acda0850e4a9c78a0923bd4e44b3 (patch) | |
| tree | c06de34993470dfbd13aecc38285a76d218f5540 /org.eclipse.egit.mylyn.ui.test | |
| parent | c823483b86304a9fff4eff0f97e2012f12531eec (diff) | |
| download | egit-e77998338de9acda0850e4a9c78a0923bd4e44b3.tar.gz egit-e77998338de9acda0850e4a9c78a0923bd4e44b3.tar.xz egit-e77998338de9acda0850e4a9c78a0923bd4e44b3.zip | |
Revert "Save Actions for removing braces of one-line statements"
This reverts commit f95c105980fc8dc1c356665bb796f256d62bf1ca.
The problem with the save action is that it also results in removed
braces for the if in the following example:
if (foo)
try {
bar();
} finally {
baz();
}
Which, altough it's a single statement, isn't recommended because it's
longer than one line and confusing.
See I1279030286ef177f3360ff70660a9365e4f3119a for the exact reasons.
Change-Id: I4564e3d312fe3663d22ac784de83408509275219
Signed-off-by: Robin Stocker <robin@nibor.org>
Diffstat (limited to 'org.eclipse.egit.mylyn.ui.test')
| -rw-r--r-- | org.eclipse.egit.mylyn.ui.test/.settings/org.eclipse.jdt.ui.prefs | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/org.eclipse.egit.mylyn.ui.test/.settings/org.eclipse.jdt.ui.prefs b/org.eclipse.egit.mylyn.ui.test/.settings/org.eclipse.jdt.ui.prefs index 84c952fb5b..66184464a9 100644 --- a/org.eclipse.egit.mylyn.ui.test/.settings/org.eclipse.jdt.ui.prefs +++ b/org.eclipse.egit.mylyn.ui.test/.settings/org.eclipse.jdt.ui.prefs @@ -1,4 +1,4 @@ -#Sun Jan 22 14:20:31 CET 2012 +#Wed Feb 03 18:26:19 PST 2010 eclipse.preferences.version=1 editor_save_participant_org.eclipse.jdt.ui.postsavelistener.cleanup=true formatter_profile=_JGit Format @@ -15,7 +15,6 @@ sp_cleanup.add_missing_deprecated_annotations=false sp_cleanup.add_missing_methods=false sp_cleanup.add_missing_nls_tags=false sp_cleanup.add_missing_override_annotations=false -sp_cleanup.add_missing_override_annotations_interface_methods=false sp_cleanup.add_serial_version_id=false sp_cleanup.always_use_blocks=false sp_cleanup.always_use_parentheses_in_expressions=false @@ -30,7 +29,7 @@ sp_cleanup.make_parameters_final=false sp_cleanup.make_private_fields_final=false sp_cleanup.make_type_abstract_if_missing_method=false sp_cleanup.make_variable_declarations_final=false -sp_cleanup.never_use_blocks=true +sp_cleanup.never_use_blocks=false sp_cleanup.never_use_parentheses_in_expressions=false sp_cleanup.on_save_use_additional_actions=true sp_cleanup.organize_imports=false @@ -53,7 +52,7 @@ sp_cleanup.remove_unused_private_methods=false sp_cleanup.remove_unused_private_types=false sp_cleanup.sort_members=false sp_cleanup.sort_members_all=false -sp_cleanup.use_blocks=true +sp_cleanup.use_blocks=false sp_cleanup.use_blocks_only_for_return_and_throw=false sp_cleanup.use_parentheses_in_expressions=false sp_cleanup.use_this_for_non_static_field_access=false |