Re: Eclipse command line formatter

[Tom Tromey]

>>>>> "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