A few more comments:

 - The patch adds just 8KB to libsqlite3.a and the shell, and this is
true regardless of whether the before and after compilations use -g or
not, -O0 vs. -O2, and -DSQLITE_DEBUG=1 or not.

 - I've tried to keep the SQLite3 C style as best I could.

 - My earlier characterization of performance and memory footprint
impact of this patch has not changed, but I can now also tell you that
the memory footprint (excluding cache and alignment effects) is two
bytes per-attached DB.

 - I will probably add a SQLITE_OMIT_DB_TRIGGERS #define so that DB
triggers can be omitted separately from triggers, just in case that
8KB of text is precious to folks who use triggers (anyone?).

 - There's a bug in my patch where the AFTER BEGIN triggers, if
defined, run before any AFTER CONNECT triggers, if defined.  I've got
a fix for that.

After I switch to Fossil, write tests, and cleanup, I'll stop there --
the unimplemented DB triggers can be added later, possibly by someone
else as I don't need them.

Nico
--
_______________________________________________
sqlite-users mailing list
sqlite-users@sqlite.org
http://sqlite.org:8080/cgi-bin/mailman/listinfo/sqlite-users

Reply via email to