Merge the accidental fork.

FossilOrigin-Name: 86a06dd0494c2fe83d4fde517557600956cedd9e

Fix a typo on a comment in sqlite3VdbeIntegerAffinity().

FossilOrigin-Name: b5a709d3609d40a6e5ef77f9889077d7395d3d26

Changes to the query planner that improve the order in which tables/indexes are scanned in join queries.

FossilOrigin-Name: 19f799b32f9d1be25d4185ce18b13f4dd502e199

Fixed some compiler warnings in WINCE only sections when using the MSVC compiler.

FossilOrigin-Name: 1f0a93e17d6291268da909699ce1a5a7619ae637

Make sure the large-file support macros occur first in sqliteInt.h.
Fix for CVSTrac ticket #4022.

FossilOrigin-Name: b0848925babde5241aefe0a117ebb10299c94a15

Instead of adding SQLITE_API macros sqlite3.h while creating sqlite3.c, add them as part of generating the sqlite3.h target, while copying from sqlite.h.in to sqlite3.h. This fixes a build problem that was causing multiple SQLITE_API macros to be added to a single line of sqlite3.h, and the problem cited by #4016. (CVS 6971)

FossilOrigin-Name: 7f4810747b0864981f27edbd504bfab2efea1e3c

Reduce the maximum supported sector size to 64KiB. (CVS 6970)

FossilOrigin-Name: 8e1e0ba292150f8968e5295e8ba9cd7360b6a64b

Version 3.6.17 (CVS 6969)

FossilOrigin-Name: 36650102285ef9663853cacd5daf8dd6d7cf418f

Update a couple of tests in corrupt7.test to account for a change in the error message text emitted by SQLite. (CVS 6968)

FossilOrigin-Name: 3172a296926c6b50af5b6edfe38013056e155fa0

Exclude a static function in test1.c from the build when OMIT_DEPRECATED is defined. This avoids a compiler warning that occurs with some compilers. (CVS 6967)

FossilOrigin-Name: ace499506e3c7a315f66928d8a98de255491b2b9

Change mallocI.test to close a database connection that was being left open. (CVS 6966)

FossilOrigin-Name: 4f0096d7ef43f4b9f32c4d78b3dfe34271f2120a

Removed some warnings from the MSVC compile. (CVS 6965)

FossilOrigin-Name: ee034cfc71425745dda8d5fee58ae95f937c86cd

Fixed "Interlocked" logic error that prevented the "static" mutexes from being released when sqlite3_shutdown() is called. Ticket #4011. (CVS 6964)

FossilOrigin-Name: 04cafb7ee3b67a68c0604d7f1143feaec44be610

Suppress various compiler warnings. (CVS 6963)

FossilOrigin-Name: 257e9b575029f63277ed602d1402f87cfa15d22e

Enhancement to (6844). If sqlite3_exec() is to return SQLITE_NOMEM because it fails to allocate space for an error message, set the database error-code to SQLITE_NOMEM too. (CVS 6962)

FossilOrigin-Name: 8293fa19c02abe0e2a7594b9bceac28929de8fdf

Mark the rtreeUpdate function as static. (CVS 6961)

FossilOrigin-Name: b6bdfdc69df4fc6cad669fd8b2cbaa9ecb95cb78

Alternative fix for ticket #3810.  This is a replacement for check-in (6956). (CVS 6960)

FossilOrigin-Name: ba1afc040171810d0c996708d7b9cb11abcd99d8

Change a hyperlink label on shared cache mode documentation.  No changes
to code. (CVS 6959)

FossilOrigin-Name: 3d08ca0e54dd4c292b3db3a8364ab237713c5c43

Simplified conditional for read-only lock in winceLockFile() function.  Ticket #3991. (CVS 6958)

FossilOrigin-Name: a1c184cd2ac5a7559d787e6c34b1d0ce034d10cf

Make sure all database connections are closed at the end of the test case
for ticket #3810. (CVS 6957)

FossilOrigin-Name: 9bca5a0e8f22da6be41b0eda298fd2e19f64e8b6

Return a better error message when problems are encountered parsing a
TEMP trigger or TEMP view that references objects in other databases that
have been modified or dropped.  Ticket #3810. (CVS 6956)

FossilOrigin-Name: 102785b9fbc5ab5dd740110243f080e3f3661e87

