DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=18166>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND
INSERTED IN THE BUG DATABASE.
http://nagoya.apache.org/bugzilla/show_bug.cgi?id=18166
Concat enhancement
------- Additional Comments From peter.reilly@corvil.com 2003-03-25 13:05 -------
Cool... Thanks for the feedback.
Ok i had a quick look at the list and the code.
>> * I've reverted parts of your patch that contained removal of comments
and the addition of wildcard imports.
Cool, I had attempted a merge between my concat2 and concat, but missed
these.
>> * you completely ignore all <fileset>'s and so on if nested text is present.
I think this is the original behaviour of concat. - it throws a buildexception.
>> * encoding is ignored when reading from files.
This is a bug I introduced when fixing the multi file bug, I
can attach a new concat file, if needed.
>> * The handling of header and footer would reintroduce the bug where a newline
incorrectly gets appended - I've sort of fixed that by using print instead
of println.
This is true, especially since I added the file attribute. I'll check
to see what effect this has on my build scripts.
>> * I don't think TextElement#setFile should ignore that the referenced file
doesn't exist.
This is true, I sort of put setFile in as a last-minute idea
>> * The trim attribute in TextElement should apply to nested text as well (instead
of unconditionally trimming it.
It only looks like it unconditionally trims it.
>> * We might need the sanitizeText logic for header and footer as well.
The line (if (value.trim().length() == 0) sort of does this.
>> * What do you need trimleading for?
This is in to make the xml look nice...
The useage is like this:
<header stripleading="true"><![CDATA[
#include <common/unittest.h>
#include "register_suites.h"
void register_suites() {
]]></header>
I want the output to be:
#include <common/unittest.h>
#include "register_suites.h"
void register_suites() {
>> * Why do you apply filterchains to nested text but not to header and footer?
I use this task generate header and c files like this:
<property name="suite.pat" value="^ *SUITE\(.*,\s*(.*)\s*\).*"/>
<concat destfile="${gen.dir}/register_suites.h" overwrite="no">
<header trimleading="true">
#ifndef R
#define R
</header>
<fileset dir="test/unit" includes="**/unit_*.cpp"/>
<filterchain>
<tokenfilter>
<containsregex
pattern="${suite.pat}"
replace="void register_\1();"/>
</tokenfilter>
</filterchain>
<footer trimleading="true">
void register_suites();
#endif
</footer>
</concat>
>> * changed semantics of the append attribute
I am not sure I changed the semantics. I found the original code
and explaination confusing.
Also: I dropped a public method "reset", this has been reintroduced
Will we need to modify it to to reset the new fields of concat?
Peter.
|