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

T Jake Luciani edited comment on THRIFT-554 at 7/30/09 6:32 PM:
----------------------------------------------------------------

Committed thx.

      was (Author: tjake):
    Commited thx.
  
> Perl improper namespace check for exception handling and writeMessageEnd 
> missing on processor calls
> ---------------------------------------------------------------------------------------------------
>
>                 Key: THRIFT-554
>                 URL: https://issues.apache.org/jira/browse/THRIFT-554
>             Project: Thrift
>          Issue Type: Bug
>          Components: Compiler (Perl)
>         Environment: Ubuntu Jaunty x86_64
>            Reporter: Eric Waters
>            Assignee: Eric Waters
>            Priority: Minor
>             Fix For: 0.2
>
>         Attachments: thrift-554.diff
>
>
> There are two more issues affecting the functioning of a Perl service server:
>  1. Failure to prepend the Perl namespace to the exception name when checking 
> the exception type from a eval'ed method call.
>  2. writeMessageEnd() should be present after a method call writes its result.
> I'm attaching a patch which addresses these issues, in addition to the 
> following more minor changes:
>  1. Tried to make indentation and line breaks more consistent to ensure 
> readability of the generated code.
>  2. Added a few best practice ideas to improve the code in minor ways.
>  3. Added a readAll() function to the Thrift::MemoryBuffer, as the one found 
> in Thrift::Transport uses a while loop to consume the data, which results in 
> a endless loop.

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

Reply via email to