Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 328083 - Configure FieldsPerClass in addition to ClassSplitting
Summary: Configure FieldsPerClass in addition to ClassSplitting
Status: CLOSED FIXED
Alias: None
Product: TMF
Classification: Modeling
Component: Xtext (show other bugs)
Version: unspecified   Edit
Hardware: Macintosh Mac OS X - Carbon (unsup.)
: P3 normal (vote)
Target Milestone: SR2   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-10-18 17:25 EDT by Philipp Salvisberg CLA
Modified: 2017-09-19 17:55 EDT (History)
1 user (show)

See Also:
sebastian.zarnekow: indigo+

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 Philipp Salvisberg CLA 2010-10-18 17:25:34 EDT 
Build Identifier: I20100608-0911

For the grammar of SQL*Plus I've needed to reduce the value of FIELDS_PER_CLASS in org.eclipse.xtext.generator.parser.antlr.splitting.AntlrParserSplitter manually. 1500 worked for me. However, it'll be nice if in a coming version of Xtext a setter for FieldsPerClass could be introduce to overwrite the default of 2500 if needed.

Since I really have to change the Xtext source to get rid of the "The code for the static initializer is exceeding the 65535 bytes limit" error message, I consider this a bug. 
 

Reproducible: Always
Comment 1 Sebastian Zarnekow CLA 2010-11-29 05:28:14 EST 
We have to reenable class splitting since it is currently broken in master due to the Antlr 3.2 upgrade.

We should make FIELDS_PER_CLASS configurable in the helios maintenance branch, too.
Comment 2 Sebastian Zarnekow CLA 2010-12-06 03:41:05 EST 
Removed target milestone.
Comment 3 Sebastian Zarnekow CLA 2011-11-10 08:34:34 EST 
Fields_per_class and methods_per_class are configurable in indigo and indigo maintenance.
Comment 4 Karsten Thoms CLA 2017-09-19 17:44:18 EDT 
Closing all bugs that were set to RESOLVED before Neon.0
Comment 5 Karsten Thoms CLA 2017-09-19 17:55:34 EDT 
Closing all bugs that were set to RESOLVED before Neon.0