[sane-devel] New magicolor backend for inclusion in git

[Adrian Glaubitz]

Damn, I should have figured out the problem and written a patch yesterday
and I could have had my first contribution to SANE already :(. I actually
stumbled across this already yesterday when playing around with the
HP2410 code in the genesys backend.

But anyway, I hope it gets fixed anyway  ;).

Adrian

On Jan 20, 2011, at 8:11 PM, Brian Shaver wrote:

> I can confirm this change also eliminates my compilation error. I'm running Fedora 13 64-bit on Intel Core2 T5500 CPU.
> 
> Thanks,
> Brian ..
> 
> On Thu, Jan 20, 2011 at 12:13 PM, Adrian Glaubitz <glaubitz at physik.fu-berlin.de> wrote:
> Hi Allan,
> 
> On Jan 20, 2011, at 6:00 PM, m. allan noah wrote:
> 
> > Hmm- looks like *p1 should not be void ptr. Try this change:
> >
> > sane-backends/backend/magicolor.c line 672
> >
> > void *p1;
> >
> > should be
> >
> > unsigned char * p1;
> >
> > perhaps?
> 
> Indeed, that fixes the problem along with a second change in the same
> backend. I generated a patch so you can see what I actually changed.
> 
> See attached.
> 
> Adrian
>