On Apr 2, 2010, at 15:43 , Andreas Jellinghaus wrote:
> I don't understand why you changed the default value of verbose
> from "0" to "-1". it only leeds to more complex code paths in many
> places.
> 
> if you don't want to verwrite ctx->debug (fine with me), a
> single line change like this:
> -     ctx->debug = verbose;
> +     if (verbose) ctx->debug = verbose;
> would be enough.
> 
> did I miss some subtil reason for the "-1" default?

Probably not. The same result but a more difficult implementation, can't 
remember why I made it so.

Will change.

-- 
Martin Paljak
http://martin.paljak.pri.ee
+3725156495

_______________________________________________
opensc-devel mailing list
opensc-devel@lists.opensc-project.org
http://www.opensc-project.org/mailman/listinfo/opensc-devel

Reply via email to