[sane-devel] Current TODO list

Henning Meier-Geinitz henning@meier-geinitz.de
Sat, 31 Aug 2002 16:03:23 +0200


On Fri, Aug 30, 2002 at 04:11:39PM -0700, David Paschal wrote:
> Henning Meier-Geinitz wrote:
> >   * All backends should check for TL_X < BR_X and TL_Y < BR_Y to
> >     avoid segfaults or even scanner damage. This should NOT be done
> >     in sane_control_option, it should be possible to temporary set
> >     TL_X > BR_X or TL_ > BR, otherwise it is hard for a frontend to
> >     set the correct values.
> Hi, Henning.  I take it sane_start is the correct place to check for
> this, right?  What is the correct response?  Perhaps it should return
> SANE_STATUS_INVAL, but the SANE standard doesn't list this as a possibility.

True, I didn't realize that SANE_STATUS_INVAL is missing in the list.
I'm using it nevertheless in sane_start. I'm returning that error code
also when a bad handle is used. I think I will put an addition to the
SANE standard on the TODO list.

> Another option, which is what I did in the hpoj backend, would be for the
> backend to keep shadow copies of the "effective" values for these options
> and swap the top and bottom values for X and Y if necessary.

I think that's even better but haven't tried in my backends yet.