[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/
- Prev by Date:
[Octave-patch-tracker] [patch #7990] Add builtin Qt equivalents of UI dialogs, i.e., , qterrordlg, qtinputdlg, etc.
- Next by Date:
[Octave-patch-tracker] [patch #7990] Add builtin Qt equivalents of UI dialogs, i.e., , qterrordlg, qtinputdlg, etc.
- Previous by thread:
[Octave-patch-tracker] [patch #7991] Remove _terminal pointer from main_window.
- Next by thread:
[Octave-patch-tracker] [patch #7994] ind2rgb - Enabling handling of ND images
- Index(es):