[Octave-bug-tracker] [bug #41367] append folder separator in second output of uigetfile

[Rik]

Update of bug #41367 (project octave):

                  Status:                    None => Patch Submitted        

    _______________________________________________________

Follow-up Comment #8:

This is a minor compatibility issue.  I would place it on the gui-release
branch rather than the stable branch.

>From uigetfile.m I see


  if (__octave_link_enabled__ ())
    [retfile, retpath, retindex] = __octave_link_file_dialog__ (outargs{:});
  else
    [retfile, retpath, retindex] = feval (funcname, outargs{:});
  endif


In the CLI version, uigetfile eventually calls out to __fltk_uigetfile__.cc
which is in libinterp/dldfcn.

In the C++ code for __fltk_uigetfile__ I see this:


//fltk uses forward slash even for windows
std::string sep = "/";
...
retval(1) = std::string (fc.directory ()) + sep;


So that would explain things for the CLI interface.

It seems like it would be simple enough to add either a regexprep call after
the feval in the m-file (regexprep (retpath, '/', filesep), or to add the
search and replace code in C++ to __fltk_uigetfile__ using
file_ops::dir_sep_char (), string.find, and string.replace calls.

This seems like a useful enough function that the best option would be to add
the equivalent of QDir::toNativeSeparators function to Octave's own liboctave
library.  The class to change is file_ops which lives in
liboctave/system/file_ops.[h|cc].  The function would probably need to go
through the path character by character so that it would not replace instances
of dir_sep_char that had been escaped.







    _______________________________________________________

Reply to this item at:

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

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