search

Quuxplusone / LLVMBugzillaTest

0 stars 0 forks source link

Support for SWIG #29442

Open Quuxplusone opened 8 years ago

Quuxplusone commented 8 years ago
Bugzilla Link PR30469
Status NEW
Importance P enhancement
Reported by Eugene Zelenko (eugene.zelenko@gmail.com)
Reported on 2016-09-20 20:59:40 -0700
Last modified on 2016-09-22 13:16:39 -0700
Version unspecified
Hardware All All
CC djasper@google.com, klimek@google.com, llvm-bugs@lists.llvm.org, zturner@google.com
Fixed by commit(s)
Attachments
Blocks
Blocked by
See also

I'll be great to have support for SWIG. LLDB has a lot of SWIG files in scripts/interface which currently formatted in old way, what is inconsistent with rest of code formatted in LLVM style.

Quuxplusone commented 8 years ago

I'm not sure I understand what you're asking with this bug.

Do you mean that we should reformat the swig interface files to confirm to LLVM coding style?

Quuxplusone commented 8 years ago 
(In reply to comment #1)
> I'm not sure I understand what you're asking with this bug.
>
> Do you mean that we should reformat the swig interface files to confirm to
> LLVM coding style?

Clang-format should be improved first to handle SWIG files. Then SWIG files in
LLDB could be reformatted.
Quuxplusone commented 8 years ago

Ahh, right. I usually don't get CC'ed on bugs about clang-format, so I assumed this bug was about LLDB. But you've marked the clang-format component. I get it now.