https://bz.apache.org/bugzilla/show_bug.cgi?id=65424

Jaikiran Pai <jaiki...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |1.10.12
         Resolution|---                         |FIXED
             Status|NEW                         |RESOLVED

--- Comment #4 from Jaikiran Pai <jaiki...@apache.org> ---
I wanted to include this fix in an upcoming release of Ant 1.10.x. So I went
ahead and used this patch (and modified it a bit) and committed it to upstream.

Stefan, it would be good to have your review on the commit I did. Overall, the
patch is same as what is proposed here with some minor changes:
 - I moved the project check to the start of the method (just to make it clear
that we don't cache the helper in such cases)
 - added some code comments
 - Removed the "synchronized" on clearCache() since it's no longer needed
because we are now using the lock on the "HELPERS" instance instead of on the
"IntrospectionHelper".

ED-Apache, I'm not sure if you are a first time contributor to Ant or if you
have contributed previously. If this is your first time, please let us know
what name (first name, last name) you would like us to add in our contributors
list that we maintain in the project.

-- 
You are receiving this mail because:
You are the assignee for the bug.

Reply via email to