Test coverage enhancements. Additional documentation detail on the new

sqlite3_log() interface.

FossilOrigin-Name: d986e9289388fd72257b26cb2f9c972177255cd4

diff --git a/manifest b/manifest
index 672b547a755c22a8cd3cbc58f4601b4f5abd80bf..0541a4bac6a33bbad88e0a2555c236825dfce0fb 100644 (file)
--- a/manifest
+++ b/manifest
@@ -1,5 +1,8 @@
-C Add\sexperimental\sfix\sfor\scorruption\sdetection\sproblem.\sThis\smay\swell\sbe\srevised\syet.
-D 2010-02-25T12:09:16
+-----BEGIN PGP SIGNED MESSAGE-----
+Hash: SHA1
+
+C Test\scoverage\senhancements.\s\sAdditional\sdocumentation\sdetail\son\sthe\snew\nsqlite3_log()\sinterface.
+D 2010-02-25T21:27:59
 F Makefile.arm-wince-mingw32ce-gcc fcd5e9cd67fe88836360bb4f9ef4cb7f8e2fb5a0
 F Makefile.in 4f2f967b7e58a35bb74fb7ec8ae90e0f4ca7868b
 F Makefile.linux-gcc d53183f4aa6a9192d249731c90dbdffbd2c68654
@@ -160,13 +163,13 @@ F src/pcache.h c683390d50f856d4cd8e24342ae62027d1bb6050
 F src/pcache1.c 2bb2261190b42a348038f5b1c285c8cef415fcc8
 F src/pragma.c d610d4f4fc55dca58919890fb5323d441b385362
 F src/prepare.c c80630169d82945070a50ebf298611fd9f1485f1
-F src/printf.c f5c160b471e4a4dd22b756cb4ffe0c6979fd6d24
+F src/printf.c aa793722d793520fa31c54d81ddab26b0b410562
 F src/random.c cd4a67b3953b88019f8cd4ccd81394a8ddfaba50
 F src/resolve.c a1648d98e869937b29f4f697461fe4d60f220a7b
 F src/rowset.c 69afa95a97c524ba6faf3805e717b5b7ae85a697
 F src/select.c 4113ef360430ed4e7533690ef46d06c20204adce
 F src/shell.c 31cd555125a1fda4a6d8dce019690f043a8e5d75
-F src/sqlite.h.in a9f51302109f32e3048358122d1e38fd3ead593f
+F src/sqlite.h.in 336ed7bab1080e7e63d5fc48e39bd3fa995da6f0
 F src/sqlite3ext.h 69dfb8116af51b84a029cddb3b35062354270c89
 F src/sqliteInt.h 32e6eb47d2f6fad587e16b0abbbfe3f7d55f2ec0
 F src/sqliteLimit.h 3afab2291762b5d09ae20c18feb8e9fa935a60a6
@@ -214,7 +217,7 @@ F src/vacuum.c deb50c41c39849770ab1bf27e8a35ba1036e3962
 F src/vdbe.c 6de14d95ade1193f2a694ce673f69ac20032fa17
 F src/vdbe.h 471f6a3dcec4817ca33596fe7f6654d56c0e75f3
 F src/vdbeInt.h ae1e6ba0dd3fb4a886898d2829d748be701b01f8
-F src/vdbeapi.c e0398d74af46911033b92088f740582f3c400515
+F src/vdbeapi.c 74c25680046a116b24b95393914d3669c23305dc
 F src/vdbeaux.c 9089e0cdcc7ed3bc4564f2684ce84f5911973ea9
 F src/vdbeblob.c 5327132a42a91e8b7acfb60b9d2c3b1c5c863e0e
 F src/vdbemem.c aeba77b59f3553d3cc5b72c18a8267c6fba546b9
@@ -792,7 +795,14 @@ F tool/speedtest2.tcl ee2149167303ba8e95af97873c575c3e0fab58ff
 F tool/speedtest8.c 2902c46588c40b55661e471d7a86e4dd71a18224
 F tool/speedtest8inst1.c 293327bc76823f473684d589a8160bde1f52c14e
 F tool/vdbe-compress.tcl d70ea6d8a19e3571d7ab8c9b75cba86d1173ff0f
