omtcyf0 added a comment.

Yup. FeelsBadMan.

One note about that header removal: how do I do it then? I thought attaching 
such changes to a random patch is not a problem. Otherwise there will be some 
one-line patches for such things, won't they?


================
Comment at: clang-rename/tool/ClangRename.cpp:39
@@ -38,3 +38,2 @@
 #include "llvm/Support/Host.h"
-#include <cstdlib>
 #include <string>
----------------
bkramer wrote:
> This looks unrelated.
Right, but I'm not sure one-line patches are welcome.

Is it not alright to put such into random patches? Otherwise creating 
one-liner...


http://reviews.llvm.org/D22087



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to