Hi, This is out of date for me... It is the only one, the others are really solved :) Vincent
-----Original Message----- From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED]] On Behalf Of Andrew Stevens Sent: vendredi 8 mars 2002 23:02 To: [EMAIL PROTECTED] Subject: Re: [Xdoclet-devel] [ xdoclet-Feature Requests-501598 ] Ignore whitespace before b A wise old hermit known only as <[EMAIL PROTECTED]> once said: > Feature Requests item #501598, was opened at 2002-01-10 03:42 You can > respond by visiting: > http://sourceforge.net/tracker/?func=detail&atid=402707&aid=501598&gro > up > _id=31602 > > Category: None > Group: None > >Status: Closed > Priority: 5 > Submitted By: Fred Loney (loney) > Assigned to: Vincent Harcq (vharcq) > Summary: Ignore whitespace before block tag Is this one implemented, Vincent, or did you just close it because there was no response? (it affects whether I need to mention it in changes.txt) Likewise for those other FRs you closed. Andrew. > Initial Comment: > If a block tag is preceded by a linebreak and > whitespace, then that linebreak and whitespace should > not be printed. E.g. for template fragment: > > <XDoclet:forAllPersistentFields> > <XDoclet:ifIsAggregate> > // This is an aggregate > </XDoclet:ifIsAggregate> > </XDoclet:forAllPersistentFields> > > xdoclet should not print an extraneous blank line for > a persistent field if that field is not an aggregate. > > This convention is simple to understand and > appreciate, and leads to more legible templates and > output. This is esp. true for XML output, where blank > lines can't be intelligently factored out by a > reformatting utility. > > I have coded and tested a change to the 1.0.1 > TemplateEngine.java that makes this work, but the > extensive CVS changes to TemplateEngine don't > accomodate the change. If this change is desirable, > then I can make a compatible change and submit it to > the assigned CVS updater. > > > ---------------------------------------------------------------------- > > Comment By: Vincent Harcq (vharcq) > Date: 2002-01-25 19:30 > > Message: > Logged In: YES > user_id=125677 > > This kind of added blanks appears in different places. > Does your patch works in all cases ? > The rule was also to let Pretty do this kind of job. For > example > if (...) { > } > instead of > if (...) > { > } > > IMHO these things will never be possible through the kind > of patch you have. > Anyway attach it here I will commit it. _______________________________________________ Xdoclet-devel mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/xdoclet-devel _______________________________________________ Xdoclet-devel mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/xdoclet-devel
