[llvm-commits] Tablgen bugfix

[Malul, Elior]

Hi Dag, thx for the comment.
There were two lines that exceeded the 80-chars limit. (Fixed it).
Thx, Elior

-----Original Message-----
From: dag at cray.com [mailto:dag at cray.com] 
Sent: Tuesday, January 22, 2013 18:20
To: Malul, Elior
Cc: llvm-commits at cs.uiuc.edu
Subject: Re: [llvm-commits] Tablgen bugfix

"Malul, Elior" <elior.malul at intel.com> writes:

> Yes it is, in fact the logics in this line has not been changed. (I just replaced the hard-coded '80' with MAX_LINE_LEN).

Sorry, I was asking about the coding standards, not the output line
length.  Is the loop statement within the 80 char limit?

                           -David

> Thx, Elior
>
> -----Original Message-----
> From: dag at cray.com [mailto:dag at cray.com] 
> Sent: Monday, January 21, 2013 22:31
> To: Malul, Elior
> Cc: llvm-commits at cs.uiuc.edu
> Subject: Re: [llvm-commits] Tablgen bugfix
>
> "Malul, Elior" <elior.malul at intel.com> writes:
>
>> Attached it a fix to the function, which when it encounters a 
>> description string longer than 80 character, it splits to several 
>> lines.  I will appreciate if those who care about tblgen will review 
>> it.
>
>> +  for (unsigned i = OS.tell() - Pos, e = MAX_LINE_LEN - 
>> + Suffix.size(); i < e; ++i)
>
> Is this still within the 80 column limit?
>
> Otherwise, LGTM.
>
>                           -David
> ---------------------------------------------------------------------
> Intel Israel (74) Limited
>
> This e-mail and any attachments may contain confidential material for
> the sole use of the intended recipient(s). Any review or distribution
> by others is strictly prohibited. If you are not the intended
> recipient, please contact the sender and delete all copies.
---------------------------------------------------------------------
Intel Israel (74) Limited

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: tblgen_header_bugfix.patch
Type: application/octet-stream
Size: 1965 bytes
Desc: tblgen_header_bugfix.patch
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130123/bac37fc5/attachment.obj>