Add a testcase for ticket #3810. (CVS 6955)

FossilOrigin-Name: 29972f7445cede64d99c2433742572120c92b393

Fix a segfault following OOM that was introduced by check-in (6949) which
was a fix for ticket #3997. (CVS 6954)

FossilOrigin-Name: 359d78e144c2399791d341eda1760eb486f9740a

Increment the version number in preparation for the next release. (CVS 6953)

FossilOrigin-Name: 12d9b87316cf072d8071fb43ca1232d36720bbca

Fix a bug in where.c that could cause SQLite to write to freed memory while compiling a query with many terms in the WHERE clause. (CVS 6952)

FossilOrigin-Name: 8161af840e8fd2b06457e75023f934e1b8fe5fd6

Add a test to mallocI.test to show that a shared lock is correctly released if an OOM condition occurs while parsing a database schema. (CVS 6951)

FossilOrigin-Name: 5a82620efa1298530760e69e4e34d446a30233b8

Add the experimental API sqlite3_strnicmp(). Modify fts3 so that in terms like 'column_name:token' the column_name is interpreted in a case-insenstive fashion. Ticket #3996. (CVS 6950)

FossilOrigin-Name: 4571aa9e9142db465ae8250b0adf27e0a094331a

When flattening a SELECT query, do not discard collation sequences explicitly attached to expressions in the parent query. Fix for #3997. (CVS 6949)

FossilOrigin-Name: b2784cf5d8b520fb714efdb2d2746b2c88b56edf

Fixed reversed labels in WHERETRACE() statement.  Debugging code only. (CVS 6948)

FossilOrigin-Name: 614a8d83158eb59e3fbe78d62461635bfcc13525

If there is a hot-journal in the file-system, a connection with journal_mode=memory set may have to open it to effect rollback. Account for this in pager_end_transaction(). This prevents an assert from failing in the in-memory journal permutation test. (CVS 6947)

FossilOrigin-Name: 224bc69a04f4fe6d1004125847761b7842c2bfe0

Do not simulate OOM conditions in the sqlite3OsXXX() calls if the underlying file is an in-memory journal file. (CVS 6946)

FossilOrigin-Name: d486811715350f315374cc41f3d808a75d140afb

When extracting values from a record to use in an UPDATEd version of that record, apply OP_RealAffinity if required. Fix for #3992. (CVS 6945)

FossilOrigin-Name: 3616766a6f5c8179cc55444c29ecf29cc69f88ce

The noReadlock property on a pager object implies the readOnly property.
Use this fact to simplify the logic in sqlite3PagerSharedLock(). (CVS 6944)

FossilOrigin-Name: 886e665f6cf780464b280c286b03c9fb5c1a6b2a

Remove obsolete assert and testcase macros related to ticket #2565 (CVS 6943)

FossilOrigin-Name: f0024b3189245d608a0cb57fa7e472c6daa1406f

Adjust memory size computations to avoid a conditional that is always
false on 64-bit systems. (CVS 6942)

FossilOrigin-Name: 04211e6af9cdd3e7f19b458c72b722f8f8584245

Change an unreachable condition in PagerSharedLock to a NEVER(...). Add an assert to pager_error() to show that it is never called to put an in-memory pager to the error-state. (CVS 6941)

FossilOrigin-Name: 1d931f77519baf3586708c77cbd161c0e75bcbaf

Change a condition to a '%' operator in bitvec.c. (CVS 6940)

FossilOrigin-Name: 10250fe5c039dbef2e1614e6320f7bd354c10211

More simplifications to pager.c. (CVS 6939)

FossilOrigin-Name: 533d6cc67a3b6467db5b83c5ca535d229d36ad27

Simplifications and comment improvements to pager.c in support of coverage
testing. (CVS 6938)

FossilOrigin-Name: 5b70b5c19cd587a8afbf2909ac7a4c04aea20f44

Add a NEVER() on an always-false conditional in pager.c.
Make sure the text of the master journal file is zero-terminated before
trying to process it - to prevent a buffer overrun in strlen(). (CVS 6937)

FossilOrigin-Name: cc9430e334fe98e1c35d408f81a2d8953377cda6

Remove the "proc zeroblob" implementation from incrvacuum2.test. It is no longer required as of (6906) and, as #3988 points out, unreliable. (CVS 6936)

