[lld] r190554 - Run clang-format.

Rui Ueyama ruiu at google.com
Wed Sep 11 19:23:48 PDT 2013


On Wed, Sep 11, 2013 at 7:09 PM, Shankar Easwaran
<shankare at codeaurora.org>wrote:

> On 9/11/2013 6:29 PM, Rui Ueyama wrote:
>
>> I can resurrect the table layout and fix only the indentation errors if
>> you want.
>>
> Yes, that would be nice.
>

Because this file had been left untouched with such indentation errors, and
some of them are actually confusing, I don't think that we really care much
about the layout. And because you are a person who made one of the
indentation error, I guess you'd want to use clang-format to prevent such
error in future. If I hand-format this file you won't be able to run the
command on this file, or otherwise you'll have to hand-format it after
editing a line part of a table.

So I feel like I'd want to let clang-format do the stuff and stop worrying
about it. I'm not sticking to this point, so if there's still a push-back,
I'm ok to resurrect the table layout though.


> Thanks
>
> Shankar Easwaran
>
> --
> Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted
> by the Linux Foundation
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130911/80f39f13/attachment.html>


More information about the llvm-commits mailing list