Login | Register
My pages Projects Community openCollabNet

Discussions > issues > [Issue 320] New - FigEdge children handled inconsistently for drawing

gef
Discussion topic

Back to topic list

[Issue 320] New - FigEdge children handled inconsistently for drawing

Reply

Author tfmorris
Full name Tom Morris
Date 2008-10-14 14:28:25 PDT
Message http://gef.tigris.or​g/issues/show_bug.cg​i?id=320
                 Issue #|320
                 Summary|FigEdge children handled inconsistently for drawing
               Component|gef
                 Version|current
                Platform|All
                     URL|
              OS/Version|All
                  Status|NEW
       Status whiteboard|
                Keywords|
              Resolution|
              Issue type|DEFECT
                Priority|P3
            Subcomponent|core
             Assigned to|issues@gef
             Reported by|tfmorris






------- Additional comments from tfmorris at tigris dot org Tue Oct 14 14:28:25 -0700 2008 -------
See ArgoUML issue 5460. The following code from FigEdge.intersects()

        for(int i = 0; i < size; i++) {
            Fig f = ((PathItem)_pathItem​s.elementAt(i)).getF​ig();
            //only pathitems represented in a layer (i.e. being displayed) are
of interest
            if(f.getLayer() != null && f.intersects(r)) {
                //System.out.println​("Intersects");
                return true;
            }
        }

will fail to draw child Figs which don't have their layer set, but there must be
other places in the code which don't enforce this behavior, because they do
often get drawn.

The handling should be consistent. Since the drawing code is driven by the
active layer and sub Figs should all be in the same layer as the parent fig, it
seems unnecessary (and needlessly expensive) to be checking for a null layer.

« Previous message in topic | 1 of 7 | Next message in topic »

Messages

Show all messages in topic

[Issue 320] New - FigEdge children handled inconsistently for drawing tfmorris Tom Morris 2008-10-14 14:28:25 PDT
     [Issue 320] FigEdge children handled inconsistently for drawing bobtarling Bob Tarling 2008-10-26 17:31:09 PDT
     [Issue 320] FigEdge.intersects broken - causes redraw failures tfmorris Tom Morris 2008-11-16 08:27:13 PST
     [Issue 320] FigEdge.intersects broken - causes redraw failures dthompson Dave Thompson 2008-11-16 10:50:00 PST
     [Issue 320] FigEdge.intersects broken - causes redraw failures tfmorris Tom Morris 2008-11-17 07:48:53 PST
     [Issue 320] FigEdge.intersects broken - causes redraw failures bobtarling Bob Tarling 2008-11-19 17:32:16 PST
     [Issue 320] FigEdge.intersects broken - causes redraw failures bobtarling Bob Tarling 2008-11-23 07:30:22 PST
Messages per page: