Hi Joachim, you thoughts sound good to me.
Will try to have a look at the grammer and help you get it translated to JavaCC but as I'm quite busy these days I can not promis when I will be able to do so. Regards Ralf Joachim Grüneis schrieb: > Hello, > > please read inline... > > 2008/10/28 Ralf Joachim <[EMAIL PROTECTED]>: >> Hi Joachim, >> >> I do not like the inflationary increase of modules we see at the moment. >> Please keep in mind that our users have to add a new dependency for each of >> our modules. > > either I check-in the pull parser as part of xml module or in a > complete new project; having it as additional module makes no sense as > we would still introduce the new dependencies > >> Having said that the increasing number of dependecies to external jar's >> isn't a good direction either but there doesn't seam to be much alternatives >> for the pull parser. > > This is exactly my point and why I kicked off this thread... both > choices aren't fine > >> In which Java version is the StAX API included? > > JDK6 and JAXP 1.4 > >> StAX implementation should be required for tests only? > > yep > >> Is ANTLR needed at runtime or only during build? > > that was one point why you choose to take JavaCC - ANTLR requires a > runtime library > for the prototype - and as you hadn't had a decision at this point - > we took ANTLR as it is easier to build a grammar with it > >> As you may remember our discussion about parser generators during >> implementation of pull parser. JavaCC, which will be used at JDO in the >> future, is only required during build but not at runtime. >> >> IMHO we should already try to reduce number of modules and external >> dependencies with future releases. I also think of throwing away features or >> parts of code that get never used at this step. > > I'm currently thinking we should: > * check it into xml module > * transfer the grammar to JavaCC > * introduce the StAX dependency > > Have fun > Joachim > >> Regards >> Ralf >> >> >> Joachim Grüneis schrieb: >> >> Hello, >> >> after adapting the existing pull purser patch I'm not 100% sure if we >> should have it in the main source tree of Castor... >> it requires StAX API's, StAX implementation and two ANTLR libraries >> maybe - even that it is a very small piece of code - we should have it >> in an own project? >> >> what is your opinion? >> >> Joachim >> >> 2008/10/23 Werner Guttmann <[EMAIL PROTECTED]>: >> >> >> Hi, >> >> any chances of committing yourself to a firm deadline? I would like to >> ship 1.3 final any time soon, and I do not want to be waiting for weeks. >> >> Werner >> >> Joachim Grüneis wrote: >> >> >> Hello, >> >> I plan to add it to the code base within the next days... >> >> Joachim >> >> 2008/10/22 Werner Guttmann <[EMAIL PROTECTED]>: >> >> >> Hi Joachim, >> >> what are your plans of committing the pull-parsing add-on that has been >> developed as part of GSoC 2008 ? >> >> Werner >> >> --------------------------------------------------------------------- >> To unsubscribe from this list, please visit: >> >> http://xircles.codehaus.org/manage_email >> >> >> >> >> >> --------------------------------------------------------------------- >> To unsubscribe from this list, please visit: >> >> http://xircles.codehaus.org/manage_email >> >> >> >> >> --------------------------------------------------------------------- >> To unsubscribe from this list, please visit: >> >> http://xircles.codehaus.org/manage_email >> >> >> >> >> >> --------------------------------------------------------------------- >> To unsubscribe from this list, please visit: >> >> http://xircles.codehaus.org/manage_email >> >> >> >> -- >> >> Syscon Ingenieurbüro für Meß- und Datentechnik GmbH >> Ralf Joachim >> Raiffeisenstraße 11 >> 72127 Kusterdingen >> Germany >> >> Tel. +49 7071 3690 52 >> Mobil: +49 173 9630135 >> Fax +49 7071 3690 98 >> >> Internet: www.syscon.eu >> E-Mail: [EMAIL PROTECTED] >> >> Sitz der Gesellschaft: D-72127 Kusterdingen >> Registereintrag: Amtsgericht Stuttgart, HRB 382295 >> Geschäftsleitung: Jens Joachim, Ralf Joachim >> > > --------------------------------------------------------------------- > To unsubscribe from this list, please visit: > > http://xircles.codehaus.org/manage_email > --------------------------------------------------------------------- To unsubscribe from this list, please visit: http://xircles.codehaus.org/manage_email
