|
From: Nicholas N. <n.n...@gm...> - 2009-08-11 01:01:02
|
On Mon, Aug 10, 2009 at 11:42 PM, Julian Seward<js...@ac...> wrote: > On Monday 10 August 2009, sv...@va... wrote: >> Author: njn >> Date: 2009-08-10 08:50:00 +0100 (Mon, 10 Aug 2009) >> New Revision: 10767 >> >> Log: >> Count leak as errors. Fixes bug 152393. > > But then we get this: > > ==29908== ERROR SUMMARY: 71 errors from 0 contexts (suppressed: 111 from 2) > > which is confusing. I'm wondering if there's a different way to fix > 152393, perhaps by adding a --warning-exitcode= flag. Problem is the > core has no notion of major vs minor errors. Hmm. I think treating (definitely lost and possibly lost) leaks as errors is the right thing to do. The context counting was just broken. It's fixed in r10773. Nick |