Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 332271 - [quick assist] Use ()+ icon for quick assists which add parentheses around expressions
Summary: [quick assist] Use ()+ icon for quick assists which add parentheses around ex...
Status: RESOLVED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.7   Edit
Hardware: All All
: P3 minor (vote)
Target Milestone: 3.7 M5   Edit
Assignee: Deepak Azad CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-12-10 01:15 EST by Deepak Azad CLA
Modified: 2010-12-15 02:11 EST (History)
2 users (show)

See Also:


Attachments
fix (2.05 KB, patch)
2010-12-10 01:15 EST, Deepak Azad CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Deepak Azad CLA 2010-12-10 01:15:52 EST
Created attachment 184920 [details]
fix

I found three, which do not use this icon
- Add paranoiac parenthesis for conditions
- Put 'instanceof' in parentheses
- Put 'instanceof' in parentheses

Is there a reason for not doing this ?

(Note: JavaPluginImages.IMG_CORRECTION_CAST is not a great name for this icon but I have not changed this for now)
Comment 1 Deepak Azad CLA 2010-12-15 02:11:35 EST
Fixed in HEAD

The list in comment 0 should have read
- Add paranoiac parenthesis for conditions
- Put 'instanceof' in parentheses
- Put expression in parentheses