| Index: third_party/sqlite/src/src/test8.c | 
| diff --git a/third_party/sqlite/src/src/test8.c b/third_party/sqlite/src/src/test8.c | 
| index b3e6058acb19d331715a9a0cdd0d39d9ce6c0417..8bc835d638c322ce4c0353bdfd2ed46d8d0b69dd 100644 | 
| --- a/third_party/sqlite/src/src/test8.c | 
| +++ b/third_party/sqlite/src/src/test8.c | 
| @@ -192,7 +192,7 @@ static int getColumnNames( | 
| rc = SQLITE_NOMEM; | 
| goto out; | 
| } | 
| -      nBytes += strlen(zName)+1; | 
| +      nBytes += (int)strlen(zName)+1; | 
| } | 
| aCol = (char **)sqlite3MallocZero(nBytes); | 
| if( !aCol ){ | 
| @@ -265,6 +265,7 @@ static int getIndexArray( | 
| while( pStmt && sqlite3_step(pStmt)==SQLITE_ROW ){ | 
| const char *zIdx = (const char *)sqlite3_column_text(pStmt, 1); | 
| sqlite3_stmt *pStmt2 = 0; | 
| +    if( zIdx==0 ) continue; | 
| zSql = sqlite3_mprintf("PRAGMA index_info(%s)", zIdx); | 
| if( !zSql ){ | 
| rc = SQLITE_NOMEM; | 
| @@ -831,13 +832,10 @@ static int echoBestIndex(sqlite3_vtab *tab, sqlite3_index_info *pIdxInfo){ | 
| if( !isIgnoreUsable && !pConstraint->usable ) continue; | 
|  | 
| iCol = pConstraint->iColumn; | 
| -    if( pVtab->aIndex[iCol] || iCol<0 ){ | 
| -      char *zCol = pVtab->aCol[iCol]; | 
| +    if( iCol<0 || pVtab->aIndex[iCol] ){ | 
| +      char *zCol = iCol>=0 ? pVtab->aCol[iCol] : "rowid"; | 
| char *zOp = 0; | 
| useIdx = 1; | 
| -      if( iCol<0 ){ | 
| -        zCol = "rowid"; | 
| -      } | 
| switch( pConstraint->op ){ | 
| case SQLITE_INDEX_CONSTRAINT_EQ: | 
| zOp = "="; break; | 
| @@ -870,13 +868,12 @@ static int echoBestIndex(sqlite3_vtab *tab, sqlite3_index_info *pIdxInfo){ | 
| ** on a column that this virtual table has an index for, then consume | 
| ** the ORDER BY clause. | 
| */ | 
| -  if( pIdxInfo->nOrderBy==1 && pVtab->aIndex[pIdxInfo->aOrderBy->iColumn] ){ | 
| +  if( pIdxInfo->nOrderBy==1 && ( | 
| +        pIdxInfo->aOrderBy->iColumn<0 || | 
| +        pVtab->aIndex[pIdxInfo->aOrderBy->iColumn]) ){ | 
| int iCol = pIdxInfo->aOrderBy->iColumn; | 
| -    char *zCol = pVtab->aCol[iCol]; | 
| +    char *zCol = iCol>=0 ? pVtab->aCol[iCol] : "rowid"; | 
| char *zDir = pIdxInfo->aOrderBy->desc?"DESC":"ASC"; | 
| -    if( iCol<0 ){ | 
| -      zCol = "rowid"; | 
| -    } | 
| zNew = sqlite3_mprintf(" ORDER BY %s %s", zCol, zDir); | 
| string_concat(&zQuery, zNew, 1, &rc); | 
| pIdxInfo->orderByConsumed = 1; | 
| @@ -895,7 +892,7 @@ static int echoBestIndex(sqlite3_vtab *tab, sqlite3_index_info *pIdxInfo){ | 
| pIdxInfo->estimatedCost = cost; | 
| }else if( useIdx ){ | 
| /* Approximation of log2(nRow). */ | 
| -    for( ii=0; ii<(sizeof(int)*8); ii++ ){ | 
| +    for( ii=0; ii<(sizeof(int)*8)-1; ii++ ){ | 
| if( nRow & (1<<ii) ){ | 
| pIdxInfo->estimatedCost = (double)ii; | 
| } | 
| @@ -1221,7 +1218,7 @@ static int echoRename(sqlite3_vtab *vtab, const char *zNewName){ | 
| } | 
|  | 
| if( p->isPattern ){ | 
| -    int nThis = strlen(p->zThis); | 
| +    int nThis = (int)strlen(p->zThis); | 
| char *zSql = sqlite3_mprintf("ALTER TABLE %s RENAME TO %s%s", | 
| p->zTableName, zNewName, &p->zTableName[nThis] | 
| ); | 
| @@ -1232,12 +1229,50 @@ static int echoRename(sqlite3_vtab *vtab, const char *zNewName){ | 
| return rc; | 
| } | 
|  | 
| +static int echoSavepoint(sqlite3_vtab *pVTab, int iSavepoint){ | 
| +  assert( pVTab ); | 
| +  return SQLITE_OK; | 
| +} | 
| + | 
| +static int echoRelease(sqlite3_vtab *pVTab, int iSavepoint){ | 
| +  assert( pVTab ); | 
| +  return SQLITE_OK; | 
| +} | 
| + | 
| +static int echoRollbackTo(sqlite3_vtab *pVTab, int iSavepoint){ | 
| +  assert( pVTab ); | 
| +  return SQLITE_OK; | 
| +} | 
| + | 
| /* | 
| ** A virtual table module that merely "echos" the contents of another | 
| ** table (like an SQL VIEW). | 
| */ | 
| static sqlite3_module echoModule = { | 
| -  0,                         /* iVersion */ | 
| +  1,                         /* iVersion */ | 
| +  echoCreate, | 
| +  echoConnect, | 
| +  echoBestIndex, | 
| +  echoDisconnect, | 
| +  echoDestroy, | 
| +  echoOpen,                  /* xOpen - open a cursor */ | 
| +  echoClose,                 /* xClose - close a cursor */ | 
| +  echoFilter,                /* xFilter - configure scan constraints */ | 
| +  echoNext,                  /* xNext - advance a cursor */ | 
| +  echoEof,                   /* xEof */ | 
| +  echoColumn,                /* xColumn - read data */ | 
| +  echoRowid,                 /* xRowid - read data */ | 
| +  echoUpdate,                /* xUpdate - write data */ | 
| +  echoBegin,                 /* xBegin - begin transaction */ | 
| +  echoSync,                  /* xSync - sync transaction */ | 
| +  echoCommit,                /* xCommit - commit transaction */ | 
| +  echoRollback,              /* xRollback - rollback transaction */ | 
| +  echoFindFunction,          /* xFindFunction - function overloading */ | 
| +  echoRename                 /* xRename - rename the table */ | 
| +}; | 
| + | 
| +static sqlite3_module echoModuleV2 = { | 
| +  2,                         /* iVersion */ | 
| echoCreate, | 
| echoConnect, | 
| echoBestIndex, | 
| @@ -1257,12 +1292,16 @@ static sqlite3_module echoModule = { | 
| echoRollback,              /* xRollback - rollback transaction */ | 
| echoFindFunction,          /* xFindFunction - function overloading */ | 
| echoRename,                /* xRename - rename the table */ | 
| +  echoSavepoint, | 
| +  echoRelease, | 
| +  echoRollbackTo | 
| }; | 
|  | 
| /* | 
| ** Decode a pointer to an sqlite3 object. | 
| */ | 
| extern int getDbPointer(Tcl_Interp *interp, const char *zA, sqlite3 **ppDb); | 
| +extern const char *sqlite3ErrName(int); | 
|  | 
| static void moduleDestroy(void *p){ | 
| sqlite3_free(p); | 
| @@ -1277,6 +1316,7 @@ static int register_echo_module( | 
| int objc,              /* Number of arguments */ | 
| Tcl_Obj *CONST objv[]  /* Command arguments */ | 
| ){ | 
| +  int rc; | 
| sqlite3 *db; | 
| EchoModule *pMod; | 
| if( objc!=2 ){ | 
| @@ -1284,9 +1324,24 @@ static int register_echo_module( | 
| return TCL_ERROR; | 
| } | 
| if( getDbPointer(interp, Tcl_GetString(objv[1]), &db) ) return TCL_ERROR; | 
| + | 
| +  /* Virtual table module "echo" */ | 
| pMod = sqlite3_malloc(sizeof(EchoModule)); | 
| pMod->interp = interp; | 
| -  sqlite3_create_module_v2(db, "echo", &echoModule, (void*)pMod, moduleDestroy); | 
| +  rc = sqlite3_create_module_v2( | 
| +      db, "echo", &echoModule, (void*)pMod, moduleDestroy | 
| +  ); | 
| + | 
| +  /* Virtual table module "echo_v2" */ | 
| +  if( rc==SQLITE_OK ){ | 
| +    pMod = sqlite3_malloc(sizeof(EchoModule)); | 
| +    pMod->interp = interp; | 
| +    rc = sqlite3_create_module_v2(db, "echo_v2", | 
| +        &echoModuleV2, (void*)pMod, moduleDestroy | 
| +    ); | 
| +  } | 
| + | 
| +  Tcl_SetResult(interp, (char *)sqlite3ErrName(rc), TCL_STATIC); | 
| return TCL_OK; | 
| } | 
|  | 
| @@ -1328,8 +1383,8 @@ int Sqlitetest8_Init(Tcl_Interp *interp){ | 
| Tcl_ObjCmdProc *xProc; | 
| void *clientData; | 
| } aObjCmd[] = { | 
| -     { "register_echo_module",   register_echo_module, 0 }, | 
| -     { "sqlite3_declare_vtab",   declare_vtab, 0 }, | 
| +     { "register_echo_module",       register_echo_module, 0 }, | 
| +     { "sqlite3_declare_vtab",       declare_vtab, 0 }, | 
| }; | 
| int i; | 
| for(i=0; i<sizeof(aObjCmd)/sizeof(aObjCmd[0]); i++){ | 
|  |