You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Patrick Collison (JIRA)" <ji...@apache.org> on 2008/07/15 02:47:32 UTC

[jira] Created: (THRIFT-82) Common Lisp support

Common Lisp support
-------------------

                 Key: THRIFT-82
                 URL: https://issues.apache.org/jira/browse/THRIFT-82
             Project: Thrift
          Issue Type: New Feature
            Reporter: Patrick Collison
         Attachments: thrift-cl.patch

Common Lisp support is attached

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (THRIFT-82) Common Lisp support

Posted by "Patrick Collison (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/THRIFT-82?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Patrick Collison updated THRIFT-82:
-----------------------------------

    Attachment: thrift-cl.patch

> Common Lisp support
> -------------------
>
>                 Key: THRIFT-82
>                 URL: https://issues.apache.org/jira/browse/THRIFT-82
>             Project: Thrift
>          Issue Type: New Feature
>            Reporter: Patrick Collison
>         Attachments: thrift-cl.patch
>
>
> Common Lisp support is attached

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Re: [jira] Commented: (THRIFT-82) Common Lisp support

Posted by Chad Walters <ch...@powerset.com>.
>David Reiss commented on THRIFT-82:
>...
>Let's see how this turns out: http://wiki.apache.org/thrift/NamingVote

David,

Where do you intend for these names to be used? Directories? Namespace identifiers in .thrift? Options to the compiler?

Chad

On 7/14/08 5:59 PM, "David Reiss (JIRA)" <ji...@apache.org> wrote:



    [ https://issues.apache.org/jira/browse/THRIFT-82?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=12613505#action_12613505 <https://issues.apache.org/jira/browse/THRIFT-82?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12613505#action_12613505>  ]

David Reiss commented on THRIFT-82:
-----------------------------------

What is the point of temporary_var?
This should inherit directly from t_generator, not from the poorly-named t_oop_generator, which is for c-style syntax.
generated_package is unused, and I don't think it works because of the hyphen.
In the README, why do you pass the client to the server?
Let's see how this turns out: http://wiki.apache.org/thrift/NamingVote

> Common Lisp support
> -------------------
>
>                 Key: THRIFT-82
>                 URL: https://issues.apache.org/jira/browse/THRIFT-82
>             Project: Thrift
>          Issue Type: New Feature
>            Reporter: Patrick Collison
>         Attachments: thrift-cl.patch
>
>
> Common Lisp support is attached

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.




[jira] Commented: (THRIFT-82) Common Lisp support

Posted by "David Reiss (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/THRIFT-82?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12613505#action_12613505 ] 

David Reiss commented on THRIFT-82:
-----------------------------------

What is the point of temporary_var?
This should inherit directly from t_generator, not from the poorly-named t_oop_generator, which is for c-style syntax.
generated_package is unused, and I don't think it works because of the hyphen.
In the README, why do you pass the client to the server?
Let's see how this turns out: http://wiki.apache.org/thrift/NamingVote

> Common Lisp support
> -------------------
>
>                 Key: THRIFT-82
>                 URL: https://issues.apache.org/jira/browse/THRIFT-82
>             Project: Thrift
>          Issue Type: New Feature
>            Reporter: Patrick Collison
>         Attachments: thrift-cl.patch
>
>
> Common Lisp support is attached

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.