Many hyperlinks are disabled.
Use anonymous login
to enable hyperlinks.
Overview
Comment: | Fix harmless compiler warnings in the ALTER TABLE logic. Rephrase an error message to use active voice. |
---|---|
Downloads: | Tarball | ZIP archive |
Timelines: | family | ancestors | descendants | both | alter-table-rename-column |
Files: | files | file ages | folders |
SHA3-256: |
a7dae59a7e7b187fa2e9457bbb4b01cf |
User & Date: | drh 2018-08-23 19:32:04.605 |
Context
2018-08-23
| ||
20:09 | Add the "atrc" test program to the Makefiles. Fix a typo in the instructions in the header comment of the atrc program. (check-in: 2130a407dd user: drh tags: alter-table-rename-column) | |
19:32 | Fix harmless compiler warnings in the ALTER TABLE logic. Rephrase an error message to use active voice. (check-in: a7dae59a7e user: drh tags: alter-table-rename-column) | |
18:50 | The Expr.iColumn field must also be initialized in tokenExpr(). (check-in: 772985f18b user: drh tags: alter-table-rename-column) | |
Changes
Changes to src/alter.c.
︙ | ︙ | |||
809 810 811 812 813 814 815 | #ifndef SQLITE_OMIT_VIRTUALTABLE if( IsVirtual(pTab) ){ zType = "virtual table"; } #endif if( zType ){ sqlite3ErrorMsg( | | | 809 810 811 812 813 814 815 816 817 818 819 820 821 822 823 | #ifndef SQLITE_OMIT_VIRTUALTABLE if( IsVirtual(pTab) ){ zType = "virtual table"; } #endif if( zType ){ sqlite3ErrorMsg( pParse, "cannot rename columns of %s \"%s\"", zType, pTab->zName ); return 1; } return 0; } #else /* !defined(SQLITE_OMIT_VIEW) || !defined(SQLITE_OMIT_VIRTUALTABLE) */ # define isRealTable(x,y) (0) |
︙ | ︙ | |||
1069 1070 1071 1072 1073 1074 1075 | */ static void renameColumnParseError( sqlite3_context *pCtx, sqlite3_value *pType, sqlite3_value *pObject, Parse *pParse ){ | | | | 1069 1070 1071 1072 1073 1074 1075 1076 1077 1078 1079 1080 1081 1082 1083 1084 | */ static void renameColumnParseError( sqlite3_context *pCtx, sqlite3_value *pType, sqlite3_value *pObject, Parse *pParse ){ const char *zT = (const char*)sqlite3_value_text(pType); const char *zN = (const char*)sqlite3_value_text(pObject); char *zErr; zErr = sqlite3_mprintf("error processing %s %s: %s", zT, zN, pParse->zErrMsg); sqlite3_result_error(pCtx, zErr, -1); sqlite3_free(zErr); } |
︙ | ︙ |
Changes to test/altercol.test.
︙ | ︙ | |||
499 500 501 502 503 504 505 | do_execsql_test 12.2.1 { CREATE VIEW v1 AS SELECT * FROM t1; CREATE VIEW v2(c, d) AS SELECT * FROM t1; } do_catchsql_test 12.2.2 { ALTER TABLE v1 RENAME a TO z; | | | | 499 500 501 502 503 504 505 506 507 508 509 510 511 512 513 514 515 516 | do_execsql_test 12.2.1 { CREATE VIEW v1 AS SELECT * FROM t1; CREATE VIEW v2(c, d) AS SELECT * FROM t1; } do_catchsql_test 12.2.2 { ALTER TABLE v1 RENAME a TO z; } {1 {cannot rename columns of view "v1"}} do_catchsql_test 12.2.3 { ALTER TABLE v2 RENAME c TO y; } {1 {cannot rename columns of view "v2"}} ifcapable fts5 { do_execsql_test 12.3.1 { CREATE VIRTUAL TABLE ft USING fts5(a, b, c); } do_catchsql_test 12.3.2 { ALTER TABLE ft RENAME a TO z; |
︙ | ︙ | |||
631 632 633 634 635 636 637 | SELECT sqlite_rename_column(sql, type, object, db, tbl, icol, znew, bquote) FROM ddd; } {{} {} {}} finish_test | < | 631 632 633 634 635 636 637 | SELECT sqlite_rename_column(sql, type, object, db, tbl, icol, znew, bquote) FROM ddd; } {{} {} {}} finish_test |