Skip to content

sqlite: make sourceSQL and expandedSQL string-valued properties

Rodrigo Muino Tomonari requested to merge github/fork/tniessen/sqlite-sourcesql-expandedsql-properties into main

Change sourceSQL and expandedSQL from being methods to being string-valued properties. These fields

  • are conceptually properties (and not actions),
  • are derived deterministically from the current state of the object,
  • require no parameters, and
  • are inexpensive to compute.

Also, following the naming conventions of ECMAScript for new features, most function names should usually contain a verb, whereas names of (dynamically computed) properties generally should not, so the current names also seem more appropriate for properties than for functions.

Merge request reports