[ 
https://issues.apache.org/jira/browse/SLING-6536?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15875643#comment-15875643
 ] 

Stefan Seifert commented on SLING-6536:
---------------------------------------

someone extracted the JSON implementation from android here:
https://github.com/kbsriram/android-json
but this page states as well that the API was derived from the crockford 
library, but the two APIs have diverged since then.

> Replace JSON implementation derived from org.json code
> ------------------------------------------------------
>
>                 Key: SLING-6536
>                 URL: https://issues.apache.org/jira/browse/SLING-6536
>             Project: Sling
>          Issue Type: Improvement
>          Components: Commons
>    Affects Versions: Commons JSON 2.0.18
>            Reporter: Stefan Seifert
>            Priority: Critical
>             Fix For: Commons JSON 2.1.0
>
>
> following the discussion in
> https://lists.apache.org/thread.html/ee51bace078681765d5dcfeda1939628ccefb9b4261b1d7f6a56d420@%3Cdev.sling.apache.org%3E
> we have to replace the implementation of all classes in Commons JSON that 
> were derived from the original org.json code.
> the affected packages are
> * 
> [org.apache.sling.commons.json|https://github.com/apache/sling/tree/trunk/bundles/commons/json/src/main/java/org/apache/sling/commons/json]
> * 
> [org.apache.sling.commons.json.http|https://github.com/apache/sling/tree/trunk/bundles/commons/json/src/main/java/org/apache/sling/commons/json/http]
> * 
> [org.apache.sling.commons.json.io|https://github.com/apache/sling/tree/trunk/bundles/commons/json/src/main/java/org/apache/sling/commons/json/io]
> * 
> [org.apache.sling.commons.json.util|https://github.com/apache/sling/tree/trunk/bundles/commons/json/src/main/java/org/apache/sling/commons/json/util]
> * 
> [org.apache.sling.commons.json.xml|https://github.com/apache/sling/tree/trunk/bundles/commons/json/src/main/java/org/apache/sling/commons/json/xml]
> unfortunately the unit test coverage for those packages is not very high. the 
> main package has a line coverage of 42%, the other packages less than that or 
> no at all.
> although we might encourage modules to switch to a standard JSON 
> implementation a lot of modules inside sling and perhaps much more outside 
> sling depend on this json implementation, so we should try to keep the 
> exported API and functionality.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to