Skip to content

Commit fddb223

Browse files
committed
Drop an unverified claim about who enables the option
The doc comment named a specific vendor's build as shipping with SQLITE_ENABLE_UPDATE_DELETE_LIMIT. I could not verify it, so it goes; the part that carries the argument -- the option rewrites the parser tables, so it lives in a build meyer cannot detect from the SQL -- stands on its own. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_019kZMPnsTWywB5icVh1wJrA
1 parent f887bbe commit fddb223

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

parser/parser.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -91,10 +91,10 @@ type Options struct {
9191
// UpdateDeleteLimit accepts ORDER BY and LIMIT on UPDATE and DELETE,
9292
// as SQLITE_ENABLE_UPDATE_DELETE_LIMIT does. The pinned build defines
9393
// neither it nor SQLITE_UDL_CAPABLE_PARSER, so those clauses are a
94-
// syntax error by default -- but the option is common enough (Android's
95-
// SQLite ships with it, and it is what updateDeleteLimitError in
96-
// parse.y exists to complain about) that the clauses have to be
97-
// parseable on request.
94+
// syntax error by default. The option selects grammar rules, so a
95+
// database built with it can only be built from canonical sources --
96+
// which also means meyer cannot tell from the SQL alone, and the caller
97+
// has to say.
9898
//
9999
// cmd ::= with DELETE FROM xfullname indexed_opt where_opt_ret
100100
// orderby_opt limit_opt.

0 commit comments

Comments
 (0)