[sane-devel] Patch for sane-fujitsu: Fixes fi-5110eox ignoring page-width

spammed spammed at get2net.dk
Fri Jul 11 20:25:15 UTC 2008


Here is a small fix for this scanner's page-width problem (diff against
fujitsu.c from CVS). In the set_window function, if the model_name is a
match, it calculates a horizontal offset to center the scanning area. It
has been tested with a range of modes, paper sizes, and tl and br
settings, without unpleasant surprises... 

Some additional feedback on the current 1.1.0-code with this scanner:

- The new option --emphasis works as advertised.

- The double-feed options --dfdetect and --dfdiff have been replaced by
a new one, --df-action, which is good, because it is now much clearer
what the scanner can do. But it seems to be ignored at this point.
(Double-feeds are not detected. Work in progress?) Previously
double-feed detection worked, if only for --dfdetect=Length and ignoring
--dfdiff.

- If --df-action is eventually fixed, then --overscan is the only option
which is exposed by the backend for this scanner, but which does not
actually work. Maybe the option could be hidden when the scanner returns
(os_x_basic, os_y_basic) = (0, 0), as this scanner does? Interestingly,
the paper guides limit page width to 220 mm, so there is 2-3 mm of
unused CCD on each side - presumably for overscan, but it appears to be
not implemented, or maybe deliberately disabled for this scanner :-(

- The scanner has a scan-button, which is not mentioned in the sensors
section of the output of scanimage --help -d fujitsu. Should it be?

Best regards, 
Jacob Nielsen
 


-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.alioth.debian.org/pipermail/sane-devel/attachments/20080711/a60d01ce/attachment.htm 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: pagewidth-patch.diff
Type: text/x-patch
Size: 719 bytes
Desc: not available
Url : http://lists.alioth.debian.org/pipermail/sane-devel/attachments/20080711/a60d01ce/attachment.bin 


More information about the sane-devel mailing list