On Sat, Jun 25, 2011 at 11:34:01AM +0200, Denys Vlasenko wrote:
> Slight optimization and cleanup in trace():
> 
> - do not recalculate "cflag ? &ru : NULL" again and again
> - do not clear errno unnecessarily
> - consistently check wait errors as pid < 0, not pid == -1
> - indent ifdefs for better readability
> - remove comments after endif if ifdef/endif block is really tiny
> 
> Does it look ok to you?

OK for me.


-- 
ldv

Attachment: pgpXYulEXDj0V.pgp
Description: PGP signature

------------------------------------------------------------------------------
BlackBerry&reg; DevCon Americas, Oct. 18-20, San Francisco, CA
The must-attend event for mobile developers. Connect with experts. 
Get tools for creating Super Apps. See the latest technologies.
Sessions, hands-on labs, demos & much more. Register early & save!
http://p.sf.net/sfu/rim-blackberry-1
_______________________________________________
Strace-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/strace-devel

Reply via email to