|
From: Julian S. <js...@ac...> - 2005-02-25 23:45:34
|
Nice summary. Definitely a good thing. > > - increase the default --num-callers size to 12 > > Yes! I was about to propose this. 4 has been way too small for a long > time. Also, the default --leak-resolution=low (2 frames matches) is too > low as well; I think it should default to "med". Yes, perhaps time to increase --num-callers. Problem with large values is that error messages take up tons of screen space. Oh well. Default 8 ? 12 ? > And should we consider defaulting to memcheck again? Personally I would like that. I find it annoying to constantly have to type --tool=memcheck (but not _always_, so .valgrindrc isn't the answer). Indeed, I am not the only one: the V that comes packaged with SuSE seems to already have been defaulted to memcheck, presumably by the SuSE folks themselves. (iow, "valgrind ls" works fine on SuSE). > I find --show-reachable is too noisy for common use. I agree. I hesitate to run the leak checker by default at exit. It can cause a huge amount of disk activity because it inspects bits of address space that had been swapped out during long runs of leaky programs, and which then gets forced back in as the checker traverses it all. But perhaps we should. In an underhand way it creates the expectation amongst programmers that their programs should have no detectable leaks, and that leaks will be tested for routinely -- every time V is run. And it helps our software engineering too. Currently the checker is basically an optional component and doesn't get tested enough. If it was engaged by default, we would know much sooner if there was anything broken about it. J |