[llvm-commits] [llvm] r135289 - in /llvm/trunk: include/llvm/Object/ObjectFile.h tools/llvm-objdump/llvm-objdump.cpp

Eric Christopher echristo at apple.com
Mon Jul 18 11:27:44 PDT 2011


On Jul 15, 2011, at 11:39 AM, Benjamin Kramer wrote:

> +        if (error(si->getName(Name))) break;

I know it isn't your fault but this idiom pains me.

The patches themselves look great.

Thanks!

-eric
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20110718/94205ede/attachment.html>


More information about the llvm-commits mailing list