r.stahl added a comment.
@xazax.hun Yes, I planned to just commit. Set you as Subscriber not Reviewer in
Arc. Was just a bit confused why it fails at first :)
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D61002/new/
https://reviews.llvm.org/D61002
___
xazax.hun accepted this revision.
xazax.hun added a comment.
This revision is now accepted and ready to land.
Herald added a subscriber: rnkovacs.
Feel free to commit such trivial fixes without reviews. Alternatively, you
could use LLVM_FALLTHROUGH, but I have no strong preference in this case.
r.stahl abandoned this revision.
r.stahl added a comment.
Was already done:
https://github.com/llvm-mirror/clang/commit/bacdda22396c39181aa0e641182e01a0b3cf43ea
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D61002/new/
https://reviews.llvm.org/D61002
_
r.stahl created this revision.
Herald added subscribers: cfe-commits, Charusso, dkrupp, donat.nagy, Szelethus,
mikhail.ramalho, a.sidorin, szepet, baloghadamsoftware.
Herald added a project: clang.
Missing break/fallthrough
Repository:
rC Clang
https://reviews.llvm.org/D61002
Files:
tools
