Review: Approve

Looks good - nice job!

I made three small changes:

1. Moved a <p/> in the metadata() doc to immediately after the leading sentence 
of the comment.

2. Moved SqliteModule() constructor definition out-of-line.

3. Per mailing list discussion, marked a number of module functions as 
nondeterministic rather than sequential (per Matthias).
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug-1189784/+merge/177489
Your team Zorba Coders is subscribed to branch lp:zorba/sqlite-module.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to     : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

Reply via email to