Hello devs, Ralf Joachim schrieb: > Hi Werner and Joachim > > I share Joachim's opinion that we should not create an own module for > pull parsing but instead integrate into into the XML module even if this > introduces the new dependency to StaX-API. >
I personally don't agree with that opinion. Basically I think, pull parsing is a very interesting feature, but for most of our users not so important right now. Including this functionality into existing modules would mean: Putting a new jar (even if it's not that big) into the lib folder, which I think a) forces users to deal with more dependencies - they might not use - and larger castor jars b) blocks developers from getting rid of fixed classpath files and usage of dependency management and automated build processes. But what is most important to me, it reduces cohesion of existing modules, which I think is one of the basic principles of OO design. > Having said that I would also appreciate the XML_DIFF module to disapear > and be integrated into one of the other modules. > > In other words, I do not prefer to create new modules for a few classes > that only refer to small general purpose API's like StaX. If a bunch of > classes introduce a dependency to big or specific libs I would take a > new module into account. I think the decision to create a new module does not depend on the quantity of classes introduced, but on the added functionality and it's impact on our users. Best Regards, Lukas > > Regards > Ralf > > > Werner Guttmann schrieb: >> Joachim Grüneis wrote: >> >>> Hello, >>> >>> as for another question of this topic... >>> >>> wouldn't it be good to have the StAX-SAX translation inside the xml module? >>> >> I don't think this is teh right way forward. Castor XML uses SAX only >> for (un-)marshalling, and has its dependencies optimized for this purpose. >> >> We are introducing a new (and OPTIONAL) module that enables pull parsing >> using STaX on top of it. Why force folks to have to 'import' the >> StaX-API dependency when all they want is to use SAX-absed (un)marshalling. >> >> >>> and the rest a new module... >>> I'm just a bit reluctant to still increase the numbers of modules >>> Castor consists of >>> >> Why. It has so many benefits .... which I ma happy to enumerate agin, >> if required. >> >> >>> and besides the translation it would be only very few classes the >>> fragment mapper introduces... >>> the next patch will show this in more detail >>> >>> Joachim >>> >>> 2008/11/13 Joachim Grüneis <[EMAIL PROTECTED]>: >>> >>>> Hello, >>>> >>>> I'm currently converting the ANTLR stuff to JavaCC - to be exact I >>>> first searched for a ready JavaCC solution, which I didn't find, and >>>> now I rewrite the ANTLR grammer to JavaCC to reduce the dependencies >>>> and technologies we use... >>>> >>>> Joachim >>>> >>>> 2008/11/13 Werner Guttmann <[EMAIL PROTECTED]>: >>>> >>>>> Joachim, >>>>> >>>>> any progress on your side ? An opinion ? An updated Maven POM ? Anything >>>>> ? I am sure you have read the mentor agreement you have signed with >>>>> Google, right ? >>>>> >>>>> Werner >>>>> >>>>> Joachim Grüneis wrote: >>>>> >>>>>> 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 >>>>>> >>>>>> >>>>>> >>>>> --------------------------------------------------------------------- >>>>> 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
