Add testcase() macros to make sure boundary cases in BEFORE UPDATE triggers

are well tested.

FossilOrigin-Name: 27175caa28919fe145d369f27f52346b8bbadb1e

diff --git a/manifest b/manifest
index 17c2eb545b379c403dfbb2cb2284379c19772d0f..3b5a32ed19141f1830c2e46d309e48c120200424 100644 (file)
--- a/manifest
+++ b/manifest
@@ -1,8 +1,8 @@
 -----BEGIN PGP SIGNED MESSAGE-----
 Hash: SHA1
 
-C Fix\sa\sfaulty\sfunction\sprototype\sin\stest_intarray.c.
-D 2009-12-01T18:46:07
+C Add\stestcase()\smacros\sto\smake\ssure\sboundary\scases\sin\sBEFORE\sUPDATE\striggers\nare\swell\stested.
+D 2009-12-01T22:09:25
 F Makefile.arm-wince-mingw32ce-gcc fcd5e9cd67fe88836360bb4f9ef4cb7f8e2fb5a0
 F Makefile.in c5827ead754ab32b9585487177c93bb00b9497b3
 F Makefile.linux-gcc d53183f4aa6a9192d249731c90dbdffbd2c68654
@@ -208,7 +208,7 @@ F src/test_thread.c 00fed80690ae7f1525483a35861511c48bc579f2
 F src/test_wsd.c 41cadfd9d97fe8e3e4e44f61a4a8ccd6f7ca8fe9
 F src/tokenize.c 9f7d39da1a1346fa0cf106ea0bf10bb6b8b61ddf
 F src/trigger.c d46f9389e3bf3dd1cc1d288aba2f289c96b34200
-F src/update.c edf5649ffc9fca6d6884e34b6da6358e724919fd
+F src/update.c c0dc6b75ad28b76b619042d934f337b02acee208
 F src/utf.c dad16adcc0c35ef2437dca125a4b07419d361052
 F src/util.c ad4f03079ba0fe83590d1cc9197e8e4844e38592
 F src/vacuum.c 03309a08d549f9389cc3a3589afd4fadbdaf0679
@@ -778,14 +778,14 @@ F tool/speedtest2.tcl ee2149167303ba8e95af97873c575c3e0fab58ff
 F tool/speedtest8.c 2902c46588c40b55661e471d7a86e4dd71a18224
 F tool/speedtest8inst1.c 293327bc76823f473684d589a8160bde1f52c14e
 F tool/vdbe-compress.tcl d70ea6d8a19e3571d7ab8c9b75cba86d1173ff0f
-P 7737db490ceba02c29c36fe181b4e7895b40aa75
-R 5025e837faa6899de52099ca849ef8ea
+P 110956a6505a9f17d08142d47003cf3bc4e515d3
+R f7247a9524ee2d81021437c8fa2ecf38
 U drh
-Z 838cf1d6597aa02e40799aec47ba0584
+Z 1559414c8fea985346b66bc73803caef
 -----BEGIN PGP SIGNATURE-----
 Version: GnuPG v1.4.6 (GNU/Linux)
 
-iD8DBQFLFWRyoxKgR168RlERAnXoAJ9J1Nx/oh+3EzCsyEt/PQJ0DgWIBgCbB+h4
-fTph1NV8X0DU452xgPJaOXw=
-=aE0n
+iD8DBQFLFZQYoxKgR168RlERAp1sAJ9Du7y1J7wA/qkWiRYSTIIzGUs3CQCfbVt1
+9eUdCszKYUvVc/2XRoho8c0=
+=wGuC
 -----END PGP SIGNATURE-----

diff --git a/manifest.uuid b/manifest.uuid
index f65e2eab8d974275da38142dc7d2edc103e086bf..907debfa74840bb67a8acec34f41202020a229e3 100644 (file)
--- a/manifest.uuid
+++ b/manifest.uuid
@@ -1 +1 @@
-110956a6505a9f17d08142d47003cf3bc4e515d3
\ No newline at end of file
+27175caa28919fe145d369f27f52346b8bbadb1e
\ No newline at end of file

diff --git a/src/update.c b/src/update.c
index c927059aa5888101e241f9df3f3f283379779f55..66bf8ca9ffdc44412ced0cff3b0f492e99d4d1e7 100644 (file)
--- a/src/update.c
+++ b/src/update.c
@@ -119,7 +119,7 @@ void sqlite3Update(
   Trigger *pTrigger;     /* List of triggers on pTab, if required */
   int tmask;             /* Mask of TRIGGER_BEFORE|TRIGGER_AFTER */
 #endif
-  int newmask;
+  int newmask;           /* Mask of NEW.* columns accessed by BEFORE triggers */
 
   /* Register Allocations */
   int regRowCount = 0;   /* A count of rows changed */
@@ -436,6 +436,8 @@ void sqlite3Update(
         ** if there are one or more BEFORE triggers that use this value via
         ** a new.* reference in a trigger program.
         */
+        testcase( i==31 );
+        testcase( i==32 );
         sqlite3VdbeAddOp3(v, OP_Column, iCur, i, regNew+i);
         sqlite3ColumnDefault(v, pTab, i, regNew+i);
       }