[Top][All Lists]

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

[Octave-bug-tracker] [bug #37573] edit.m cannot open multiple file names

From: Richard
Subject: [Octave-bug-tracker] [bug #37573] edit.m cannot open multiple file names
Date: Mon, 15 Oct 2012 14:20:55 +0000
User-agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:16.0) Gecko/20100101 Firefox/16.0

Follow-up Comment #2, bug #37573 (project octave):

Most of the changes shown in the changeset are due to me putting a lot of the
original code inside an if statement, and therefore adding spaces at the start
of the lines.

In the original code you could supply either one argument (a file name) or two
arguments (an editor state to be changed and the new value of that state). In
the second case then the first argument (named file) was already no longer
actually a file name anyway.

With my changes you have the following options:

one arg:
either a string, or cell array of strings, each containing a file name to be
opened by the editor.

two arg:
either a state name and new state (as in original)
two strings each containing file names to be opened by the editor

more args:
multiple strings, each a file to be opened by the editor

depending on the inputs, I create a cell array of strings containing the list
of files to be opened. If there are more than one in this list I call edit.m
on each one. 

If edit.m is called with a single string, the length of the list is one and
the original behaviour is used instead.

I could add more comments to explain this if you like. I am very unfamiliar
with hg, is the exact sequence of things I need to do the following?

hg up -r 30c8b5d22e
make my edits to edit.m
hg commit edit.m --amend -m"new commit msg"

I had read the guidelines, but misunderstood them, edit.m does contain
multiple functions (it has subfunctions), so I though I would need to be
specific from what I read.


Reply to this item at:


  Message sent via/by Savannah

reply via email to

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