|
From: Dan K. <da...@ke...> - 2009-03-13 18:50:55
|
This bit in memcheck/mc_main.c is annoying me:
if (VG_(clo_xml)) {
/* Extract as much info as possible from the leak checker. */
/* MC_(clo_show_reachable) = True; */
MC_(clo_leak_check) = LC_Full;
}
Our project's valgrind builder is way red right now,
and I wanted to add --leak-check=no so we could
defer fixing leaks until more serious bugs are fixed,
but valgrind merrily ignores my --leak-check=no
and outputs full leak checking. Bleah. I guess
I can go filter out the leaks in our valgrind xml parser,
but it would have been easier if I didn't have to do that.
|
|
From: Ashley P. <as...@pi...> - 2009-03-13 20:41:55
|
On 13 Mar 2009, at 18:50, Dan Kegel wrote: > Our project's valgrind builder is way red right now, > and I wanted to add --leak-check=no so we could > defer fixing leaks until more serious bugs are fixed, > but valgrind merrily ignores my --leak-check=no > and outputs full leak checking. Bleah. > I guess > I can go filter out the leaks in our valgrind xml parser, > but it would have been easier if I didn't have to do that. This is what I did, I'd be interested in seeing your parser if it's open source... Given that xml is designed to be machine-readable I've no problem with it being verbose, i just looked at the args section of the xml and change the code accordingly. Ashley Pittman. |
|
From: Dan K. <da...@ke...> - 2009-03-13 20:43:51
|
On Fri, Mar 13, 2009 at 1:42 PM, Ashley Pittman <as...@pi...> wrote: >> I guess >> I can go filter out the leaks in our valgrind xml parser, >> but it would have been easier if I didn't have to do that. > > This is what I did, I'd be interested in seeing your parser if it's open > source... It is. The cl to filter out leaks is at http://codereview.chromium.org/42183/show > Given that xml is designed to be machine-readable I've no problem with it > being verbose, i just looked at the args section of the xml and change the > code accordingly. Yeah, but that side effect of --xml=yes is just too counterintuitive to stand... |