Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 314039

Summary: [dom] [rewrite] more complete rewriting for method mappings
Product: [Tools] Objectteams Reporter: Stephan Herrmann <stephan.herrmann>
Component: OTDTAssignee: Stephan Herrmann <stephan.herrmann>
Status: VERIFIED FIXED QA Contact:
Severity: normal    
Priority: P3    
Version: 0.7   
Target Milestone: 0.7 M3   
Hardware: Other   
OS: Linux   
Whiteboard:
Bug Depends on:    
Bug Blocks: 310923    
Attachments:
Description Flags
tests & fix none

Description Stephan Herrmann CLA 2010-05-23 05:25:46 EDT
Currently, DOM rewriting cannot handle some changes of method mappings,
notably adding/removing signatures.
Comment 1 Stephan Herrmann CLA 2010-05-24 08:14:46 EDT
Created attachment 169668 [details]
tests & fix

Implementation:
- ASTRewriteAnalyzer: implement rewriting incl. adding/removing signatures
- Parser: ensure methodSpec.declarationSourceStart does include a type parameter
- new helper AbstractMethodMappingDeclaration.removeSignatures()
- fix MethodSpec/FieldAccessSpec: 
  {field,return}Type is optional, don't init with "void"
Comment 2 Stephan Herrmann CLA 2010-05-24 08:17:04 EDT
Patch has been committed as r369.
Comment 3 Stephan Herrmann CLA 2010-05-29 17:28:17 EDT
verified for M3 using 201005282024 
(indirectly via bug 310923 and by way of tests).