Change an assert() added by the previous commit into a testcase().

Fix a separate assert() in btree.c:freeSpace().

FossilOrigin-Name: fe51d3aa0ce7021213293a0647e31164073e78f7

diff --git a/manifest b/manifest
index 6883385671231f6511c83fc4ec10a4dbdf82b90b..7a156440dc5f9f625c4929af12eea2f8e99f9dd3 100644 (file)
--- a/manifest
+++ b/manifest
@@ -1,5 +1,5 @@
-C Further\ssize\sreduction\sand\sperformance\simprovement\sin\sbtree.c:allocateSpace().
-D 2014-08-20T11:56:14.338
+C Change\san\sassert()\sadded\sby\sthe\sprevious\scommit\sinto\sa\stestcase().\nFix\sa\sseparate\sassert()\sin\sbtree.c:freeSpace().
+D 2014-08-20T13:17:43.884
 F Makefile.arm-wince-mingw32ce-gcc d6df77f1f48d690bd73162294bbba7f59507c72f
 F Makefile.in 5eb79e334a5de69c87740edd56af6527dd219308
 F Makefile.linux-gcc 91d710bdc4998cb015f39edf3cb314ec4f4d7e23
@@ -167,7 +167,7 @@ F src/auth.c 523da7fb4979469955d822ff9298352d6b31de34
 F src/backup.c a729e63cf5cd1829507cb7b8e89f99b95141bb53
 F src/bitvec.c 19a4ba637bd85f8f63fc8c9bae5ade9fb05ec1cb
 F src/btmutex.c 976f45a12e37293e32cae0281b15a21d48a8aaa7
-F src/btree.c 398ecbdb4a19230940955c10b4f35de958e9a05c
+F src/btree.c 53eb576f72093d5138f5b7468994121c6eb6fe98
 F src/btree.h 4245a349bfe09611d7ff887dbc3a80cee8b7955a
 F src/btreeInt.h cf180d86b2e9e418f638d65baa425c4c69c0e0e3
 F src/build.c 5abf794fe8a605f2005b422e98a3cedad9b9ef5b
@@ -1186,7 +1186,7 @@ F tool/vdbe_profile.tcl 67746953071a9f8f2f668b73fe899074e2c6d8c1
 F tool/warnings-clang.sh f6aa929dc20ef1f856af04a730772f59283631d4
 F tool/warnings.sh 0abfd78ceb09b7f7c27c688c8e3fe93268a13b32
 F tool/win/sqlite.vsix deb315d026cc8400325c5863eef847784a219a2f
-P 6c66beae97ba1799c908d3a33371dedbc7f3f58c
-R 28706ecaca72a1e3574ba7b8d51854c6
+P 1cb1cd64930a6bc371143f7d2e77eb1c51498cb0
+R 2da2148faf73b1d43a764ae813ef9e60
 U drh
-Z ca9532aeb7742cb91c75177cb9ee4fbc
+Z 36b8d62cf946d2fc64ee098831079eb8

diff --git a/manifest.uuid b/manifest.uuid
index d1bf93d7a5f4807f5efb5ac0a11a267ae14fd1cb..4a04c24fb14c8aa390eb36f24d74b053c83e85ac 100644 (file)
--- a/manifest.uuid
+++ b/manifest.uuid
@@ -1 +1 @@
-1cb1cd64930a6bc371143f7d2e77eb1c51498cb0
\ No newline at end of file
+fe51d3aa0ce7021213293a0647e31164073e78f7
\ No newline at end of file

diff --git a/src/btree.c b/src/btree.c
index 61848dcb838555fad972c845826b9e3ce1a2ea10..a09b82fe5e44b395cdf2a155f3e30002b978c90f 100644 (file)
--- a/src/btree.c
+++ b/src/btree.c
@@ -1267,7 +1267,7 @@ static int allocateSpace(MemPage *pPage, int nByte, int *pIdx){
   testcase( gap+2+nByte==top );
   if( gap+2+nByte>top ){
 defragment_page:
-    assert( pPage->nCell>0 );
+    testcase( pPage->nCell==0 );
     rc = defragmentPage(pPage);
     if( rc ) return rc;
     top = get2byteNotZero(&data[hdr+5]);
@@ -1306,7 +1306,7 @@ static int freeSpace(MemPage *pPage, int start, int size){
   assert( start>=pPage->hdrOffset+6+pPage->childPtrSize );
   assert( (start + size) <= (int)pPage->pBt->usableSize );
   assert( sqlite3_mutex_held(pPage->pBt->mutex) );
-  assert( size>=0 );   /* Minimum cell size is 4 */
+  assert( size>=4 );   /* Minimum cell size is 4 */
 
   if( pPage->pBt->btsFlags & BTS_SECURE_DELETE ){
     /* Overwrite deleted information with zeros when the secure_delete