Re: [sane-devel] More Proposed Upgrades

2015-09-28 Thread Yury Tarasievich
On 09/29/2015 04:25 AM, m. allan noah wrote: I just pushed all of these. I did re-write the microtek2 patch to use the logic I pointed out in a prior mail. /meaning sequenced `% 2` changed to sequenced '!'/ Isn't this fancy-coding now, though? I didn't look in the backend and may have comple

Re: [sane-devel] More Proposed Upgrades

2015-09-28 Thread m. allan noah
I just pushed all of these. I did re-write the microtek2 patch to use the logic I pointed out in a prior mail. allan On Mon, Sep 28, 2015 at 8:39 AM, Olaf Meeuwissen wrote: > Hi All(an), > > I've pushed a few new Proposed Upgrades (PU) to my GitLab clone[1]. > > [1] https://gitlab.com/sane-proj

Re: [sane-devel] Fixing sequencing issues (was Re: Proposed Upgrade (pu) branches on GitLab)

2015-09-28 Thread m. allan noah
On Mon, Sep 28, 2015 at 7:04 AM, Johannes Meixner wrote: > > Hello, > > On Sep 28 18:59 Olaf Meeuwissen wrote (excerpt): >> >> With the sequencing issues, we can only speculate at what >> the intended behaviour is, so we can't really fix things. >> That notwithstanding, I think we can make a decen

[sane-devel] More Proposed Upgrades

2015-09-28 Thread Olaf Meeuwissen
Hi All(an), I've pushed a few new Proposed Upgrades (PU) to my GitLab clone[1]. [1] https://gitlab.com/sane-project/backends/branches I know that there are some restrictions on what can go in at this point but I will leave that to the discretion of whomever merges them. Apart from the first PU

Re: [sane-devel] Fixing sequencing issues (was Re: Proposed Upgrade (pu) branches on GitLab)

2015-09-28 Thread Olaf Meeuwissen
Olaf Meeuwissen writes: > Johannes Meixner writes: > >> Hello, > > Hi Johannes, > >> On Sep 28 18:59 Olaf Meeuwissen wrote (excerpt): >>> With the sequencing issues, we can only speculate at what >>> the intended behaviour is, so we can't really fix things. >>> That notwithstanding, I think we ca

Re: [sane-devel] Fixing sequencing issues (was Re: Proposed Upgrade (pu) branches on GitLab)

2015-09-28 Thread Olaf Meeuwissen
Johannes Meixner writes: > Hello, Hi Johannes, > On Sep 28 18:59 Olaf Meeuwissen wrote (excerpt): >> With the sequencing issues, we can only speculate at what >> the intended behaviour is, so we can't really fix things. >> That notwithstanding, I think we can make a decent, educated >> guess at

Re: [sane-devel] manual "exposure time" in Genesys Backend

2015-09-28 Thread Alexander Hofmann
Hi, Thank's for your answer. Am 26.09.2015 um 18:33 schrieb Stef: > > have you tried to tweak genesys frontend's analog gain ? Exposure > is also related to motor speed, and will be really complicated to set up. Hmm, yea, I was afraid of this - the link between Motor-Drive and Exposure . I d

Re: [sane-devel] Fixing sequencing issues (was Re: Proposed Upgrade (pu) branches on GitLab)

2015-09-28 Thread Johannes Meixner
Hello, On Sep 28 18:59 Olaf Meeuwissen wrote (excerpt): With the sequencing issues, we can only speculate at what the intended behaviour is, so we can't really fix things. That notwithstanding, I think we can make a decent, educated guess at a fix for both cases and make the warning go away :-|

Re: [sane-devel] Fixing sequencing issues (was Re: Proposed Upgrade (pu) branches on GitLab)

2015-09-28 Thread Olaf Meeuwissen
m. allan noah writes: > I think we are all on the same page here regarding clarity of code, > and our concern over warnings. Long term, it would be great if sane > were warning free- but that would require that code get maintainers, > or get ejected. Given that some scanners we support have not b

Re: [sane-devel] Fixing sequencing issues (was Re: Proposed Upgrade (pu) branches on GitLab)

2015-09-28 Thread Olaf Meeuwissen
Johannes Meixner writes: > Hello, > > On Sep 25 21:02 Olaf Meeuwissen wrote (excerpt): >> In the >> mean time, I think we should any code that is flagged as -Wunsequenced >> (by clang) or -Wsequence-point (by gcc). > > I assume the meaning is clear but there is an omission. > What exactly should b

Re: [sane-devel] Proposed Upgrade (pu) branches on GitLab

2015-09-28 Thread Olaf Meeuwissen
m. allan noah writes: > On Thu, Sep 24, 2015 at 9:22 AM, Olaf Meeuwissen > wrote: >> >> Anyway, please consider merging the pu/ branches at my GitLab clone. > > These all made sense to me, so I pushed them up. Thanks for doing the > legwork here. Thanks. I've removed the branches from my GitLab