:Release Notes:
Fix parsing negative fractional numbers with zero whole part.
:Detailed Notes:
Proprietary number parsing algorithm is buggy. Whole part was used to
carry sign to the fractional part. If the whole part is zero, the
information about the sign was lost.
:Testing Performed:
Unit tests.
:QA Notes:
:Issues Addressed:
[BHV-12121] SmartShare application doesn't have support for -1/2x rewind
Open-webOS-DCO-1.0-Signed-off-by: Anatolii Sakhnik <anatolii.sakhnik@lge.com>
Change-Id: I6002daa31d1132c26df4bccefa4d11072305b0ab
Reviewed-on: https://g2g.palm.com/6371
Reviewed-by: DCO Verification
Tested-by: Anatolii Sakhnik <anatolii.sakhnik@lge.com>
Reviewed-by: Andrii Zora <andrii.zora@lge.com>
Reviewed-by: Oleksii Rudenko <oleksii.rudenko@lge.com>
Reviewed-by: Anatolii Sakhnik <anatolii.sakhnik@lge.com>
:Release Notes:
:Detailed Notes:
- removed unused stack of fragments with associated tests
- escape_json_pointer changed to work with non-ASCIIz strings
- fixed compiler/gcc/builtins.h to be able included alone without others
- factored out uri_scope_get_uri_unsafe as access to top of uri_scope
with expectation that it present
:Testing Performed:
unit tests under desktop
:QA Notes:
:Issues Addressed:
[BHV-8631] libpbnjson: fix 'oneOf' behaviour for fragments without id's
Open-webOS-DCO-1.0-Signed-off-by: Nikolay Orliuk <nikolay.orliuk@lge.com>
Change-Id: Ic9256d8388b3ab35fd4b588cadb6c59864302570
Reviewed-on: https://g2g.palm.com/5797
Reviewed-by: DCO Verification
Reviewed-by: Nikolay Orliuk <nikolay.orliuk@lge.com>
Tested-by: Nikolay Orliuk <nikolay.orliuk@lge.com>
Reviewed-by: Anatolii Sakhnik <anatolii.sakhnik@lge.com>
:Release Notes:
Fix message "The same fragment XXX is added second time!" reported
during oneOf/anyOf/allOf schemas parsing. Fix for refs in definitions.
:Detailed Notes:
- put assert for fragment duplication check to track such issues
- factor out collect_schema from collect_uri_enter to grab more control
over process of traversing
- cleaned out address calculation for non-addressable elements
- collect definitions with ref-schema
- add tests for definitions with ref-schema
:Testing Performed:
unit tests under desktop build
:QA Notes:
:Issues Addressed:
[BHV-8631] libpbnjson: fix 'oneOf' behaviour for fragments without id's
Open-webOS-DCO-1.0-Signed-off-by: Nikolay Orliuk <nikolay.orliuk@lge.com>
Change-Id: If92cef529bb7f24b94c5d547fa664f2418595b41
Reviewed-on: https://g2g.palm.com/5766
Reviewed-by: DCO Verification
Reviewed-by: Nikolay Orliuk <nikolay.orliuk@lge.com>
Tested-by: Nikolay Orliuk <nikolay.orliuk@lge.com>
Reviewed-by: Anatolii Sakhnik <anatolii.sakhnik@lge.com>
:Release Notes:
Add NO_LOGGING cmake flag to allow building
without any logging, to avoid PmLogLib dependency.
:Detailed Notes:
:Testing Performed:
Unit-tests.
:QA Notes:
:Issues Addressed:
[BHV-8] pbnjson: Refactor logging to use pmloglib
Open-webOS-DCO-1.0-Signed-off-by: Alexandr Smoktal <alexandr.smoktal@lge.com>
Change-Id: Ie034c50d6120ced7e5806df0ec1e8e126c38361e
Reviewed-on: https://g2g.palm.com/5177
Reviewed-by: DCO Verification
Reviewed-by: Alexandr Smoktal <alexandr.smoktal@lge.com>
Tested-by: Alexandr Smoktal <alexandr.smoktal@lge.com>
Reviewed-by: Anatolii Sakhnik <anatolii.sakhnik@lge.com>
:Release Notes:
Provide getType() method for JSON value.
:Detailed Notes:
Add getType() method to JValue class.
Add new method to unit tests.
:Testing Performed:
Unit tests
:QA Notes:
:Issues Addressed:
[BHV-36] pbnjson: Provide JSON value type in C++ API
Open-webOS-DCO-1.0-Signed-off-by: Andrii Zora <andrii.zora@lge.com>
Change-Id: Ib326b95df46d75bcd3d9647dbb4a29d6cf1ab3cc
Reviewed-on: https://g2g.palm.com/5224
Reviewed-by: DCO Verification
Reviewed-by: Andrii Zora <andrii.zora@lge.com>
Tested-by: Andrii Zora <andrii.zora@lge.com>
Reviewed-by: Anatolii Sakhnik <anatolii.sakhnik@lge.com>
:Release Notes:
New API to parse schema from jvalue.
:Detailed Notes:
- tie together jtraverse and jschema_builder
- expose schema builder and parsing callbacks in public API
- add unit-tests for parsing schema from jvalue
:Testing Performed:
unit-tests
:QA Notes:
:Issues Addressed:
[BHV-3085] Provide pbnjson API for schema construction from jvalue
Open-webOS-DCO-1.0-Signed-off-by: Nikolay Orliuk <nikolay.orliuk@lge.com>
Change-Id: Ica96a9e7e6826487996e66e63785c49df2e6816c
Reviewed-on: https://g2g.palm.com/5145
Reviewed-by: DCO Verification
Reviewed-by: Nikolay Orliuk <nikolay.orliuk@lge.com>
Tested-by: Nikolay Orliuk <nikolay.orliuk@lge.com>
Reviewed-by: Anatolii Sakhnik <anatolii.sakhnik@lge.com>
:Release Notes:
Prepare code infrastructure to provide new API
:Detailed Notes:
- move yajl callbacks for schema parsing to schema_build.c as visitor
methods
- drop mentioning of yajl
- TODO: expose schema builder and parsing callbacks in public API
:Testing Performed:
unit-tests
:QA Notes:
:Issues Addressed:
[BHV-3085] Provide pbnjson API for schema construction from jvalue
Open-webOS-DCO-1.0-Signed-off-by: Nikolay Orliuk <nikolay.orliuk@lge.com>
Change-Id: I131e61a0848e62dc744abef3e280909e2798c58f
Reviewed-on: https://g2g.palm.com/5088
Reviewed-by: DCO Verification
Reviewed-by: Nikolay Orliuk <nikolay.orliuk@lge.com>
Tested-by: Nikolay Orliuk <nikolay.orliuk@lge.com>
Reviewed-by: Anatolii Sakhnik <anatolii.sakhnik@lge.com>
:Release Notes:
Prepare code infrastructure to provide new API
:Detailed Notes:
- factored out builder
- TODO: drop mentioning of yajl
- TODO: expose schema builder and parsing callbacks in public API
:Testing Performed:
unit-tests
:QA Notes:
:Issues Addressed:
[BHV-3085] Provide pbnjson API for schema construction from jvalue
Open-webOS-DCO-1.0-Signed-off-by: Nikolay Orliuk <nikolay.orliuk@lge.com>
Change-Id: I39e30dc9b39ae06d1dd53287a247f871eded7234
Reviewed-on: https://g2g.palm.com/5078
Reviewed-by: DCO Verification
Reviewed-by: Nikolay Orliuk <nikolay.orliuk@lge.com>
Tested-by: Nikolay Orliuk <nikolay.orliuk@lge.com>
Reviewed-by: Anatolii Sakhnik <anatolii.sakhnik@lge.com>
:Release Notes:
Extract the source code of the memory pool into its own file,
create unit tests for it.
:Detailed Notes:
Extract memory pool to parser_memory_pool.
Create unit tests TestParserMemPool.
Change size of the pool.
:Testing Performed:
Unit tests
:QA Notes:
:Issues Addressed:
[BHV-2369] pbnjson: Factor out the mempool into its own source file
Open-webOS-DCO-1.0-Signed-off-by: Andrii Zora <andrii.zora@lge.com>
Change-Id: I478c292522fbc319f7c8f1b14148b1986f78113a
Reviewed-on: https://g2g.palm.com/5203
Reviewed-by: DCO Verification
Reviewed-by: Andrii Zora <andrii.zora@lge.com>
Tested-by: Andrii Zora <andrii.zora@lge.com>
Reviewed-by: Sergiy Kryvonos <sergiy.kryvonos@lge.com>
:Release Notes:
Made test run successfully without linking with pmloglib
:Detailed Notes:
- Moved liblog.c to be compiled directly as a part of shared library.
Otherwise unexplained segfault happens when it comes from shared
library jvalue.
- Fixed usage and define of PJ_LOG_DBG macro.
- Fixed stub for _PmLogMsgKV
:Testing Performed:
Unit-tests.
:QA Notes:
:Issues Addressed:
[BHV-8] pbnjson: Refactor logging to use pmloglib
Open-webOS-DCO-1.0-Signed-off-by: Nikolay Orliuk <nikolay.orliuk@lge.com>
Change-Id: I6ffc5c1af487b58db058665077622e70f2c9c008
Reviewed-on: https://g2g.palm.com/5174
Reviewed-by: DCO Verification
Reviewed-by: Nikolay Orliuk <nikolay.orliuk@lge.com>
Tested-by: Nikolay Orliuk <nikolay.orliuk@lge.com>
Reviewed-by: Anatolii Sakhnik <anatolii.sakhnik@lge.com>
:Release Notes:
Changed to use new key-value pmloglib API.
:Detailed Notes:
To use PmLogLib key-value logging, user application can
be linked with PmLogLib library. This will enable logging
with PmLogLib automatically.
:Testing Performed:
Unit-tests. MiniBAT.
:QA Notes:
:Issues Addressed:
[BHV-8] pbnjson: Refactor logging to use pmloglib
Open-webOS-DCO-1.0-Signed-off-by: Alexandr Smoktal <alexandr.smoktal@lge.com>
Change-Id: Ie3a762963aa6cbe0be4731e7656736afaea7d4df
Reviewed-on: https://g2g.palm.com/4741
Reviewed-by: DCO Verification
Reviewed-by: Alexandr Smoktal <alexandr.smoktal@lge.com>
Tested-by: Alexandr Smoktal <alexandr.smoktal@lge.com>
Reviewed-by: Anatolii Sakhnik <anatolii.sakhnik@lge.com>
:Release Notes:
Add yajl_dont_validate_strings for Yajl 2.
Custom simple memory pool.
Throughput measurement in MBps.
Add pool of dom parsers.
:Detailed Notes:
Add yajl_dont_validate_strings for Yajl 2.
Memory pool gives us a big win in performance.
Fix performance tests to measure the throughput in MB/s.
Some small improvements and code refactoring.
:Testing Performed:
gtest
:QA Notes:
:Issues Addressed:
[BHV-123] pbnjson: Improve performance of json parsing with YAJL
Open-webOS-DCO-1.0-Signed-off-by: Andrii Zora <andrii.zora@lge.com>
Change-Id: Ic2b49103cf438c77e48f961258e9842fd84fb7d0
Reviewed-on: https://g2g.palm.com/4631
Reviewed-by: DCO Verification
Reviewed-by: Andrii Zora <andrii.zora@lge.com>
Tested-by: Andrii Zora <andrii.zora@lge.com>
Reviewed-by: Anatolii Sakhnik <anatolii.sakhnik@lge.com>
Tested-by: Anatolii Sakhnik <anatolii.sakhnik@lge.com>
:Release Notes:
Now pbnjson doesn't report error on comments in json
:Detailed Notes:
- turned on flag allow_comments for yajl
- bumped version to 2.5.0 for proper dependencies from other components
:Testing Performed:
qemux86 build together with luna-service2 that uses pbnjson instead of
cjson
:QA Notes:
:Issues Addressed:
[BHV-2034] We need either support for comments in libpbnjson or role-files stripping
Open-webOS-DCO-1.0-Signed-off-by: Nikolay Orliuk <nikolay.orliuk@lge.com>
Change-Id: I7836653e5bde58357104a5704b47911ddad432fc
Reviewed-on: https://g2g.palm.com/4911
Reviewed-by: DCO Verification
Reviewed-by: Nikolay Orliuk <nikolay.orliuk@lge.com>
Tested-by: Nikolay Orliuk <nikolay.orliuk@lge.com>
Reviewed-by: Anatolii Sakhnik <anatolii.sakhnik@lge.com>
:Release Notes:
Better reproducibility of benchmarking tests
:Detailed Notes:
- process speedup by avoiding extra cycle if warmup already hit the
measure_time
- tolerance to system load by using CPU usage time if available
- tolerance to spikes by using median instead of average
:Testing Performed:
gtest
:QA Notes:
:Issues Addressed:
[BHV-123] pbnjson: Improve performance of json parsing with YAJL
Open-webOS-DCO-1.0-Signed-off-by: Nikolay Orliuk <nikolay.orliuk@lge.com>
Open-webOS-DCO-1.0-Signed-off-by: Andrii Zora <andrii.zora@lge.com>
Change-Id: I661407ce08ef807361cb481578674f845c18570e
Reviewed-on: https://g2g.palm.com/4540
Reviewed-by: DCO Verification
Reviewed-by: Nikolay Orliuk <nikolay.orliuk@lge.com>
Tested-by: Nikolay Orliuk <nikolay.orliuk@lge.com>
Reviewed-by: Alekseyev Oleksandr <alekseyev.oleksandr@lge.com>
Reviewed-by: Anatolii Sakhnik <anatolii.sakhnik@lge.com>
:Release Notes:
jobject_create_var() should always iterate through all the arguments
even in case of early failure.
:Detailed Notes:
:Testing Performed:
Unit tests.
:QA Notes:
:Issues Addressed:
[ANG-552] pbnjson: jobject_create_var may fail with memory leak
Open-webOS-DCO-1.0-Signed-off-by: Anatolii Sakhnik <anatolii.sakhnik@lge.com>
Change-Id: Id7324a081695bfdbe626886ddad159414f9f874b
Reviewed-on: https://g2g.palm.com/4629
Reviewed-by: DCO Verification
Reviewed-by: Anatolii Sakhnik <anatolii.sakhnik@lge.com>
Tested-by: Anatolii Sakhnik <anatolii.sakhnik@lge.com>
Reviewed-by: Alekseyev Oleksandr <alekseyev.oleksandr@lge.com>
:Release Notes:
Made a difference between jnull() and invalid value by introducing
jinvalid().
:Detailed Notes:
- added jis_valid to check answer for erroneous result
- extended jis_null to cover jinvalid() also
- jinvalid() returns JV_NULL value
:Testing Performed:
unit tests and performance test on desktop
:QA Notes:
:Issues Addressed:
[ANG-547] pbnjson: clash between error and null-value as result of jdom_parse
Open-webOS-DCO-1.0-Signed-off-by: Nikolay Orliuk <nikolay.orliuk@lge.com>
Change-Id: I85b57891976a4bd6a8ef4ed74fec99d8f0b816ec
Reviewed-on: https://g2g.palm.com/4538
Reviewed-by: DCO Verification
Reviewed-by: Nikolay Orliuk <nikolay.orliuk@lge.com>
Tested-by: Nikolay Orliuk <nikolay.orliuk@lge.com>
Reviewed-by: Anatolii Sakhnik <anatolii.sakhnik@lge.com>
:Release Notes:
:Detailed Notes:
:Testing Performed:
unit-tests and performance test on desktop
:QA Notes:
:Issues Addressed:
[ANG-547] pbnjson: clash between error and null-value as result of jdom_parse
Open-webOS-DCO-1.0-Signed-off-by: Nikolay Orliuk <nikolay.orliuk@lge.com>
Change-Id: I100d2a67275ea902deea7189434dccfd4990a8e9
Reviewed-on: https://g2g.palm.com/4537
Reviewed-by: DCO Verification
Reviewed-by: Nikolay Orliuk <nikolay.orliuk@lge.com>
Tested-by: Nikolay Orliuk <nikolay.orliuk@lge.com>
Reviewed-by: Alexandr Smoktal <alexandr.smoktal@lge.com>
Reviewed-by: Anatolii Sakhnik <anatolii.sakhnik@lge.com>
Anatolii Sakhnik
Andrii Zora
Nikolay Orliuk
Alekseyev Oleksandr
Alexandr Smoktal
Mykola Kryvonos