[Patch, fortran] PR30554 - [4.2 and 4.1 only] ICE in mio_pointer_ref at module.c:1945

Steve Kargl sgk@troutmask.apl.washington.edu
Sun Feb 11 20:07:00 GMT 2007


On Sun, Feb 11, 2007 at 08:58:16PM +0100, Paul Thomas wrote:
> Steve,
> 
> Thanks for the review.  I'll put right the typos.
> 
> What do you think of the question of the one-liner that produces hidden 
> symtrees, versus the more complicated patch that tries to recycle 
> symtrees, if possible?
> 

I prefer the more complicated patch.  From a future
maintenance standpoint (when pault isn't around), the
hidden symtree approach may mislead one into thinking
the hidden symtree has some magical importance.  The
more complicated approach and its comments in the code
make it clear what is going on.

-- 
Steve



More information about the Fortran mailing list