You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Jens Geyer (JIRA)" <ji...@apache.org> on 2016/01/09 00:45:40 UTC

[jira] [Resolved] (THRIFT-3314) Dots in file names of includes causes dots in javascript variable names

     [ https://issues.apache.org/jira/browse/THRIFT-3314?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jens Geyer resolved THRIFT-3314.
--------------------------------
       Resolution: Fixed
         Assignee: Kapil Joshi
    Fix Version/s: 1.0

Committed.

> Dots in file names of includes causes dots in javascript variable names
> -----------------------------------------------------------------------
>
>                 Key: THRIFT-3314
>                 URL: https://issues.apache.org/jira/browse/THRIFT-3314
>             Project: Thrift
>          Issue Type: Bug
>          Components: JavaScript - Compiler
>            Reporter: Matt Fysh
>            Assignee: Kapil Joshi
>             Fix For: 1.0
>
>         Attachments: thrift-3314-feature.patch
>
>
> Hi there,
> {code:title=a.thrift}
> include "b.c.thrift"
> {code}
> {code:title=b.c.thrift}
> # nada
> {code}
> {{$ thrift -r --gen js:node a.thrift}}
> {code:title=gen-nodejs/a_types}
> //
> // Autogenerated by Thrift Compiler (0.9.2)
> //
> // DO NOT EDIT UNLESS YOU ARE SURE THAT YOU KNOW WHAT YOU ARE DOING
> //
> var thrift = require('thrift');
> var Thrift = thrift.Thrift;
> var Q = thrift.Q;
> var b.c_ttypes = require('./b.c_types')
> var ttypes = module.exports = {};
> {code}
> One variable in the auto-generated thrift output contains a dot, which is not valid JavaScript and will not compile.
> Cheers,



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