[
https://issues.apache.org/jira/browse/THRIFT-77?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12612967#action_12612967
]
David Reiss commented on THRIFT-77:
-----------------------------------
I want to make a few tweaks to this patch, but I can't get to apply. Can you
email your ssh public key to the list so we can give you an account on
thrift-rpc.org and you can push your branch? Instructions here:
<http://wiki.apache.org/thrift/GitRepository>.
> 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.
