[ 
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]

Reply via email to