http://codereview.appspot.com/153054/diff/1003/5
File src/com/google/caja/tools/AbstractCajaAntTask.java (right):
http://codereview.appspot.com/153054/diff/1003/5#newcode238
src/com/google/caja/tools/AbstractCajaAntTask.java:238: List<File>
getFiles() {
this is going to put all <file>s before all <fileset>s
so it's going to be hard to do something like
<file file="header"/>
<fileset>...</fileset>
<file file="footer"/>
I think you need to expand the fileset at the time it's declared.
http://codereview.appspot.com/153054/diff/1003/5#newcode244
src/com/google/caja/tools/AbstractCajaAntTask.java:244: //
TODO(mikesamuel): impose some ordering within a group
how about always sort the results by name?
sorting was the main reason I got lost when trying to do this earlier.
filesets are normally unordered, and the way to order them
is to enclose them in a <sort>, which means dealing with
ant's resource collections, not just the "legacy" filesets.
but for this use, I don't think there's a reason to do
anything other than sort by name.
http://codereview.appspot.com/153054
