[lld] r190554 - Run clang-format.
Rui Ueyama
ruiu at google.com
Wed Sep 11 16:29:10 PDT 2013
I ran clang-format because the file has many indentation errors. I've
noticed that that would also change the table layout but I thought that
that was not really bad.
It looks like there's no way to tell clang-format to keep this table
layout. That means we wouldn't be able to use clang-format on this file, or
hand-edit after running clang-format, if we maintain this table layout.
I can resurrect the table layout and fix only the indentation errors if you
want.
On Wed, Sep 11, 2013 at 4:19 PM, Nick Kledzik <kledzik at apple.com> wrote:
> The YAML I/O templates are basically tables of how to map yaml keys to
> struct fields.
> I had all them all carefully aligned into two columns (key and field).
> clang-format
> went and destroyed all that.
>
> Is there someway to tell clang-format to not change some formatting?
> Or should we just not run clang-format blindly?
>
> -Nick
>
> On Sep 11, 2013, at 3:21 PM, Rui Ueyama <ruiu at google.com> wrote:
>
> - io.mapRequired("name", keys->_name);
> - io.mapOptional("ref-name", keys->_refName, StringRef());
> - io.mapOptional("scope", keys->_scope);
> - io.mapRequired("value", keys->_value);
> + io.mapRequired("name", keys->_name);
> + io.mapOptional("ref-name", keys->_refName, StringRef());
> + io.mapOptional("scope", keys->_scope);
> + io.mapRequired("value", keys->_value);
> }
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130911/5958d9e5/attachment.html>
More information about the llvm-commits
mailing list