https://issues.apache.org/bugzilla/show_bug.cgi?id=54151

          Priority: P2
            Bug ID: 54151
          Assignee: [email protected]
           Summary: wasted work in FailureRecorder.setProject()
          Severity: normal
    Classification: Unclassified
                OS: Linux
          Reporter: [email protected]
          Hardware: PC
            Status: NEW
           Version: 1.8.4
         Component: Core
           Product: Ant

Created attachment 29602
  --> https://issues.apache.org/bugzilla/attachment.cgi?id=29602&action=edit
patch

The problem appears in version 1.8.4 and in revision 1409946.  I
attached a one-line patch that fixes it.

In method "FailureRecorder.setProject", the loop over "allListeners"
should call "break" (not "continue", as it does currently) immediately
after "alreadyRegistered" is set to "true".  All the iterations after
"alreadyRegistered" is set to "true" do not perform any useful work,
at best they just set "alreadyRegistered" again to "true".

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

Reply via email to