|
From: Nicholas N. <nj...@cs...> - 2006-02-05 01:04:54
|
On Sun, 5 Feb 2006, Julian Seward wrote: >> Valgrind still has some nested functions in m_debuginfo/symtypes.c >> (newvar and genstring). What's the policy with respect to nested >> functions? Would you all be OK if I recoded this a little to remove the >> nesting? > > I'd love to be rid of them. That'd be great, yes please. I agree. I tried once, but those two are tricky suckers and I didn't want to break anything so I didn't. They're in the the debug sym code that only Helgrind uses, and so it may have bitrotted somewhat. N |