Inconsistent check for symbol SQLITE_DEBUG in amalgamation version 3.16.2

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

Inconsistent check for symbol SQLITE_DEBUG in amalgamation version 3.16.2

Ulrich Telle
In the documentation for the compile time options for debugging

http://sqlite.org/compile.html#debug

the symbol SQLITE_DEBUG is shown without a value. And in most places in
the SQLite source code only the definition status is checked using

#ifdef SQLITE_DEBUG

or

#if defined(SQLITE_DEBUG)

However, at several places the *value* of the symbol is checked using

#if SQLITE_DEBUG

IMHO this is inconsistent with the documentation and probably should be
changed to

#ifdef SQLITE_DEBUG

The following lines are affected in the amalgamation source code of version
3.16.2:

17369
24482
24543
43985
63517
64886
74550
74655
80020
93533
179745

Please consider to fix this (or to adjust the documentation). Thanks.

Regards,

Ulrich

_______________________________________________
sqlite-users mailing list
[hidden email]
http://mailinglists.sqlite.org/cgi-bin/mailman/listinfo/sqlite-users