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

[jira] Updated: (THRIFT-77) Remove fprintf from C++ lib

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

Alexander Shigin updated THRIFT-77:
-----------------------------------

    Attachment: thrift-lib-cpp-remove-printf.patch

> Remove fprintf from C++ lib
> ---------------------------
>
>                 Key: THRIFT-77
>                 URL: https://issues.apache.org/jira/browse/THRIFT-77
>             Project: Thrift
>          Issue Type: Improvement
>          Components: Library (C++)
>            Reporter: Alexander Shigin
>            Priority: Minor
>         Attachments: thrift-lib-cpp-remove-printf.patch
>
>
> The patch introduces two new methods for TOutput: perror and printf. These two methods are needed because:
>   1. TNonblocking has a couple of "fprintf(stderr, ""...);" clauses. 
>   2. There is a lot similar lines in lib/cpp:
> {code}
> string errStr = "TNonblockingServer::Task: close, possible resource leak "  + TOutput::strerror_s(errno);
> GlobalOutput(errStr.c_str());
> {code}
> The patch also uses the introduced methods to fix all these issues.

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