9 Apr
2011
9 Apr
'11
6:13 a.m.
On Apr 8, 2011, at 6:48 PM, Krzysztof KosiĆski wrote:
This looks like uninitialized memory in NRStyle. I added explicit NULL initializers in rev. 9588, see whether it fixes the crash.
Well, since we're C++ we should be using 0 instead of NULL. :-)
But more importantly, this is exactly the sort of issue that Valgrind is very good at spotting and pointing out.
When problems like this (subtle corruptions or crashing) are hit, running under valgrind should be a first step. It will generally catch a problem when it first occurs, not later on after it has cascaded enough to actually cause a crash, etc.
If anyone needs more detail on the wiki for this, let me know. Also drop on in to IRC to hash it out (that really helps).