[Patch] debug_printf edits

Christopher Faylor cgf-no-personal-reply-please@cygwin.com
Sat Nov 20 19:36:00 GMT 2004


On Sat, Nov 20, 2004 at 02:27:37PM -0500, Pierre A. Humblet wrote:
>At 02:20 PM 11/20/2004 -0500, Christopher Faylor wrote:
>>On Sat, Nov 20, 2004 at 01:51:16PM -0500, Pierre A. Humblet wrote:
>>>Here are minor changes that facilitate grepping traces. 
>>>
>>>Pierre
>>>
>>>2004-11-20  Pierre Humblet <pierre.humblet@ieee.org>
>>>
>>>	* fhandler.cc (fhandler::write): Remove debug_printf.
>>>	* pipe.cc (fhandler_pipe::create): Edit syscall_printf format.
>>
>>What does the inclusion of the word "pipe" add for grepping traces?
>>So, you can do a grep on '[0-9]+ = [a-z]'?
>
>Most syscall_printf have the form "res = function (args)",
>but the one in pipe.cc is missing "function". 
>So the change facilitates grepping, in particular when looking
>for many functions at once.

It didn't make sense to me that you were removing this usage in the case
of fhandler::write but tweaking it in the case of fhandler_pipe::create.

cgf



More information about the Cygwin-patches mailing list