<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">2016-03-10 13:13 GMT+01:00 Maksim Ivanov <span dir="ltr"><<a href="mailto:emaxx@google.com" target="_blank">emaxx@google.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hello all,<br></blockquote><div><br></div><div>Hello Maksim,<br> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
I have a couple of small suggestions regarding the tracing feature of<br>
the PC/SC-Lite's client side library. This feature is controlled by<br>
the "DO_TRACE" preprocessor definition.<br>
<br>
First, it's impossible to trigger this feature using the compiler<br>
flags: for some reason, the winscard_clnt.c file contains directive<br>
"#undef DO_TRACE". If it were a commented "#define" directive, then<br>
this would be more useful: the feature could be triggered both by<br>
editing the source file and by modifying the compiler flags.<br></blockquote><div><br></div><div>Fixed in ea0a50bf2088906d888cf1888b4439f3391eb111<br><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Second, there is an inconsistency between the printf format specifier<br>
and the passed value in the trace function, that produces a warning.<br>
E.g. with clang 3.7.0:<br>
<br>
> pcsc-lite/src-1.8.15/src/winscard_clnt.c:162:14: error: format specifies type 'unsigned long' but the argument has type 'pthread_t'<br>
>       (aka 'struct __nc_basic_thread_data *') [-Werror,-Wformat]<br>
>                 direction, pthread_self(), func);<br>
>                            ^~~~~~~~~~~~~~<br></blockquote><div><br></div><div>Can you propose a patch for that?<br></div></div><br>pthread_t may not be a numerical value. It may be a structure.<br><a href="http://stackoverflow.com/questions/1759794/how-to-print-pthread-t">http://stackoverflow.com/questions/1759794/how-to-print-pthread-t</a><br clear="all"><br></div><div class="gmail_extra">Regards,<br><br></div><div class="gmail_extra">-- <br><div class="gmail_signature"> Dr. Ludovic Rousseau</div>
</div></div>