database/sqlite
1
0
Fork 0
mirror of https://github.com/sqlite/sqlite.git synced 2025-08-10 01:02:56 +03:00
Code Activity

Remove an faulty assertion inserted a few days ago. (CVS 6160)

Browse Source 
FossilOrigin-Name: 495f017e03f32420e6a6a7685125a86929cef142
This commit is contained in:
drh
2009-01-10 17:02:02 +00:00
parent 47c3b3e099
commit ceb92adb9d
3 changed files with 8 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
src/pager.c
View File

@@ -18,7 +18,7 @@
** file simultaneously, or one process from reading the database while
** another is writing.
**
** @(#) $Id: pager.c,v 1.545 2009/01/10 16:15:09 danielk1977 Exp $
** @(#) $Id: pager.c,v 1.546 2009/01/10 17:02:03 drh Exp $
*/
#ifndef SQLITE_OMIT_DISKIO
#include "sqliteInt.h"
@@ -3637,19 +3637,12 @@ int sqlite3PagerWrite(DbPage *pDbPage){
** writing to any of these nPage pages may damage the others, the
** journal file must contain sync()ed copies of all of them
** before any of them can be written out to the database file.
**
** 2009-01-07: This block of code appears to be a no-op. I do not
** believe it is possible for any page on the sector to not have
** the PGHDR_NEED_SYNC flag set. The "pPage->flags |= PGHDR_NEED_SYNC"
** line below does nothing, I think. But it does no harm to leave
** this code in place until we can definitively prove this is the case.
*/
if( needSync ){
assert( !MEMDB && pPager->noSync==0 );
for(ii=0; ii<nPage && needSync; ii++){
PgHdr *pPage = pager_lookup(pPager, pg1+ii);
if( pPage ){
assert( pPage->flags & PGHDR_NEED_SYNC ); /* 2009-01-07 conjecture */
pPage->flags |= PGHDR_NEED_SYNC;
sqlite3PagerUnref(pPage);
}