Bug 109646 - [DOM] Parsing using K_STATEMENTS doesn't return the right tree for multiple local declarations
Summary: [DOM] Parsing using K_STATEMENTS doesn't return the right tree for multiple l...
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.2   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.1.1   Edit
Assignee: Olivier Thomann CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 128539 (view as bug list)
Depends on:
Blocks:
 
Reported: 2005-09-15 14:24 EDT by Olivier Thomann CLA
Modified: 2006-02-20 16:46 EST (History)
2 users (show)

See Also:


Attachments
Proposed fix (1.02 KB, patch)
2005-09-15 14:35 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 Olivier Thomann CLA 2005-09-15 14:24:08 EDT
The following source:

String s, t, u, v;

ends up as four statements (VariableDeclarationStatement) instead of one
statement with four fragments.
Comment 1 Olivier Thomann CLA 2005-09-15 14:35:45 EDT
Created attachment 27183 [details]
Proposed fix
Comment 2 Olivier Thomann CLA 2005-09-15 14:36:16 EDT
Philippe, Candidate for 3.1.1?
The fix is trivial.
Comment 3 Olivier Thomann CLA 2005-09-15 14:48:17 EDT
Fixed and released in HEAD.
Regression test added in
org.eclipse.jdt.core.tests.dom.ASTConverterTestAST3_2.test0619
Comment 4 Maxime Daniel CLA 2005-09-20 12:58:50 EDT
Verified for 3.2 M2 on build Build id: I20050920-0010.
(Checking that the provided test case is in line with the bug, that it runs
OK with 3.2M2 and runs KO with 3.1.)
Comment 5 Olivier Thomann CLA 2005-09-20 13:03:47 EDT
Verified in I20050920-0010 for 3.2M2
Comment 6 Philipe Mulet CLA 2005-09-22 10:52:41 EDT
+1 for 3.1.1
Comment 7 Olivier Thomann CLA 2005-09-22 17:41:23 EDT
Backported to 3.1. maintenance stream.
Regression test added in
org.eclipse.jdt.core.tests.dom.ASTConverterTestAST3_2.test0616

Comment 8 Olivier Thomann CLA 2006-02-20 16:46:23 EST
*** Bug 128539 has been marked as a duplicate of this bug. ***