[PATCH] [clang-tidy] Don't leak the TodoCommentHandler object

Hans Wennborg hans at chromium.org
Thu Sep 18 10:57:07 PDT 2014


Hi bkramer,

Preprocessor:addCommentHandler() does not take ownership, so we'd end up leaking the TodoCommentHandler.

This patch makes it owned by the Check object.

http://reviews.llvm.org/D5402

Files:
  clang-tidy/google/TodoCommentCheck.cpp
  clang-tidy/google/TodoCommentCheck.h
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D5402.13839.patch
Type: text/x-patch
Size: 1794 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140918/89c5f08e/attachment.bin>


More information about the cfe-commits mailing list