[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Eclipse command line formatter
From: |
Tom Tromey |
Subject: |
Re: Eclipse command line formatter |
Date: |
11 Jan 2005 13:16:39 -0700 |
User-agent: |
Gnus/5.09 (Gnus v5.9.0) Emacs/21.3.50 |
>>>>> "Ben" == Ben Konrath <address@hidden> writes:
Ben>
http://www.bagu.org/eclipse/org.eclipse.jdt.core-formatter-application.patch
I noticed a typo, "formmatting", in FormatterAppMessages.properties.
Ben> Also, I'd like to know if there are any requests for this patch
Ben> (like what to output ie quiet mode for cron jobs, etc) before I
Ben> post it to the eclipse.org bugzilla
Ben> (https://bugs.eclipse.org/bugs/show_bug.cgi?id=75333).
I read through processCommandLine a little. Letting it work on
multiple files might be useful.
Ben> I didn't add an option to fix the imports because the solution involves a
Ben> little refactoring and therefore would be more difficult to
Ben> maintain if it is not accepted upstream.
IMO that's fine.
What we really need to do is make sure we can really handle Classpath
style. With Eclipse 3.0 I think we fail to re-indent inside a block,
so we end up with:
if (foo)
{
code here;
}
Instead of:
if (foo)
{
code here;
}
I haven't tried Eclipse 3.1 yet... maybe it has this. Or maybe I just
made a mistake when configuring. I'll try to get around to checking
this stuff; I'll file in eclipse.org bugzilla if needed.
Tom