[sane-devel] "hp4200" backend detects non-HP scanners as HP ScanJet 4200

[Henning Meier-Geinitz]

Hi,

On Fri, Sep 05, 2003 at 06:12:56PM +0200, Till Kamppeter wrote:
> The patch I have taken is
> 
> sane-backends-1.0.10-hp4200-0.4.patch.gz
> 
> from
> 
> http://sourceforge.net/projects/hp4200-backend

Ah, ok. I have only checked the hp4200 homepage which had only 1.0.8.

> I hope I did not break anything.

I don't think it's your fault. The 1.0.10 code doesn't seem to have a
check for which scanner is used. Instead in add_device() there is the
following comment:

  /* put here some code to probe that the device attached to the
     device file is a supported scanner. Maybe some ioctl */

So there is no check at all. And still open() is used so I'm pretty
sure it won't work with 1.0.12 anyway. If it works, I'd disable it in
dll.conf or hp4200.conf by default.

This backend needs quite some work to be usable with SANE 1.0.12 in my
opinion.
     
Bye,
  Henning