OK, thanks a lot, Aaron. I found these files. Yes, org.eclipse.jdt.core.prefs contains a second, duplicating set of formatting options, together with settings
about compiler warnings and errors. Indeed, as long as we have two
sets of settings, we should at least keep them identical. The proposed
patch modifies the org.eclipse.jdt.core.prefs, changing formatting code from 16 into 21 (drop list of related keywords into new line and indent by one, as far as I
was able to hack). With my Eclipse, it works fine.

The old coding style seems equally good for me. If more people think it is not
worthwhile to change, we can revert both patches and add two words into the
hackers guide instead.

Regards.

2005-08-03  Audrius Meskauskas  <[EMAIL PROTECTED]>

* .settings/org.eclipse.jdt.core.prefs: Drop extends, implements, throws
into new line and indent by one,


<cid:[email protected]>
Index: .settings/org.eclipse.jdt.core.prefs
===================================================================
RCS file: /cvsroot/classpath/classpath/.settings/org.eclipse.jdt.core.prefs,v
retrieving revision 1.2
diff -u -r1.2 org.eclipse.jdt.core.prefs
--- .settings/org.eclipse.jdt.core.prefs        1 May 2005 22:05:52 -0000       
1.2
+++ .settings/org.eclipse.jdt.core.prefs        3 Aug 2005 21:08:12 -0000
@@ -65,11 +65,11 @@
 
org.eclipse.jdt.core.formatter.alignment_for_parameters_in_constructor_declaration=18
 
org.eclipse.jdt.core.formatter.alignment_for_parameters_in_method_declaration=18
 org.eclipse.jdt.core.formatter.alignment_for_selector_in_method_invocation=0
-org.eclipse.jdt.core.formatter.alignment_for_superclass_in_type_declaration=16
+org.eclipse.jdt.core.formatter.alignment_for_superclass_in_type_declaration=21
 
org.eclipse.jdt.core.formatter.alignment_for_superinterfaces_in_enum_declaration=16
-org.eclipse.jdt.core.formatter.alignment_for_superinterfaces_in_type_declaration=16
-org.eclipse.jdt.core.formatter.alignment_for_throws_clause_in_constructor_declaration=16
-org.eclipse.jdt.core.formatter.alignment_for_throws_clause_in_method_declaration=16
+org.eclipse.jdt.core.formatter.alignment_for_superinterfaces_in_type_declaration=21
+org.eclipse.jdt.core.formatter.alignment_for_throws_clause_in_constructor_declaration=21
+org.eclipse.jdt.core.formatter.alignment_for_throws_clause_in_method_declaration=21
 org.eclipse.jdt.core.formatter.blank_lines_after_imports=1
 org.eclipse.jdt.core.formatter.blank_lines_after_package=1
 org.eclipse.jdt.core.formatter.blank_lines_before_field=1
_______________________________________________
Classpath-patches mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/classpath-patches

Reply via email to