[OctDev] proposed addpath patch

[Hall, B. <ben...@pw...>]

In octave-2.9.3, the addpath command generates warning messages when used
with double-quoted strings as input, e.g., addpath("/home/me/mydir")

The following proposed patch disables the warnings during the execution of
the addpath function.  I don't think this should cause any problems
(unexpected results) for end users based on the earlier discussions
(http://www.octave.org/octave-lists/archive/octave-maintainers.2005/msg00280
.html)

Does this seem reasonable?


*** addpath_old.m	2005-09-08 09:13:14.000000000 -0400
--- addpath_new.m	2005-09-08 09:27:18.000000000 -0400
***************
*** 49,54 ****
--- 49,59 ----
      path = LOADPATH;
    end
  
+   try   wrn_state = warn_string_concat;
+   catch wrn_state = 0;
+   end
+   warn_string_concat = 0;
+ 
    dir = '';
    if length(varargin) > 0
      append = 0;
***************
*** 106,111 ****
--- 111,118 ----
      LOADPATH = path; 
    end
  
+   warn_string_concat = wrn_state;
+ 
  %!assert(addpath('','hello'),'hello');
  %!assert(addpath('','hello','world'),'hello:world')
  %!assert(addpath(':','hello'),'hello:');