[
https://issues.apache.org/jira/browse/JAXME-94?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12524019
]
Jochen Wiedmann commented on JAXME-94:
--------------------------------------
Had some time yesterday to inspect it. My first impressions:
- This won't make it into the maintenance branch. Although the test suite
runs fine, I would still consider this as a disruptive change. Such changes
are reserved for the unstable branch.
- I have a couple of performance issues. Some of them are basically nitpicks,
but at least two of them are serious:
* The regular expression pattern can be generated by the generator. It
doesn't need to be generated at runtime.
* The map and the compiled pattern can be static and created when the class
is loaded.
Question to Jose: Do you prefer to fix this for yourself or do you want me to
change this?
> startElement method larger than 64KB when using a large amount of
> complexType's
> -------------------------------------------------------------------------------
>
> Key: JAXME-94
> URL: https://issues.apache.org/jira/browse/JAXME-94
> Project: JaxMe
> Issue Type: Improvement
> Components: JaxMe Core
> Affects Versions: 0.5, 0.6
> Environment: All
> Reporter: Robert Eric Reeves
> Assignee: Jochen Wiedmann
> Priority: Critical
> Fix For: 0.5, 0.6
>
> Attachments: test.xsd, veryLongSequences-patch-0.5.2.diff,
> veryLongSequences-patch-0.6-SNAPSHOT.diff
>
>
> Using an XSD that has a complexType with ~50 child complexTypes in a
> sequence, the generate Java code has a method startElement that is too large
> for javac.
> Jose Luis Huertas Fernández has created a patch to fix this error. I will be
> attaching it to this issue. There are two patches. One for 5.2 and the other
> for 6.0.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]