FossilOrigin-Name: 03153831635342a744ee42f14cb041499cdece2a

Add files that should have gone in with the previous checkin. (CVS 6935)

FossilOrigin-Name: 64c7afc704e95e13fb3315be402525d277a0d8d8

Add a complex assert() that checks a pager invariant. (CVS 6934)

FossilOrigin-Name: 2e08ad7bf629485a79ccebcf4c413e9dcc0d3ccf

Add assert() statements to more tightly constrain the state of pager.c.
Remove the three pager*.test scripts since they violate the constraints
asserted above by modifying the state of the system in ways that it cannot
be modified in a live system. (CVS 6933)

FossilOrigin-Name: 3b6d370ed68eaf9636b26c7240a8b3a43d2edd70

Remove two more overzealous assert() statements. (CVS 6932)

FossilOrigin-Name: 788aa9dde9e76f67f22c07a904faf03e97e032c6

Remove an overzealous assert() from pager.c. (CVS 6931)

FossilOrigin-Name: 4a924b475aa0f7f2f69caed0adc2e60d29c47f21

Fix an issue with the previous check-in, (6929). (CVS 6930)

FossilOrigin-Name: e5165c80abfdba0ad74e1042e4c94fe00d4b3308

Further simplifications to pager.c in support of structural coverage testing. (CVS 6929)

FossilOrigin-Name: 866216c0b6ba23b66483046fb402ddbe4f9a7b14

Allow virtual tables to be used in shared-cache mode. (CVS 6928)

FossilOrigin-Name: 5d9e767a05e381235e064061043e30cc03a11a07

Simplifications to pager.c in support of structural coverage testing. (CVS 6927)

FossilOrigin-Name: 7222ad2667b95d6021d9ae47f548b76b224f46aa

Simplifications and comment improvements in pager.c. (CVS 6926)

FossilOrigin-Name: 2d2f42ca0a24ed8b33f9ad560c76a6c1301c757b

Simplifications and comment enhancements on btree.c. (CVS 6925)

FossilOrigin-Name: 5ba880dde8a219543ced6f792c7f9ecdcd8c1cbb

Modify mksqlite3c.tcl so that it inserts SQLITE_API macros into sqlite3.h. Ticket #3983. (CVS 6924)

FossilOrigin-Name: 803ec79f3b05fdd680f9ab762685bbd50a087b9b

Simplifications to btree.c in support of coverage testing. (CVS 6923)

FossilOrigin-Name: 375fd6f9c54fa5aa8cfae30845c7bfc0ec49e8fa

Avoid leaving a suspect page in the page-cache if an error occurs during sqlite3PagerAcquire(). (CVS 6922)

FossilOrigin-Name: 2594f01c73301f9432be9d007a079a19a3d383f9

Fix a double-free that can occur after database corruption is detected. (CVS 6921)

FossilOrigin-Name: 5260e8b1048af2e00d49706466d9aded90c24ed2

Add a NEVER() macro around a pager.c conditional that must always be false
in the current implementation. (CVS 6920)

FossilOrigin-Name: 1e510d7fc5cc47fe2714dbb1ed3ea7e90acbff7b

Changes to test code (only) to allow veryquick.test to run with OMIT_UTF16 builsd. Ticket #3985. (CVS 6919)

FossilOrigin-Name: 7f3fc7f20f6dc205eedbe445e35f5071f1c01185

Remove an ALWAYS() from a condition that can in fact be false under very
obscure circumstances. (CVS 6918)

FossilOrigin-Name: 38083228956ac05df45f44447ab39c64852239d2

Further simplifications to btree.c, especially the sqlite3BtreeKey() and
sqlite3BtreeData() functions.  New assert() statements added to verify
that these routines are called correctly. (CVS 6917)

FossilOrigin-Name: 96cfd079528501f6f1e658ce8a5a4e3bdea729be

Change getAndInitPage() (btree.c) to use only PagerAcquire(), not PagerLookup() and PagerAcquire(). (CVS 6916)

FossilOrigin-Name: 0b41dfc066b60ccabbf1a9ab4db41ebcb73a2799

Additional simplifications to btree.c in support of coverage testing. (CVS 6915)

FossilOrigin-Name: 716fccea58c4c217e68e04e0776e44ae39c11950

