← Back to team overview

zorba-coders team mailing list archive

Re: [Merge] lp:~zorba-coders/zorba/bug1109853 into lp:zorba/jdbc-module

 

Review: Needs Fixing

Unless I'm missing something, the following if condition (and the corresponding block of code) is repeated twice in jsonitemsequence.cpp :

+        } else if (SQLTypes::isString(columnTypes[i])) {

Also, what's the story behind the various new  // ****  and // ----  comments?
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1109853/+merge/145525
Your team Zorba Coders is subscribed to branch lp:zorba/jdbc-module.


References