[PATCH] D54834: [analyzer][MallocChecker] Improve warning messages on double-delete errors

Artem Dergachev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 17 09:14:02 PST 2018


NoQ added a comment.

In D54834#1332679 <https://reviews.llvm.org/D54834#1332679>, @Szelethus wrote:

> Interesting, I've been watching the bots closely, but got no mail after a while. I'm not sure what the cause is, so I'll revert one-by-one.


One of the common reasons for that is that the buildbot was already failing when you committed your stuff. Which is why for many buildbots there are special people who look at them and tell people to fix their stuff manually.

This time it wasn't the case though: build 2661 was green. So dunno. I definitely did receive a mail from this buildbot when it failed on my patch.

But generally it's fine if you don't notice that you're breaking a buildbot or two. It's not something to worry about. Sooner or later, they will come after you :)


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D54834/new/

https://reviews.llvm.org/D54834





More information about the cfe-commits mailing list