Remove an assert() in btree.c which is no longer true due to changes in
the error reporting behavior of ptrmapPut(). (CVS 6914)

FossilOrigin-Name: 110998f18a7ad1ddaffab048cabef675d882cbb8

Changes to btree.c in support of coverage testing. (CVS 6913)

FossilOrigin-Name: 4cf23e9e860bd6245344884ec84f487fdf36f86f

Reverse the order of two conditionals in a test in order to achieve coverage
of them both.  Also: clarifications to comments in btree.c. (CVS 6912)

FossilOrigin-Name: a159e9d24708dfe6c5a98d220e39f3b1b2a3d281

Modify various routines inside btree.c to take a pointer to the return-code
as a parameter and to no-op if the return-code storage location already
contains a non-zero code. (CVS 6911)

FossilOrigin-Name: 7dcf2a78727ce448d9d26bd6e2e23b553456ec54

Use an ALWAYS() on a conditional in btmutex.c that is always true. (CVS 6910)

FossilOrigin-Name: 609022caff12c010575e704be550da6f52226d33

Corrections to the comment that is inserted at the top of the amalgamation.
Ticket #3981. (CVS 6909)

FossilOrigin-Name: 7572991bc2232f57d04c972d516a3e8f85f34f14

Always provide a name for the TEMP database even if SQLite is compiled using
SQLITE_OMIT_TEMPDB. (CVS 6908)

FossilOrigin-Name: 0ba17c04834c0a44c8690169c0096c1d9cf325e4

Code tweaks in support of full-coverage testing. (CVS 6907)

FossilOrigin-Name: 22c6dbfdce965e32f3df9aec28e46e714620c362

Add some code to sqlite3VdbeMemSetZeroBlob() so that it works (less efficiently) even when OMIT_INCRBLOB is defined. Ticket #3977. (CVS 6906)

FossilOrigin-Name: f5f7259d811ce8c2bf0bc64128256c4cd95f676a

When the asynchronous IO backend opens a file with the EXCLUSIVE flag set, make sure only a single file-descriptor is opened (not one for reading and one for writing). This change fixes #3978. (CVS 6905)

FossilOrigin-Name: 630e669b97a81f9125d4bdc18517738b74eecdec

Changes to test scripts so that (make test) works when OMIT_INCRBLOB is defined. (CVS 6904)

FossilOrigin-Name: 1dd834a3d61496dfa138c36676a5fa6afaa4c817

Fix a subtle problem that can occur when a transaction is automatically rolled back, and afterwards a VM that had opened a statement transaction within the abandoned transaction is finalized. Fixes an assert() that was failing in rtree3.test. (CVS 6903)

FossilOrigin-Name: 910df46e0f570cbffa9ff9b50d75632c091838b6

Return a meaningful error message if a keyword is used as an rtree table column name. Ticket #3970. (CVS 6902)

FossilOrigin-Name: 046efe46b50fbe928f39a0cda1b1006d486ce9f5

Skip all tests in tableapi.test if SQLITE_OMIT_GET_TABLE defined. Ticket #3975. (CVS 6901)

FossilOrigin-Name: 0219a543367638013ccd06295eeb9efc822e2dd1

Code simplifications in support of structural testing. (CVS 6900)

FossilOrigin-Name: fb1b955dda5105025ef199880afa871e44331d65

Code simplifications and comment improvements in support of structural
coverage testing. (CVS 6899)

FossilOrigin-Name: 945251798144110787b197f9eb552a2dd4a25cb4

Remove an unused variable from sqlite3ExprListDup().  Ticket #3973. (CVS 6898)

FossilOrigin-Name: 17c05aefd6c9e938a4cc5d3de348d8c8fdc9394d

Fix a potential database corruption following DROP TABLE when the pending
byte page corresponds to a ptrmap page.  This situation cannot happen in
a real deployment - but it still needs to be fixed. (CVS 6897)

FossilOrigin-Name: 6242db39f7f4ead5017bf684139d1f8312799f81

Simplifications to btree.c in support of structural testing.  Renamed the
"skip" field of the BtCursor object to "skipNext" to make it easier to
search for places where it is used. (CVS 6896)

FossilOrigin-Name: d3897235d77e48ad09f7edb0a7641458afa0a282

