Re: [Gzz-commits] fenfire/org/fenfire/view/buoy MainNode2D.java

[Tuomas Lukka]

On Thu, May 08, 2003 at 08:26:45PM +0300, Matti Katila wrote:
> On Thu, 8 May 2003, Tuomas Lukka wrote:
> > On Wed, May 07, 2003 at 12:04:54PM -0400, Matti Katila wrote:
> >> Modified files:
> >>    org/fenfire/view/buoy: MainNode2D.java 
> >> Log message:
> >>    check for check
> >>  
> >> -  if (RDFUtil.isNodeType(fen, plane, PP.Paper) ||
> >> -      RDFUtil.isNodeType(fen, plane, RST.Canvas)) ;
> >> -  else throw new Error("plane is NOT a container!");
> >> +  if (fen != null)
> >> +            if (RDFUtil.isNodeType(fen, plane, PP.Paper) ||
> >> +                RDFUtil.isNodeType(fen, plane, RST.Canvas)) ;
> >> +            else throw new Error("plane is NOT a container!");
> >>  
> >>    view2d.render(vs, plane, into, into, box2paper);
> > 
> > I think this is probably the wrong fix: you should rather see *why* fen
> > might be null...
> 
> Yes, check was in wrong place firstly. I moved it in CanvasView2D where it 
> belongs.

Great, the next patch looks good.

        Tuomas