New valgrind results for Oct 21, 2009
Dan Kegel
dank at kegel.com
Thu Oct 22 07:21:04 CDT 2009
I added suppressions for a few frequent spurious warnings and reran,
the updated results are in
http://kegel.com/wine/valgrind/logs/2009-10-21-19.42.log
and broken out by test in
http://kegel.com/wine/valgrind/logs/2009-10-21-19.42/
Please send me the suppression record for any warnings that are
spurious or benign, and mention the reason for each.
There's also an overall diff from the previous run,
http://kegel.com/wine/valgrind/logs/2009-10-21-19.42-diff.txt
but reading it is kind of a slog.
A few suggestions for people who would like to help fix these warnings:
1) Don't fix any error you don't fully understand. Many
of these warnings are easy, but some are subtle. When
in doubt, move on to an easier one.
2) Fixing leaks in test code is usually pretty safe, and
it's easier to get Alexandre to accept simple patches to tests,
so start with those if you don't have many patches under your belt.
3) the file
http://kegel.com/wine/valgrind/logs/2009-10-21-19.42-count-by-file.txt
lists how many warnings are in each test; pick one with
just one error, and fix it. Voila, one less failing test!
4) the file
http://kegel.com/wine/valgrind/logs/2009-10-21-19.42-count-by-error.txt
lists how many of each warning there are. Pick a warning
that happens a lot, and fix it. Voila, lots fewer warnings!
5) if you know a warning is spurious, send it to me along with
the mangled version below it (that's in suppression format)
and why it's spurious or benign or not our fault, and I'll add it to
http://code.google.com/p/winezeug/source/browse/trunk/valgrind/valgrind-suppressions
Have fun, be careful, and thanks to everyone who's pitching in!
- Dan
More information about the wine-devel
mailing list