Remove an assert() from vdbeaux.c that might not be true if the database
file is corrupt. (CVS 6895)

FossilOrigin-Name: a42dc51e3bd258d5a357e557251fb2642708ceac

Added the SQLITE_TESTCTRL_RESERVE option to sqlite3_test_control().
Simplifications to btree.c to facilitate structural coverage testing. (CVS 6894)

FossilOrigin-Name: cfbeda1b3126503e9631471ce4713e25b007859f

Add comments and assert() statements to show that the return value from
sqlite3BtreeKeySize() and sqlite3BtreeData() usually do not matter.
Ticket #3968.  Also remove a NEVER() macro that can sometimes be true -
discovered while testing the previous change. (CVS 6893)

FossilOrigin-Name: 0c710c1be537127511d95b5b261c7bf26e1bc952

Simplifications to the PRAGMA integrity_check logic.  Remove unreachable code. (CVS 6892)

FossilOrigin-Name: 17749fa5480069cc4909acd992ab26d10afb279d

Simplifications and tweaks to vdbeaux.c resulting from structural testing. (CVS 6891)

FossilOrigin-Name: fa49666fb913f0d82e84bdfa2af3a294be04e47c

Make sure the IN operator works with zeroblobs. Ticket #3965.
Other simplifications associated with structural testing. (CVS 6890)

FossilOrigin-Name: 25dd342283046aaf66a679348ef1c7364c616402

Remove the SQLITE_CommitBusy flag.  This was an attempt to block recursion
on the sqlite3_commit_hook() interface.  But such recursion is explicitly
disallowed, so the flag is pointless. (CVS 6889)

FossilOrigin-Name: 1c2bfc43a4fd5b779a3b5b5b8ca5b41cb7250b5a

Change an unreachable condition in btree.c to a NEVER(). (CVS 6888)

FossilOrigin-Name: 9f800e11391a108d7aa57f5a96d9a58711129132

In sqlite3PagerWrite(), do not set the PGHDR_NEED_SYNC flag on a page if an IO error occured while attempting to journal it. (CVS 6887)

FossilOrigin-Name: b9be365d85fddedbfa93eebf3ee62a140cbaa426

Remove an assert ("assert( subpage>0 )") from btree.c that may not be true for a corrupt database. Also add comments and other assert() statements to btree.c function moveToRoot(). (CVS 6886)

FossilOrigin-Name: 3151dab9c78106217ec80ebadc666dfd11b42029

Remove a case from BtreeMovetoUnpacked() that is unreachable as of (6881). (CVS 6885)

FossilOrigin-Name: 39ce2097da03176e256a2ff35bb857e578f3ca2d

Simplifications to btree.c to help facilitate coverage testing. (CVS 6884)

FossilOrigin-Name: dfb146ef003bae8ff216baf68cc43d7eb7855c68

When an OOM fault occurs while moving a page for autovacuum in an in-memory
database, be sure to undo the page move prior to returning the error, to
avoid corrupting the in-memory database file. (CVS 6883)

FossilOrigin-Name: d0964b93669123e228e7ce1890167447c56753b5

Remove an unused parameter from the accessPayload() function in btree.c. (CVS 6882)

FossilOrigin-Name: 7deb6568d89335926b77336756837c6dc3985529

Fix a case where deleting a row from a corrupt database could cause an assert to fail. (CVS 6881)

FossilOrigin-Name: 6994b41a94a60f6460cf9814767db321ab3851f7

Bug fix in the computation of the number of pages to autovacuum when
nReserve is greater than zero. (CVS 6880)

FossilOrigin-Name: 618a83d65f973183d21245721dc656a35ff594a4

Unwind some complex conditions in sqlite3BtreeDelete() into
separate "if" statements. (CVS 6879)

FossilOrigin-Name: d99bde9ca61eeccfe6363ff0882fd4bcdb9a34dc

Remove another unreachable branch from btree.c. (CVS 6878)

FossilOrigin-Name: b0853100a9f8e185e8d027502822337a79a2ba0c

Modify the integrity-check code so that each b-tree page inspected is reinitialized while doing so, even if an initialized copy exists in the cache. This prevents an assert from failing when running integrity_check on a corrupt database. (CVS 6877)

FossilOrigin-Name: 709576c670f802bf4b6e5c0e8db2bbde2cc16a90