[Octave-bug-tracker] [bug #31388] Improvement for edit() which makes it work together with console vim

[anonymous]

URL:
  <http://savannah.gnu.org/bugs/?31388>

                 Summary: Improvement for edit() which makes it work together
with console vim
                 Project: GNU Octave
            Submitted by: None
            Submitted on: Mi 20 Okt 2010 22:25:57 UTC
                Category: Interpreter
                Severity: 3 - Normal
                Priority: 5 - Normal
              Item Group: Improvement/Optimization
                  Status: None
             Assigned to: None
         Originator Name: 
        Originator Email: 
             Open/Closed: Open
         Discussion Lock: Any
                 Release: 3.2.4
        Operating System: GNU/Linux

    _______________________________________________________

Details:



I struggled accross the fact that console vim can't be used as editor.
Apparently, this is due to the fact, that edit() uses system() and system()
redirect stdout "sometimes".

The code in edit() which fails is
system (sprintf (FUNCTION.EDITOR, strcat ("\"", path, "\"")), [],
FUNCTION.MODE);

Having

edit editor "vim %s";
edit mode sync;

The message I get is: "Vim: Warning: Output is not to a terminal"

I suggest the attached patch which does a simple system() call with simply
one argument, if the mode is 'sync'.

The patch applies to edit() as of octave 3.2.4




    _______________________________________________________

File Attachments:


-------------------------------------------------------
Date: Mi 20 Okt 2010 22:25:57 UTC  Name: edit.m.diff  Size: 2kB   By: None

<http://savannah.gnu.org/bugs/download.php?file_id=21734>

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?31388>

_______________________________________________
  Nachricht geschickt von/durch Savannah
  http://savannah.gnu.org/