You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@thrift.apache.org by "Hudson (JIRA)" <ji...@apache.org> on 2014/04/27 19:55:15 UTC

[jira] [Commented] (THRIFT-2495) JavaScript/Node string constants lack proper escaping

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

Hudson commented on THRIFT-2495:
--------------------------------

SUCCESS: Integrated in Thrift #1143 (See [https://builds.apache.org/job/Thrift/1143/])
THRIFT-2495 JavaScript/Node string constants lack proper escaping (jensg: rev 3290336e33a257d77d731aa8f4c86af84bd8f9ad)
* compiler/cpp/src/generate/t_js_generator.cc


> JavaScript/Node string constants lack proper escaping
> -----------------------------------------------------
>
>                 Key: THRIFT-2495
>                 URL: https://issues.apache.org/jira/browse/THRIFT-2495
>             Project: Thrift
>          Issue Type: Bug
>          Components: JavaScript - Compiler, Node.js - Compiler
>    Affects Versions: 0.9
>            Reporter: John Vines
>            Assignee: Jens Geyer
>             Fix For: 0.9.2
>
>         Attachments: test.thrift, thrift-2495-js-missing-escaping.patch
>
>
> In my .thrift file, I have an optional field with a default value of "\\". In java generated code it is properly set to "\\", but in the node.js code it results in "\" which results in an error.



--
This message was sent by Atlassian JIRA
(v6.2#6252)