/ Check-in [b93f6f3e]
Login

Many hyperlinks are disabled.
Use anonymous login to enable hyperlinks.

Overview
Comment:Add back an ALWAYS() to regain full test coverage.
Downloads: Tarball | ZIP archive | SQL archive
Timelines: family | ancestors | descendants | both | trunk
Files: files | file ages | folders
SHA1: b93f6f3e679c7710f42580a8dd9ce43136376c1d
User & Date: drh 2011-01-17 02:24:12
Context
2011-01-17
17:42
Add the SQLITE_OMIT_AUTORESET compile-time option which if enabled causes the sqlite3_step() routine to return SQLITE_MISUSE if it is called after it has previously returned anything other than SQLITE_ROW, SQLITE_BUSY, or SQLITE_LOCKED. check-in: 053ce76d user: drh tags: trunk
02:24
Add back an ALWAYS() to regain full test coverage. check-in: b93f6f3e user: drh tags: trunk
2011-01-16
22:37
On a backup from a smaller to a larger page size, do not begin committing the transaction until the source pages after the pending byte have been copied. check-in: 612e2599 user: drh tags: trunk
Changes
Hide Diffs Side-by-Side Diffs Ignore Whitespace Patch

Changes to src/pager.c.

  5581   5581       assert( pPgHdr==0 || rc==SQLITE_OK );
  5582   5582   
  5583   5583       /* If page one was fetched successfully, and this function is not
  5584   5584       ** operating in direct-mode, make page 1 writable.  When not in 
  5585   5585       ** direct mode, page 1 is always held in cache and hence the PagerGet()
  5586   5586       ** above is always successful - hence the ALWAYS on rc==SQLITE_OK.
  5587   5587       */
  5588         -    if( !DIRECT_MODE && rc==SQLITE_OK ){
         5588  +    if( !DIRECT_MODE && ALWAYS(rc==SQLITE_OK) ){
  5589   5589         rc = sqlite3PagerWrite(pPgHdr);
  5590   5590       }
  5591   5591   
  5592   5592       if( rc==SQLITE_OK ){
  5593   5593         /* Actually do the update of the change counter */
  5594   5594         pager_write_changecounter(pPgHdr);
  5595   5595