Code review comment for lp:~zorba-coders/zorba/bug1109853

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?