From: nicols <nicolas.mar...@freesurf.fr> >Should we probably change also its name to sthg more explicit, like >PIXMA_CAP_ADF_OR_ADFDUP ?
The "OR" seems a little confusing to me, as if the device could be equipped with one or the other. >BTW, did you have also opportunity to test the iclass_finish_scan() >change proposed for the ?free? issue? Not yet. Everything is working fine right now, so I'll probably get to it during some code cleanup. -Dennis Dennis Lou wrote: > > Regarding the ADF duplex issue, this line looks suspect: > > Index: pixma.c > =================================================================== > RCS file: /cvsroot/sane/sane-backends/backend/pixma.c,v > retrieving revision 1.11 > diff -a -u -r1.11 pixma.c > --- pixma.c 2 Apr 2008 20:13:48 -0000 1.11 > +++ pixma.c 21 Apr 2008 00:53:47 -0000 > @@ -641,7 +641,7 @@ > i++; > } > #if 1 > - if (cfg->cap & PIXMA_CAP_ADFDUP) > + if ((cfg->cap & PIXMA_CAP_ADFDUP) == PIXMA_CAP_ADFDUP) > { > ss->source_list[i] = SANE_I18N ("ADF Duplex"); > ss->source_map[i] = PIXMA_SOURCE_ADFDUP; > > > > -Dennis > > > > -- View this message in context: http://www.nabble.com/Canon-imageClass-MFP%27s-tp16736778p16807484.html Sent from the SANE - Dev mailing list archive at Nabble.com. -- sane-devel mailing list: sane-devel at lists.alioth.debian.org http://lists.alioth.debian.org/mailman/listinfo/sane-devel Unsubscribe: Send mail with subject "unsubscribe your_password" to sane-devel-request at lists.alioth.debian.org ____________________________________________________________________________________ Be a better friend, newshound, and know-it-all with Yahoo! Mobile. Try it now. http://mobile.yahoo.com/;_ylt=Ahu06i62sR8HDtDypao8Wcj9tAcJ