Re: [Hp5400backend-cvs-update] hp5400/hp5400 hp5400.c,1.16,1.17

[Martijn v. O. <kl...@sv...>]

On Thu, Mar 06, 2003 at 03:03:38AM -0800, sou...@us... wr=
ote:
> Log Message:
> removed black vertical strip for 1200 dpi

Actually, I think a better idea would be: i =3D j / numLoop

This would simply scale the given calibration info to the full width, so the
black edges stay on the edge and don't need any funny values. What you have
below totally breaks it even for < 300dpi scans.

> *** 207,211 ****
>     for( j=3D0; j< numLoop * numCalBlock * CALPIXBYBLOCK ; j++ ) {
>       struct CalPixel *pixel =3D & calinfo[ j / CALPIXBYBLOCK ].pixels[ j=
 % CALPIXBYBLOCK ];
> !     i =3D j % numPixels;
>      =20
>       /* This is obviously not quite right. The values on
> --- 207,211 ----
>     for( j=3D0; j< numLoop * numCalBlock * CALPIXBYBLOCK ; j++ ) {
>       struct CalPixel *pixel =3D & calinfo[ j / CALPIXBYBLOCK ].pixels[ j=
 % CALPIXBYBLOCK ];
> !     i =3D ( j % (int)( 0.80 * numPixels ) ) + (int)(0.10 * numPixels );
>      =20
>       /* This is obviously not quite right. The values on

--=20
Martijn van Oosterhout   <kl...@sv...>   http://svana.org/kleptog/
> Support bacteria! They're the only culture some people have.