This revision was automatically updated to reflect the committed changes.
Closed by commit rL327629: [change-namespace] Don't match a function
call/ref multiple times. (authored by ioeric, committed by ).
Herald added a subscriber: llvm-commits.
Repository:
rL LLVM
https://reviews.llvm.org/D44
This revision was automatically updated to reflect the committed changes.
Closed by commit rCTE327629: [change-namespace] Don't match a function
call/ref multiple times. (authored by ioeric, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D44517?vs=138550&id=138556#toc
Reposi
hokein accepted this revision.
hokein added a comment.
This revision is now accepted and ready to land.
LGTM.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D44517
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.
ioeric updated this revision to Diff 138550.
ioeric added a comment.
- small fix.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D44517
Files:
change-namespace/ChangeNamespace.cpp
unittests/change-namespace/ChangeNamespaceTests.cpp
Index: unittests/change-namespace/ChangeN
ioeric created this revision.
ioeric added a reviewer: hokein.
Herald added subscribers: cfe-commits, klimek.
Previously, the matcher matches a function call/ref multiple times, one
for each decl ancestor. This might cause problems. For example, in the following
case, `func()` would be matched onc
