[sane-devel] Current TODO list
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.