[patch] Replace reread_symbols by load+free calls

[Jan Kratochvil]

On Sat, 15 Aug 2009 01:09:59 +0200, Pedro Alves wrote:
> On Friday 14 August 2009 23:34:31, Tom Tromey wrote:
> 
> > I realize you pulled this patch from consideration.  But, FWIW, I tend
> > to think it is a good idea, because it ensures that all the hooks and
> > observers will be run identically in both cases.
> 
> I have to say that I didn't think it was a good idea, because of the
> way the change ties the objfiles to solib.c, and the fact that
> it makes it so that an objfile pointer becomes more volatile.

This is correct, because when someone holds a pointer to volatile data
valgrind should report it as invalid.  And each such code needs to be reviewed
anyway so changing it along is not such a concern.


Regards,
Jan