[freetds] 0.64 status and coverage

liam@inodes.org liam at inodes.org
Fri Jul 8 12:25:51 EDT 2005


On Thu, Jul 07, 2005 at 01:28:37PM +0200, ZIGLIO, Frediano, VF-IT wrote:

> At http://freetds.sourceforge.net/out/test/out.html you can find tests

Looking at freetds/freetds/src/dblib/unittests/thread.c shouldn't
dbinit() and dbexit() be called only once at the appropriate spot
in main() and not in test()?

Also, do not declare result and thread_count as volatile!

You should wait for the threads to exit with pthread_join() or
alternatively wrap the thread_count 'for' loop in a mutex. You may
not see the updates from other threads otherwise!

Also, as a matter of 'style' I don't think TDS_MUTEX_* functions 
should be used in applications. The caller has pthread support by
virtue of their use and as such, we should not encourage people to
use our internal opaque functions in client applications.

Should I send a patch for the thread.c dblib unit test?

Cheers.


More information about the FreeTDS mailing list