[llvm-commits] [PATCH] Remove the use of C string function in Errno.cpp

Martinez, Javier E javier.e.martinez at intel.com
Thu Sep 27 23:04:05 PDT 2012


Dmitri,

Here's the version without the trailing whitespace.

Thanks,
Javier

-----Original Message-----
From: Dmitri Gribenko [mailto:gribozavr at gmail.com] 
Sent: Thursday, September 27, 2012 10:51 PM
To: Martinez, Javier E
Cc: llvm-commits at cs.uiuc.edu
Subject: Re: [llvm-commits] [PATCH] Remove the use of C string function in Errno.cpp

On Fri, Sep 28, 2012 at 7:45 AM, Martinez, Javier E <javier.e.martinez at intel.com> wrote:
> Dmitri,
>
> Great suggestion. Attached is the revised patch.

Trailing whitespace after 'if (errnum)'.  Other than that, LGTM.

Dmitri

--
main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
(j){printf("%d\n",i);}}} /*Dmitri Gribenko <gribozavr at gmail.com>*/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Errno.cpp.patch
Type: application/octet-stream
Size: 1332 bytes
Desc: Errno.cpp.patch
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120928/a9e74e17/attachment.obj>


More information about the llvm-commits mailing list