octave-patch-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Octave-patch-tracker] [patch #7991] Remove _terminal pointer from main_


From: John W. Eaton
Subject: [Octave-patch-tracker] [patch #7991] Remove _terminal pointer from main_window.
Date: Wed, 03 Apr 2013 04:20:32 +0000
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:10.0.11) Gecko/20100101 Firefox/10.0.11 Iceweasel/10.0.11

Update of patch #7991 (project octave):

                  Status:                    None => Done                   
             Open/Closed:                    Open => Closed                 

    _______________________________________________________

Follow-up Comment #1:

Thanks for looking at this.  I checked in the changeset with a few minor
changes.

When writing changelog entries in the future, unless there is something really
unusual going on, please write what changed and not so much why.  If some
explanation is needed, I think it is better to put that in comments in the
code rather than in a changelog entry.

Also, although it is tempting to clean out cruft at the same time as making
other changes, it would help if in the future those changes are made
separately.  I much prefer to have mutliple changesets that each have one
purpose rather than one changeset that does multiple kinds of changes.

Thanks,

jwe


    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/patch/?7991>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/




reply via email to

[Prev in Thread] Current Thread [Next in Thread]