Remove a branch made redundant by the earlier exclusive-mode changes.

FossilOrigin-Name: c501b2ede6aad123bef0aa7ce8b356a134eb6d26

diff --git a/manifest b/manifest
index 2f699f4bfb4cba9ceabfdf997f9eb14e8a071c5b..81d4c726f0d9377b680af9d9b0437615b0853f31 100644 (file)
--- a/manifest
+++ b/manifest
@@ -1,5 +1,5 @@
-C Have\sos_unix.c\suse\sthe\ssuffix\s"-wal-index"\sfor\swal-index\sfiles\sinstead\sof\s"-wal-inde".
-D 2010-05-11T16:29:55
+C Remove\sa\sbranch\smade\sredundant\sby\sthe\searlier\sexclusive-mode\schanges.
+D 2010-05-12T06:54:32
 F Makefile.arm-wince-mingw32ce-gcc fcd5e9cd67fe88836360bb4f9ef4cb7f8e2fb5a0
 F Makefile.in a5cad1f8f3e021356bfcc6c77dc16f6f1952bbc3
 F Makefile.linux-gcc d53183f4aa6a9192d249731c90dbdffbd2c68654
@@ -224,7 +224,7 @@ F src/vdbeblob.c 5327132a42a91e8b7acfb60b9d2c3b1c5c863e0e
 F src/vdbemem.c 2a82f455f6ca6f78b59fb312f96054c04ae0ead1
 F src/vdbetrace.c 864cef96919323482ebd9986f2132435115e9cc2
 F src/vtab.c a0f8a40274e4261696ef57aa806de2776ab72cda
-F src/wal.c 7042647fd4c89b789da6dc934550effdf573a290
+F src/wal.c 2f747b6a6bfd9b75b837a1e31176235a21342e0f
 F src/wal.h 32f36b2a827b78373658dac5521291485dfa52b6
 F src/walker.c 3112bb3afe1d85dc52317cb1d752055e9a781f8f
 F src/where.c 75fee9e255b62f773fcadd1d1f25b6f63ac7a356
@@ -762,7 +762,7 @@ F test/vtab_alter.test 9e374885248f69e251bdaacf480b04a197f125e5
 F test/vtab_err.test 0d4d8eb4def1d053ac7c5050df3024fd47a3fbd8
 F test/vtab_shared.test 0eff9ce4f19facbe0a3e693f6c14b80711a4222d
 F test/wal.test 7a100918c45872fa19cfb4413299b9afb08727b6
-F test/wal2.test 913fc65e533593e3b5dfb193340ac32368da1914
+F test/wal2.test 2eba114ff58f0688278ad45249625af89a0bc9fa
 F test/walbak.test a0e45187c7d8928df035dfea29b99b016b21ca3c
 F test/walcrash.test f6d5fb2bb108876f04848720a488065d9deef69f
 F test/walfault.test 98df47444944a6db2161eed5cef71d6c00bcb8c3
@@ -813,7 +813,7 @@ F tool/speedtest2.tcl ee2149167303ba8e95af97873c575c3e0fab58ff
 F tool/speedtest8.c 2902c46588c40b55661e471d7a86e4dd71a18224
 F tool/speedtest8inst1.c 293327bc76823f473684d589a8160bde1f52c14e
 F tool/vdbe-compress.tcl d70ea6d8a19e3571d7ab8c9b75cba86d1173ff0f
-P cf3d1e0b8a739302cf2848ac3e6eaaae04e6a44c
-R 9e8678dd913fdf27b972a0d8a71d71aa
+P 885e854e7cdc79ecc9d5772d563ddc0f61753ab6
+R 553f0583effedb8a6b18aad4ae4a7180
 U dan
-Z 7c26350aa029a41cf2e5094ee69988d0
+Z 0b1417804855301381cb367f89f39da0

diff --git a/manifest.uuid b/manifest.uuid
index 1982a89445d0973c23df960111d695ea5e205511..338ee1c077173b72cdd228c6ac2f9806da24d882 100644 (file)
--- a/manifest.uuid
+++ b/manifest.uuid
@@ -1 +1 @@
-885e854e7cdc79ecc9d5772d563ddc0f61753ab6
\ No newline at end of file
+c501b2ede6aad123bef0aa7ce8b356a134eb6d26
\ No newline at end of file

diff --git a/src/wal.c b/src/wal.c
index 8e3c0c6469737039681522c93918ad84b61698fe..bdae2e9dfc2fd39310a09f9cb957b4d0ec674db0 100644 (file)
--- a/src/wal.c
+++ b/src/wal.c
@@ -1216,7 +1216,7 @@ void sqlite3WalDbsize(Wal *pWal, Pgno *pPgno){
 int sqlite3WalWriteLock(Wal *pWal, int op){
   int rc = SQLITE_OK;
   if( op ){
-    assert( pWal->lockState == SQLITE_SHM_READ );
+    assert( pWal->lockState==SQLITE_SHM_READ );
     rc = walSetLock(pWal, SQLITE_SHM_WRITE);
 
     /* If this connection is not reading the most recent database snapshot,
@@ -1457,13 +1457,17 @@ int sqlite3WalCheckpoint(
 
   assert( pWal->pWiData==0 );
 
-  /* Get the CHECKPOINT lock */
-  if( pWal->lockState!=SQLITE_SHM_UNLOCK ){
-    /* This can occur when locking_mode=EXCLUSIVE */
-    assert( pWal->lockState==SQLITE_SHM_READ
-         || pWal->lockState==SQLITE_SHM_READ_FULL );
-    walSetLock(pWal, SQLITE_SHM_UNLOCK);
-  }
+  /* Get the CHECKPOINT lock. 
+  **
+  ** Normally, the connection will be in UNLOCK state at this point. But
+  ** if the connection is in exclusive-mode it may still be in READ state
+  ** even though the upper layer has no active read-transaction (because
+  ** WalCloseSnapshot() is not called in exclusive mode). The state will
+  ** be set to UNLOCK when this function returns. This is Ok.
+  */
+  assert( (pWal->lockState==SQLITE_SHM_UNLOCK)
+       || (pWal->exclusiveMode && pWal->lockState==SQLITE_SHM_READ)
+  );
   do {
     rc = walSetLock(pWal, SQLITE_SHM_CHECKPOINT);
   }while( rc==SQLITE_BUSY && xBusyHandler(pBusyHandlerArg) );

diff --git a/test/wal2.test b/test/wal2.test
index e5ef2e07808143d93f86b725e8154a8d39884a9a..781482801e771ec9dd02ea542721ae717aae27ed 100644 (file)
--- a/test/wal2.test
+++ b/test/wal2.test
@@ -575,6 +575,20 @@ do_test wal2-6.4.7 {
   set ::locks
 } {READ WRITE READ UNLOCK}
 db close
+tvfs delete
+
+do_test wal2-6.5.1 {
+  sqlite3 db test.db
+  execsql {
+    PRAGMA journal_mode = wal;
+    PRAGMA locking_mode = exclusive;
+    CREATE TABLE t2(a, b);
+    PRAGMA wal_checkpoint;
+    INSERT INTO t2 VALUES('I', 'II');
+    PRAGMA journal_mode;
+  }
+} {wal exclusive wal}
+db close
 
 finish_test