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

Julian Reschke commented on OAK-2389:
-------------------------------------

a) Can we please document what encode() is for? It's not for JSON, as far as 
can tell.

b) I prefer the constants as they generate both more readable and more compact 
output for the common control characters

c) I don't see "case" in the source

> issues with JsopBuilder.encode and .escape
> ------------------------------------------
>
>                 Key: OAK-2389
>                 URL: https://issues.apache.org/jira/browse/OAK-2389
>             Project: Jackrabbit Oak
>          Issue Type: Bug
>            Reporter: Julian Reschke
>         Attachments: OAK-2389-b.diff, OAK-2389.diff
>
>
> 1) escape() escapes many characters that do not need to be escaped (>127)
> 2) encode() does not encode many control characters that would need to be 
> escaped when read through a JSON parser.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to