classpath-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [cp-patches] [RFA] update to eclipse formatter settings


From: Aaron Luchko
Subject: Re: [cp-patches] [RFA] update to eclipse formatter settings
Date: Thu, 07 Jul 2005 17:00:17 -0400

On Thu, 2005-07-07 at 16:13 -0400, Aaron Luchko wrote:
> I've found a couple issues with the style formatter included
> in .settings/org.eclipse.jdt.core.prefs
> 
> I'm still investigating how to handle a few anomalies (it looks like
> there might be some bugs in some of the eclipse formatting code) but
> I've been able to fix one setting where a "throws" clause wouldn't
> automatically be put on a new line and would be indented by two instead
> of four when it was
> 
> The guidelines I'm going by are here,
> 
> "Implements and extends on separate lines, throws too. Indent extends,
> implements, throws. Apply deep indentation for method arguments."
> http://www.gnu.org/software/classpath/docs/hacking.html#SEC6
> 
> As well eclipse automatically added a setting for enum_constants which
> shouldn't affect anything.
> 
> Aaron
> 
> ChangeLog
> 
> 
> 2005-07-05  Aaron Luchko  <address@hidden>
>       * .settings/org.eclipse.jdt.core.prefs: Fixed indent of 'throws'
>         clause for eclipse formatter

Oops, generated the patch at the wrong level and gave the wrong date on
the ChangeLog entry :)

It's also worthwhile to note that in general these changes should be
propagated to scripts/eclipse-gnu.xml, the standalone settings for the
formatter.

Unfortunately the eclipse functionality to do so seems to be broken at
the moment so I'll have to update eclipse-gnu.xml at a later date.


2005-07-07  Aaron Luchko  <address@hidden>
        * .settings/org.eclipse.jdt.core.prefs: Fixed indent of 'throws'
        clause for eclipse formatter


Attachment: classpath_settings2.patch
Description: Text Data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]