DO NOT REPLY [Bug 32597] - modified selector attribute "update=fase" doesn't work

[bu...@apache.org]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG�
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=32597>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND�
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=32597


jan@materne.de changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|dev@ant.apache.org          |jan@materne.de




------- Additional Comments From jan@materne.de  2004-12-09 09:00 -------
I could verify that.

1. update="false" should not change the cache file. That is the intention.
2. A very small testcase
   <fileset dir="."><modified update="false"/></fileset>
   has created the cache file, but shouldnt.
   A first quick view into the source [1] has shown

public boolean isSelected(...) {
    ...
    // Maybe update the cache
    if (update && rv) {
        cache.put(f.getAbsolutePath(), newValue);
        setModified(getModified() + 1);
        if (!getDelayUpdate()) {
            saveCache();
        }
    }
    ...
}

But maybe the error is somewhere deeper. I have to have a deeper look into that 
(and prepare a JUnit testcase of course :)



[1] 
http://cvs.apache.org/viewcvs.cgi/ant/src/main/org/apache/tools/ant/types/select
ors/modifiedselector/ModifiedSelector.java?rev=1.14&view=markup

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
For additional commands, e-mail: dev-help@ant.apache.org