-P 9b18dfd19e825b09c1d246c826e179b9892308df
-R eafa7db6f5fb13e5c1fd5ab7d385d1ec
-U dan
-Z 4907aa0adf782240299c0dcbeaf7f20b
+P 1cc4be7ebc463921827e61da724f0de946c061f6
+R 1c73afd7cbcdebdb258490d2c24799bd
+U drh
+Z 275be1edf4f26aa0d7cebaa1746de83b
+-----BEGIN PGP SIGNATURE-----
+Version: GnuPG v1.4.6 (GNU/Linux)
+
+iD8DBQFLhutjoxKgR168RlERAlrFAJoC7p9doKclaKmbSgjogvOsoOZUUQCeLm7H
+KHa6uF4qzTRfmkYuWOIa0Ao=
+=0tVZ
+-----END PGP SIGNATURE-----

diff --git a/manifest.uuid b/manifest.uuid
index 4335db571648707451fd4c72d64fe8ffd66b22c7..347e5ffb6b4f3f031529d46d20450ac245368542 100644 (file)
--- a/manifest.uuid
+++ b/manifest.uuid
@@ -1 +1 @@
-1cc4be7ebc463921827e61da724f0de946c061f6
\ No newline at end of file
+d986e9289388fd72257b26cb2f9c972177255cd4
\ No newline at end of file

diff --git a/src/printf.c b/src/printf.c
index 081289417ad6ea0f4aac043c17c084ce65d3e980..e67d48ee174b06cb5e5f9c7b92b923d209983ef5 100644 (file)
--- a/src/printf.c
+++ b/src/printf.c
@@ -949,7 +949,7 @@ void sqlite3_log(int iErrCode, const char *zFormat, ...){
   char *zMsg;                             /* Complete log message */
   
   xLog = sqlite3GlobalConfig.xLog;
-  if( xLog && zFormat ){
+  if( xLog ){
     va_start(ap, zFormat);
     sqlite3BeginBenignMalloc();
     zMsg = sqlite3_vmprintf(zFormat, ap);

diff --git a/src/sqlite.h.in b/src/sqlite.h.in
index 37c8df201448ea8674e692bf1b7123690c19e607..178824b8e33d496c034fc2799a1d9f4ccfc12c90 100644 (file)
--- a/src/sqlite.h.in
+++ b/src/sqlite.h.in
@@ -5696,11 +5696,15 @@ int sqlite3_strnicmp(const char *, const char *, int);
 **
 ** ^The [sqlite3_log()] interface writes a message into the error log
 ** established by the [SQLITE_CONFIG_ERRORLOG] option to [sqlite3_config()].
+** ^If logging is enabled, the zFormat string and subsequent arguments are
+** passed through to [sqlite3_vmprintf()] to generate the final output string.
 **
 ** The sqlite3_log() interface is intended for use by extensions such as
 ** virtual tables, collating functions, and SQL functions.  While there is
 ** nothing to prevent an application from calling sqlite3_log(), doing so
 ** is considered bad form.
+**
+** The zFormat string must not be NULL.
 */
 void sqlite3_log(int iErrCode, const char *zFormat, ...);
 

diff --git a/src/vdbeapi.c b/src/vdbeapi.c
index aac898a82ae49852d6f53b21f021ecc521e4c422..125f325d183d4184a95c8d36bb74086fcef5f13b 100644 (file)
--- a/src/vdbeapi.c
+++ b/src/vdbeapi.c
@@ -334,9 +334,7 @@ static int sqlite3Step(Vdbe *p){
   }
 
   if( p->pc<=0 && p->expired ){
-    if( p->rc==SQLITE_OK ){
-      p->rc = SQLITE_SCHEMA;
-    }
+    p->rc = SQLITE_SCHEMA;
     rc = SQLITE_ERROR;
     goto end_of_step;
   }
@@ -435,7 +433,7 @@ int sqlite3_step(sqlite3_stmt *pStmt){
     sqlite3_reset(pStmt);
     v->expired = 0;
   }
-  if( rc2!=SQLITE_OK && v->isPrepareV2 && db->pErr ){
+  if( rc2!=SQLITE_OK && ALWAYS(v->isPrepareV2) && ALWAYS(db->pErr) ){
     /* This case occurs after failing to recompile an sql statement. 
     ** The error message from the SQL compiler has already been loaded 
     ** into the database handle. This block copies the error message