You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by bforbis <gi...@git.apache.org> on 2017/10/08 00:01:38 UTC

[GitHub] thrift pull request #1388: THRIFT-4355 Fix javascript indentation of default...

GitHub user bforbis opened a pull request:

    https://github.com/apache/thrift/pull/1388

    THRIFT-4355 Fix javascript indentation of default values

    Using the following test thrift file to test indentation in generated ttypes output.
    
    
    ```struct Object {
      1: map<string, string> hashWithDefault = {"foo": "bar", "baz": "bat"}
      2: list<string> arrayWithDefault = ["foo", "bar", "baz"]
      3: OtherObject otherObject
      4: string foostring
    }
    struct OtherObject {
      1: Object objectWithDefault = {"hashWithDefault": {"baz": "bat", "bam": "bot"}, "arrayWithDefault": ["a", "b", "c"], "foostring": "foostringvalue"
      2: string primitiveStringType
    }


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/bforbis/thrift THRIFT-4355-javascript-indentation-incorrect

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/thrift/pull/1388.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1388
    
----
commit a2dbbfd0fe8e4b74ea78a4a7c46903c6f28fa737
Author: Brian Forbis <bf...@athenahealth.com>
Date:   2017-10-07T23:57:40Z

    THRIFT-4355 Fix javascript indentation of default values

----


---

[GitHub] thrift pull request #1388: THRIFT-4355 Fix javascript indentation of default...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/thrift/pull/1388


---