Fix harmless compiler warnings.

FossilOrigin-Name: ed2c9f3738c96d8e3dbece7ccb721cb1a8ae8fac

diff --git a/manifest b/manifest
index 6c25d39d50b56a33eb6a703038df0930fd4a6217..254f17d042d3e5268b93ac827f250dc0bcc2d884 100644 (file)
--- a/manifest
+++ b/manifest
@@ -1,5 +1,5 @@
-C Convert\ssqlite3PagerGet()\sinto\sa\spointer-dispatched\svirtual\smethod.\s\sThis\nmakes\sit\sabout\s25%\sfaster.
-D 2016-12-13T18:47:54.625
+C Fix\sharmless\scompiler\swarnings.
+D 2016-12-13T20:30:29.436
 F Makefile.in 7639c6a09da11a9c7c6f2630fc981ee588d1072d
 F Makefile.linux-gcc 7bc79876b875010e8c8f9502eb935ca92aa3c434
 F Makefile.msc b8ca53350ae545e3562403d5da2a69cec79308da
@@ -331,7 +331,7 @@ F src/auth.c 930b376a9c56998557367e6f7f8aaeac82a2a792
 F src/backup.c faf17e60b43233c214aae6a8179d24503a61e83b
 F src/bitvec.c 17ea48eff8ba979f1f5b04cc484c7bb2be632f33
 F src/btmutex.c bc87dd3b062cc26edfe79918de2200ccb8d41e73
-F src/btree.c 0d377e5769657d3ac2a1b6ba081ba77a128610d5
+F src/btree.c b2055dff0b94e03eaad48a760984a2d8e39244e6
 F src/btree.h 2349a588abcd7e0c04f984e15c5c777b61637583
 F src/btreeInt.h 10c4b77c2fb399580babbcc7cf652ac10dba796e
 F src/build.c 178f16698cbcb43402c343a9413fe22c99ffee21
@@ -375,7 +375,7 @@ F src/os_setup.h 0dbaea40a7d36bf311613d31342e0b99e2536586
 F src/os_unix.c 30e2c43e4955db990e5b5a81e901f8aa74cc8820
 F src/os_win.c cf90abd4e50d9f56d2c20ce8e005aff55d7bd8e9
 F src/os_win.h 7b073010f1451abe501be30d12f6bc599824944a
-F src/pager.c dbcac003871bf8a9bfb69d6cb66bbb85b528aaf6
+F src/pager.c c3ba8e69388d974017004aa3fd3511e5bbadbba0
 F src/pager.h d1e944291030351f362a0a7da9b5c3e34e603e39
 F src/parse.y 29153738a7322054359320eb00b5a4cd44389f20
 F src/pcache.c 51070ec9b8251bbf9c6ea3d35fd96a458752929e
@@ -1536,8 +1536,7 @@ F vsixtest/vsixtest.tcl 6a9a6ab600c25a91a7acc6293828957a386a8a93
 F vsixtest/vsixtest.vcxproj.data 2ed517e100c66dc455b492e1a33350c1b20fbcdc
 F vsixtest/vsixtest.vcxproj.filters 37e51ffedcdb064aad6ff33b6148725226cd608e
 F vsixtest/vsixtest_TemporaryKey.pfx e5b1b036facdb453873e7084e1cae9102ccc67a0
-P 850877d1ea43104cc215353414b870c340acced2 dee20ba982125ea98c280ad1571789af0f393903
-R 1925a57b32b9d19fb42c95ae0d5863ee
-T +closed dee20ba982125ea98c280ad1571789af0f393903
+P 7f88bb44129a0cd36e27e00dc7c37e87cf3c90f7
+R f2e66ee6668f0716a35acfaa4809f04b
 U drh
-Z ca81384c708270e6307c733a622217dc
+Z 404271946c196a116af17aa555379b02

diff --git a/manifest.uuid b/manifest.uuid
index ad8184c8edb0ef76ed721279c84d4accb41dd12a..99dfe7c0014a014c0af51cc45ebfa177aa66e812 100644 (file)
--- a/manifest.uuid
+++ b/manifest.uuid
@@ -1 +1 @@
-7f88bb44129a0cd36e27e00dc7c37e87cf3c90f7
\ No newline at end of file
+ed2c9f3738c96d8e3dbece7ccb721cb1a8ae8fac
\ No newline at end of file

diff --git a/src/btree.c b/src/btree.c
index e91fad3b122c4aa0c12ae3316f092870333ff242..0cd1a28778535a45228ff807e9a07f3f75fa1bc3 100644 (file)
--- a/src/btree.c
+++ b/src/btree.c
@@ -1743,7 +1743,7 @@ static int btreeInitPage(MemPage *pPage){
   assert( pPage->aData == sqlite3PagerGetData(pPage->pDbPage) );
 
   if( !pPage->isInit ){
-    u32 pc;            /* Address of a freeblock within pPage->aData[] */
+    int pc;            /* Address of a freeblock within pPage->aData[] */
     u8 hdr;            /* Offset to beginning of page header */
     u8 *data;          /* Equal to pPage->aData */
     BtShared *pBt;        /* The main btree structure */
@@ -1844,7 +1844,7 @@ static int btreeInitPage(MemPage *pPage){
       if( next>0 ){
         return SQLITE_CORRUPT_BKPT;  /* Freeblock not in ascending order */
       }
-      if( pc+size>usableSize ){
+      if( pc+size>(unsigned int)usableSize ){
         return SQLITE_CORRUPT_BKPT;  /* Last freeblock extends past page end */
       }
     }

diff --git a/src/pager.c b/src/pager.c
index 7bafc1c09ba9a86ff6935e56906d060e2cdbe51e..cd2ded4e3fa0b4ff069d3b03a514b6b9e30451cf 100644 (file)
--- a/src/pager.c
+++ b/src/pager.c
@@ -5472,11 +5472,6 @@ static int getPageMMap(
   PgHdr *pPg = 0;
   u32 iFrame = 0;                 /* Frame to read from WAL file */
 
-  assert( USEFETCH(pPager) );
-#ifdef SQLITE_HAS_CODEC
-  assert( pPager->xCodec==0 );
-#endif
-
   /* It is acceptable to use a read-only (mmap) page for any page except
   ** page 1 if there is no write-transaction open or the ACQUIRE_READONLY
   ** flag was specified by the caller. And so long as the db is not a 
@@ -5485,6 +5480,11 @@ static int getPageMMap(
    && (pPager->eState==PAGER_READER || (flags & PAGER_GET_READONLY))
   );
 
+  assert( USEFETCH(pPager) );
+#ifdef SQLITE_HAS_CODEC
+  assert( pPager->xCodec==0 );
+#endif
+
   /* Optimization note:  Adding the "pgno<=1" term before "pgno==0" here
   ** allows the compiler optimizer to reuse the results of the "pgno>1"
   ** test in the previous statement, and avoid testing pgno==0 in the
@@ -5540,6 +5540,8 @@ static int getPageError(
   DbPage **ppPage,    /* Write a pointer to the page here */
   int flags           /* PAGER_GET_XXX flags */
 ){
+  UNUSED_PARAMETER(pgno);
+  UNUSED_PARAMETER(flags);
   assert( pPager->errCode!=SQLITE_OK );
   *ppPage = 0;
   return pPager->errCode;