zorba-coders team mailing list archive
-
zorba-coders team
-
Mailing list archive
-
Message #16535
Re: [Merge] lp:~zorba-coders/zorba/zorba-for-sqlite into lp:zorba
Review: Needs Fixing
Code review comments:
1. Many of the functions are not marked %an:sequential. I believe all of them should be.
2. s:disconnect() returns xs:anyURI, but the documentation says it returns "true if everything went OK". I suspect it should be xs:boolean.
See also the discussion in email about JDBC module API vs. SQLite module API.
--
https://code.launchpad.net/~zorba-coders/zorba/zorba-for-sqlite/+merge/139108
Your team Zorba Coders is subscribed to branch lp:zorba.
References