Re: Octave/backend interface proposal

[Muthiah Annamalai]

> From: "Michael Goffioul" <address@hidden>
> Subject: Re: Octave/backend interface proposal
> To: "Shai Ayal" <address@hidden>
> Cc: Octave Maintainers List <address@hidden>
> Message-ID:
>         <address@hidden>
> Content-Type: text/plain; charset="iso-8859-1"
>
> On Jan 21, 2008 9:58 AM, Michael Goffioul <address@hidden> wrote:
>   
> > I also think that putting this class into a separated header will get me
> > into trouble, because of the classes dependencies. I think I'll put it
> > into graphics.h.in and generate a new patch. Any objection?
> >     
>
> Here's a completed (working) version of the patch. It defines the
> base_graphics_backend
> and graphics_backend classes, associate a backend with a figure, defines the
> default gnuplot backend and move drawnow to C++.
>
> I tested it and it seems to work OK.
>
> Michael.
>   
There maybe a possible oversight in your patch in the following lines,
from the function drawnow

+         if (h.ok () && h != 0)

you may have intended the checks the other way.

-Muthu