[sane-devel] Reflecta ProScan / Crystalscan 7200 PIE film scanner update
Paul Menzel
paulepanter at users.sourceforge.net
Mon Mar 11 14:16:30 UTC 2013
Dear Klaus,
Am Freitag, den 08.03.2013, 17:10 +0100 schrieb Klaus Kaempf:
> * Paul Menzel <paulepanter at users.sourceforge.net> [Mar 04. 2013 10:34]:
> >
> > In my opinion, before committing this upstream you could squash several
> > commits into the original one (as it is not upstream yet). That will
> > keep the commit log cleaner.
>
> sure, no problem. What's the usual process for getting code upstream ?
I am also new to SANE. But sending your patches to the list for review
and providing a branch to pull from is good way I think.
> Are there any coding style guidelines for SANE ?
They are not documented I think. At least I have not found them. You can
only look at already existing files.
> > If you have even more time and as you found some small errors already
> > and fixed them, you could run Cppcheck and splint on your code (and
> > mention that in the commit message).
>
> My primary goal, besides getting running code, is to make the code
> clean and maintanable. These tools will certainly help, thanks !
Awesome!
Thanks,
Paul
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <http://lists.alioth.debian.org/pipermail/sane-devel/attachments/20130311/9e7a1d75/attachment.pgp>
More information about the sane-devel
mailing list