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

Jens Geyer commented on THRIFT-3457:
------------------------------------

That's part of a bigger problem. The easiest (and by all means worst) fix is to 
add that thing to the global keywords list. I mention it here, because sooner 
or later you will find that construct anyway. *But I would like it to be fixed 
in a less invasive way: in the D generator.* 

Unfortunately we don't have a common solution yet (I'd love if we had one) but 
some isolated attempts to deal with that problem. Look at the Go, Delphi or C# 
generators. There is some internal handling to detect and circumvent such 
problems. If one of these solutions could be applied to the D generator in some 
way, please do.

> thrift client codegen fails due to problematic names
> ----------------------------------------------------
>
>                 Key: THRIFT-3457
>                 URL: https://issues.apache.org/jira/browse/THRIFT-3457
>             Project: Thrift
>          Issue Type: Bug
>          Components: D - Compiler
>            Reporter: Tolstokulakov Nikolay
>
> thrift/source/thrift/codegen/client.d-mixin-86(272,19): Error: function 
> expected before (), not skip of type int
> This happens if I have parameter with name 'skip' in my interface:
>  enum methodMeta = [
>     TMethodMeta(`getBrands`, 
>       [TParamMeta(`limit`, 1), TParamMeta(`skip`, 2)]
>     ), ....
> As workaround  I rename parameter, but compiler error is not very clear in 
> this case.



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

Reply via email to