zorba-coders team mailing list archive
-
zorba-coders team
-
Mailing list archive
-
Message #11556
Re: [Merge] lp:~zorba-coders/zorba/jsoniq into lp:zorba
Review: Needs Fixing
- the comment of some functions says "(: obsolete - use ... instead :)". I think this should say that the function is redundant and syntax could be used instead.
- query
{
"n1" : 1
},
{
"n2" : 2
}
result:
zorba: /home/mbrantner/zorba/jsandbox/src/diagnostics/dict.cpp:80: const char* zorba::diagnostic::dict::lookup(const char*): Assertion `result.first != result.second' failed.
{ "n1" : 1 }
because trying to serialize a sequence of objects
- in a simple constructor query, the function SimpleJSONObject::assertInvariant takes 90% of the time. This makes a huge performance difference between debug and release mode. I understand that such assertions are useful but maybe that's a little too much even for debug mode. I think NDEBUG is even not set for RelWithDebInfo. Hence, it's almost impossible to do a proper profile.
- /home/mbrantner/zorba/jsandbox/src/store/naive/json_items.cpp: In member function ‘virtual zorba::store::Item_t zorba::simplestore::json::SimpleJSONObject::getObjectValue(const zorba::store::Item_t&) const’:
/home/mbrantner/zorba/jsandbox/src/store/naive/json_items.cpp:565:9: warning: ‘lPosition’ may be used uninitialized in this function
/home/mbrantner/zorba/jsandbox/src/store/naive/json_items.cpp: In member function ‘virtual zorba::store::Item_t zorba::simplestore::json::SimpleJSONObject::setValue(const zorba::store::Item_t&, const zorba::store::Item_t&)’:
/home/mbrantner/zorba/jsandbox/src/store/naive/json_items.cpp:368:9: warning: ‘lPosition’ may be used uninitialized in this function
/home/mbrantner/zorba/jsandbox/src/store/naive/json_items.cpp: In member function ‘virtual zorba::store::Item_t zorba::simplestore::json::SimpleJSONObject::remove(const zorba::store::Item_t&)’:
/home/mbrantner/zorba/jsandbox/src/store/naive/json_items.cpp:271:9: warning: ‘lPosition’ may be used uninitialized in this function
--
https://code.launchpad.net/~zorba-coders/zorba/jsoniq/+merge/92171
Your team Zorba Coders is requested to review the proposed merge of lp:~zorba-coders/zorba/jsoniq into lp:zorba.
Follow ups
References