Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 340375 - [1.7] Merge TryStatementWithResources into TryStatement
Summary: [1.7] Merge TryStatementWithResources into TryStatement
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.7   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 3.7.1   Edit
Assignee: Olivier Thomann CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 340365 340372
  Show dependency tree
 
Reported: 2011-03-17 17:10 EDT by Markus Keller CLA
Modified: 2011-08-05 02:54 EDT (History)
5 users (show)

See Also:


Attachments
Proposed fix (42.20 KB, patch)
2011-03-17 22:59 EDT, Olivier Thomann CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Markus Keller CLA 2011-03-17 17:10:07 EDT
BETA_JAVA7

Looking closer at TryStatementWithResources, I think we shouldn't add this as a new node at all.

It would be better to add the "resources" property to the existing TryStatement, similar to how [ < TypeParameter { , TypeParameter } > ] has been added to TypeDeclaration.
Comment 1 Olivier Thomann CLA 2011-03-17 22:59:30 EDT
Created attachment 191481 [details]
Proposed fix

This also contains the fix for bug 340365.
Comment 2 Olivier Thomann CLA 2011-03-18 10:48:23 EDT
Released for 3.7M7 in BETA_JAVA7 branch.
Comment 3 Jay Arthanareeswaran CLA 2011-06-29 07:38:06 EDT
Verified.