Re: [ff-cvs] fenfire/org/fenfire/view/buoy doublegeometer.py

[Matti Katila]

On Tue, 2 Sep 2003, Tuomas Lukka wrote:
> On Mon, Sep 01, 2003 at 12:13:22PM -0400, Matti Katila wrote:
> > -class DoubleGeometer:
> > +class DoubleGeometer(vob.buoy.FocusViewPortsGeometer):
> >      """A class for handling the vertically split view in fenpdf.
> >      """
> >      def __init__(self):
> > @@ -73,7 +73,7 @@
> >      def getSingle(self):
> >     return self.single
> >     
> > -    def place(self, vs):
> > +    def place(self, vs, coordinatesArray):
> >     self.height = vs.size.height
> >     self.width = vs.size.width
> >  
> > @@ -86,5 +86,6 @@
> >  
> >     self._doset(vs)
> >  
> > -   return [(self.mainbox1,), 
> > -           (self.mainbox2,)]
> > +        if len(coordinatesArray) >= 2:
> > +            coordinatesArray[0] = self.mainbox1
> > +            coordinatesArray[1] = self.mainbox2
> 
> Umm, that's a little unstable way of saying it.
> 
> If it's one, the system will crash really strangely.
> Same if it's more than one.

I though that -1 can be given for cs number without fear of crashing?
At least your code does it when in single mode.


   -Matti