]> granicus.if.org Git - clang/commit
Don't leak CXStrings for replacement fix-its in c-index-test.
authorNico Weber <nicolasweber@gmx.de>
Sat, 10 May 2014 23:56:43 +0000 (23:56 +0000)
committerNico Weber <nicolasweber@gmx.de>
Sat, 10 May 2014 23:56:43 +0000 (23:56 +0000)
commita5e558a3f58e10a3dd34b50531245145aa0a3ba8
tree16dd10706261cb60d82e3101e579e7e2728888cd
parent8eccd0dc82f099aff18649a2654d39c3c311d9f4
Don't leak CXStrings for replacement fix-its in c-index-test.

The loop body used to contain a switch statement; it looks like r96685 replaced
that with an if/else if/else but accidentally left one of the three break
statements from the switch behind, skipping the clang_disposeString() call
for replacements (and the rest of the loop too, which apparently doesn't make
a differences for the test cases we have).

r96685: http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20100215/027754.html

This too might possibly the last leak in clang (PR19521).

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@208483 91177308-0d34-0410-b5e6-96231b3b80d8
tools/c-index-test/c-index-test.c