[ 
https://issues.apache.org/jira/browse/OPENJPA-832?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12708559#action_12708559
 ] 

Milosz Tylenda commented on OPENJPA-832:
----------------------------------------

Good job, Donald.

1. Have you managed to force Eclipse formatter to remove braces from one-line 
"if/else" statements? I couldn't achieve this but maybe it's possible with 
newer Eclipse versions (I use 3.2.2).

2. I noticed the formatter file on Wiki contains a few settings related to 
compiler? Is this intentional?

3. Maybe the formatter could have set to true the value "Force split" in Line 
Wrapping -> Class Declarations (and maybe some others too). I am not saying 
it's good or bad but seems to be consistent with the current formatting.


> Add eclipse formatting preferences to website
> ---------------------------------------------
>
>                 Key: OPENJPA-832
>                 URL: https://issues.apache.org/jira/browse/OPENJPA-832
>             Project: OpenJPA
>          Issue Type: Sub-task
>          Components: build / infrastructure
>            Reporter: Michael Dick
>            Assignee: Donald Woods
>            Priority: Minor
>         Attachments: openjpa-formatter.xml
>
>


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to