Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 230410 - Mismatched brackets in Class.javajet if template Class/declaredFieldGenFeature.override.javajetinc is used
Summary: Mismatched brackets in Class.javajet if template Class/declaredFieldGenFeatur...
Status: VERIFIED FIXED
Alias: None
Product: EMF
Classification: Modeling
Component: Core (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 trivial (vote)
Target Milestone: ---   Edit
Assignee: Marcelo Paternostro CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 204200
  Show dependency tree
 
Reported: 2008-05-06 11:15 EDT by Gergely Varro CLA
Modified: 2008-05-12 19:45 EDT (History)
1 user (show)

See Also:

Attachments
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 Gergely Varro CLA 2008-05-06 11:15:05 EDT 
Build ID: M20060921-0945

Steps To Reproduce:
1. Set dynamic templates to true and set the Template Directory to an arbitrary value (where JET searches for the overriding templates)
2. Prepare an empty template with name Class/declaredFieldGenFeature.override.javajetinc
3. Run Generator/Generate Model Code

Code generation fails due to a compilation error caused by a bracket mismatch in the Class_.java file in the (dynamically generated) .JETEmitters project

More information:
(File: Class.javajet)
The location where Class/declaredFieldGenFeature.override.javajetinc should be included is preceded by 2 opening brackets, but it is only succeeded by 1 closing bracket as the second closing bracket (see the second line above the 
"<%@ end %><%//Class/declaredFieldGenFeature.override.javajetinc%>" line) is incorrectly placed inside the default implementation.

I think this bug should appear in many different versions of the codegen part (e.g.,  
org.eclipse.emf.codegen.ecore_2.4.0.v200805052017.jar
org.eclipse.emf.codegen.ecore_2.2.1.v200609210005.jar)
Comment 1 Ed Merks CLA 2008-05-06 12:22:26 EDT 
The change is committed to CVS for 2.4.
Comment 2 Nick Boldt CLA 2008-05-12 19:45:47 EDT 
Fix available in HEAD: 2.4.0.I200805121800.