[Octave-patch-tracker] [patch #7857] Replace dynamic casts of GUI/IDE editor with more slots/signals configuration

[Dan Sebald]

Follow-up Comment #27, patch #7857 (project octave):

I'm fine with making the external file change a separate patch.  In fact, it
is probably best to have a good working version the first way, then a small
change for the second way.  I could make the change, but unless I can verify
proper behavior either by some test on my system or by Qt documentation it
probably makes no sense to change it.

I've created a new patch with what I think are the things I touched in the
commit message.  (Sure was a mess...must have reverted some changes you made
by forcing a forward commit.)  This seems to restore desired behavior.  Let me
know if it doesn't work correctly.

On Windows systems case is ignored?  I sort of remember that for DOS, but I'd
have thought that was changed given that spaces were allowed in file names. 
Let's make that a separate commit.  I don't see that I altered anything with
"WINDOWS", etc.  Mainly this patch is to get things more in line with
signals/slots and fix the few obvious problems with saving and opening files
in the editor.

Sorry this took a while.  Octave has been somewhat unstable for a month or so
and I thought I'd wait for things to even out a bit.

(file #27147)
    _______________________________________________________

Additional Item Attachment:

File name: octave-gui_no_casts-2012dec23.patch Size:76 KB


    _______________________________________________________

Reply to this item at:

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

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