Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 352467 - [1.7][formatter][preferences] Rename line wrapping preferences for "try" and "catch"
Summary: [1.7][formatter][preferences] Rename line wrapping preferences for "try" and ...
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.7   Edit
Hardware: All All
: P2 trivial (vote)
Target Milestone: 3.7.1   Edit
Assignee: Dani Megert CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-07-19 10:48 EDT by Ayushman Jain CLA
Modified: 2011-08-03 03:29 EDT (History)
2 users (show)

See Also:

Attachments
Fix (5.08 KB, patch)
2011-07-21 04:40 EDT, Dani Megert 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-19 10:48:06 EDT 
BETA_JAVA7

In the formatter>Line wrapping options, we currently see under "Statements" category options for "try" and "catch".
These were introduced in 1.7 specifically to tackle the try with resources and multi-catch constructs and do not apply to the old try and catch constructs. The wording should reflect this. SO try should be renamed to "try with resources" and catch to "multi-catch".


Same goes for formatter>white space.
Comment 1 Dani Megert CLA 2011-07-20 04:56:02 EDT 
(In reply to comment #0)
> BETA_JAVA7
> 
> In the formatter>Line wrapping options, we currently see under "Statements"
> category options for "try" and "catch".
> These were introduced in 1.7 specifically to tackle the try with resources and
> multi-catch constructs and do not apply to the old try and catch constructs.
> The wording should reflect this. SO try should be renamed to "try with
> resources" and catch to "multi-catch".
I agree.

> Same goes for formatter>white space.
But only for 'try' ('catch' was there before).
Comment 2 Dani Megert CLA 2011-07-20 04:56:59 EDT 
==> use
try-with-resources
multi-catch
Comment 3 Dani Megert CLA 2011-07-21 04:38:22 EDT 
Fixed in BETA_JAVA7.
Comment 4 Dani Megert CLA 2011-07-21 04:40:03 EDT 
Created attachment 200060 [details]
Fix
Comment 5 Raksha Vasisht CLA 2011-08-03 03:29:00 EDT 
Verified for Juno M1 with 4.2 build I20110729-0200.