Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 351081 - [1.7][preferences][formatter] New option to wrap before/after '|' in multi-catch
Summary: [1.7][preferences][formatter] New option to wrap before/after '|' in multi-catch
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.7   Edit
Hardware: All All
: P2 enhancement (vote)
Target Milestone: 3.7.1   Edit
Assignee: Deepak Azad CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 350895
Blocks:
  Show dependency tree
 
Reported: 2011-07-04 08:52 EDT by Ayushman Jain CLA
Modified: 2011-08-02 05:45 EDT (History)
3 users (show)

See Also:
deepakazad: review+

Attachments
proposed fix (3.71 KB, patch)
2011-07-04 08:58 EDT, Ayushman Jain CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ayushman Jain CLA 2011-07-04 08:52:07 EDT 
+++ This bug was initially created as a clone of Bug #350895 +++

BETA_JAVA7

UI for new "wrap before '|' operator" preference from bug 350895
Comment 1 Ayushman Jain CLA 2011-07-04 08:58:04 EDT 
Created attachment 199047 [details]
proposed fix

Was using these changes to play around with the fix for bug 350895. Thought I might as well attach a patch. :)
Comment 2 Markus Keller CLA 2011-07-04 09:02:33 EDT 
Deepak, please review and commit if good (after the formatter patch has been released).
Comment 3 Deepak Azad CLA 2011-07-12 01:59:54 EDT 
Ayush, there was one small typo in the patch. Rest is good.

Fixed in BETA_JAVA7.
Comment 4 Satyam Kandula CLA 2011-07-19 10:08:50 EDT 
This option should be disabled when "Do not Wrap" is selected. Filed bug 352461 to take care of this.
Comment 5 Satyam Kandula CLA 2011-07-19 10:09:16 EDT 
Verified using patch 1.0.0.v20110714-1400