This is the mail archive of the fortran@gcc.gnu.org mailing list for the GNU Fortran project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[gfortran, patch] Obvious corrections to my PR 25716 patch


The patch for PR 15716 I committed yesterday contained a few
small errors:  

1) For some reason, the version I committed wasn't exactly the
same as the one I had posted to the mailinglists: A comment was
different, and in the committed version the comment was confusing
to the point where it was misleading.

2) A formatting error.

3) I had missed one component of old_symbol that might need to be
deallocated. (The components that need deallocation can quite
easily be inferred by looking at symbol.c (gfc_undo_symbols).
(Assuming that gfc_undo_symbols doesn the right thing, of
course.)).  

I intend to commit this correction patch as abvious in 24 h,
unless anyone shouts "NO!".


        Erik


2005-01-27  Erik Edelmann  <eedelman@gcc.gnu.org>

        * symbol.c (free_old_symbol): Fix confusing comment, fix
          formatting, and add code to free old_symbol->formal. 

Attachment: obvious.diff
Description: Text document


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]