[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Gzz-commits] fenfire/org/fenfire/view/buoy MainNode2D.java
From: |
Tuomas Lukka |
Subject: |
Re: [Gzz-commits] fenfire/org/fenfire/view/buoy MainNode2D.java |
Date: |
Fri, 9 May 2003 14:39:40 +0300 |
User-agent: |
Mutt/1.4.1i |
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