Mercurial > octave-nkf
diff src/load-path.cc @ 15008:ff9a638a5555
preserve order of path elements when doing "path (new_path)"
* load-path.cc, load-path.h (load_path::do_clear): Delete argument.
Simply clear dir_info_list.
(load_path::do_set, load_path::do_append, load_path::do_add):
Eliminate is_init argument.
(load_path::dir_info::is_init): Delete data member and all uses.
(load_path::init_dirs): New data member.
(load_path::do_set): Store default path elements in list and set.
Warn about altering default load path. Preserve order of path
elements.
author | John W. Eaton <jwe@octave.org> |
---|---|
date | Tue, 24 Jul 2012 13:13:45 -0400 |
parents | f10cddda37c5 |
children |
line wrap: on
line diff
--- a/src/load-path.cc Tue Jul 24 09:34:51 2012 -0700 +++ b/src/load-path.cc Tue Jul 24 13:13:45 2012 -0400 @@ -523,36 +523,9 @@ } void -load_path::do_clear (std::set<std::string>& new_elts) +load_path::do_clear (void) { - bool warn_default_path_clobbered = false; - for (dir_info_list_iterator i = dir_info_list.begin (); - i != dir_info_list.end (); - /* conditionally advance iterator in loop body */) - { - //Don't remove it if it's gonna be added again, but remove it from - //list of items to add, to avoid duplicates later on - std::set<std::string>::iterator j = new_elts.find (i->dir_name); - if (j != new_elts.end ()) - { - new_elts.erase (j); - i++; - } - else - { - //Warn if removing a default directory and not immediately adding - //it back again - if (i->is_init) - warn_default_path_clobbered = true; - i = dir_info_list.erase (i); - } - } - - if (warn_default_path_clobbered) - warning_with_id ("Octave:remove-init-dir", - "default load path altered. Some built-in functions may " - "not be found. Try restoredefaultpath() to recover it."); - + dir_info_list.clear (); fcn_map.clear (); private_fcn_map.clear (); method_map.clear (); @@ -594,8 +567,27 @@ void load_path::do_set (const std::string& p, bool warn, bool is_init) { - std::list<std::string> elts_l = split_path (p); - std::set<std::string> elts(elts_l.begin (), elts_l.end ()); + // Use a list when we need to preserve order. + std::list<std::string> elts = split_path (p); + + // Use a set when we need to search and order is not important. + std::set<std::string> elts_set (elts.begin (), elts.end ()); + + if (is_init) + init_dirs = elts_set; + else + { + for (std::set<std::string>::const_iterator it = init_dirs.begin (); + it != init_dirs.end (); it++) + { + if (elts_set.find (*it) == elts_set.end ()) + { + warning_with_id ("Octave:remove-init-dir", + "default load path altered. Some built-in functions may not be found. Try restoredefaultpath() to recover it."); + break; + } + } + } // Temporarily disable add hook. @@ -604,12 +596,11 @@ add_hook = 0; - do_clear (elts); - - for (std::set<std::string>::const_iterator i = elts.begin (); - i != elts.end (); - i++) - do_append (*i, warn, is_init); + do_clear (); + + for (std::list<std::string>::const_iterator i = elts.begin (); + i != elts.end (); i++) + do_append (*i, warn); // Restore add hook and execute for all newly added directories. frame.run_top (); @@ -627,10 +618,10 @@ } void -load_path::do_append (const std::string& dir, bool warn, bool is_init) +load_path::do_append (const std::string& dir, bool warn) { if (! dir.empty ()) - do_add (dir, true, warn, is_init); + do_add (dir, true, warn); } void @@ -659,8 +650,7 @@ } void -load_path::do_add (const std::string& dir_arg, bool at_end, bool warn, - bool is_init) +load_path::do_add (const std::string& dir_arg, bool at_end, bool warn) { size_t len = dir_arg.length (); @@ -685,7 +675,6 @@ if (fs.is_dir ()) { dir_info di (dir); - di.is_init = is_init; if (! error_state) {