Minor bug in Andor driver

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

Minor bug in Andor driver

Torsten Landschoff
Hi list,

While skimming through the andor driver source code, I found a trivial
bug. Probably due to copy/paste error, both branches of the if
instruction are identical when creating the readout mode property.

I assume that the property should be read only if there is only a single
value. See attached patch (which removes the if alltogether).

Greetings, Torsten

--
Torsten Landschoff | Embedded software and FPGA development
nAmbition GmbH     | Tatzberg 47, 01307 Dresden
                   | HRB NR. 23095 b. Amtsgericht Dresden
Managing director  | Frank Pelzer


------------------------------------------------------------------------------
Come build with us! The BlackBerry® Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay
ahead of the curve. Join us from November 9-12, 2009. Register now!
http://p.sf.net/sfu/devconf
_______________________________________________
micro-manager-general mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/micro-manager-general

andor_readoutmode.diff (902 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: Minor bug in Andor driver

Nico Stuurman
Administrator
> While skimming through the andor driver source code, I found a trivial
> bug. Probably due to copy/paste error, both branches of the if
> instruction are identical when creating the readout mode property.

Thanks for noticing and reporting!  Your patch is applied.

Best,

Nico


------------------------------------------------------------------------------
Come build with us! The BlackBerry® Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay
ahead of the curve. Join us from November 9-12, 2009. Register now!
http://p.sf.net/sfu/devconf
_______________________________________________
micro-manager-general mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/micro-manager-general