Feature Requests item #501598, was opened at 2002-01-09 18:42
You can respond by visiting:
http://sourceforge.net/tracker/?func=detail&atid=402707&aid=501598&group_id=31602
Category: None
Group: None
Status: Open
Priority: 5
Submitted By: Fred Loney (loney)
Assigned to: Nobody/Anonymous (nobody)
Summary: Ignore whitespace before block tag
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 10: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.
----------------------------------------------------------------------
You can respond by visiting:
http://sourceforge.net/tracker/?func=detail&atid=402707&aid=501598&group_id=31602
_______________________________________________
Xdoclet-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/xdoclet-devel
