JC created FOP-2732:

             Summary: Collection added to itself
                 Key: FOP-2732
                 URL: https://issues.apache.org/jira/browse/FOP-2732
             Project: FOP
          Issue Type: Bug
            Reporter: JC
            Priority: Trivial


In a recent github mirro, I've found the following suspicious code.
Branch: master

 39     protected List objects = new java.util.ArrayList();
 86     public void addAll(AbstractGraphicsDrawingOrderContainer 
graphicsContainer) {
 87         Collection objects = graphicsContainer.getObjects();
 88         objects.addAll(objects);
 89     }

In Line 88, should `objects.addAll' be `*this*.objects.addAll? Since this 
method in an abstract class, it might not be an issue if this method is 
overridden. However, I wanted to report this just in case.


This message was sent by Atlassian JIRA

Reply via email to