# HG changeset patch
# User John W. Eaton <jwe@octave.org>
# Date 1269438794 14400
# Node ID ba932ad87a04d93e4571d78515ae04d42ea4eca3
# Parent  62f8ae5ab2cc23ce788f99fd4b3217b7f32a84c9
load_path::add_to_fcn_map: style fixes

diff -r 62f8ae5ab2cc -r ba932ad87a04 src/ChangeLog
--- a/src/ChangeLog	Wed Mar 24 13:16:25 2010 +0100
+++ b/src/ChangeLog	Wed Mar 24 09:53:14 2010 -0400
@@ -1,3 +1,7 @@
+2010-03-24  John W. Eaton  <jwe@octave.org>
+
+	* load-path.cc (load_path::add_to_fcn_map): Style fixes.
+
 2010-03-24  Jaroslav Hajek  <highegg@gmail.com>
 
 	* variables.cc (maybe_missing_function_hook): Don't call hook if
diff -r 62f8ae5ab2cc -r ba932ad87a04 src/load-path.cc
--- a/src/load-path.cc	Wed Mar 24 13:16:25 2010 +0100
+++ b/src/load-path.cc	Wed Mar 24 09:53:14 2010 -0400
@@ -1688,13 +1688,20 @@
           else
             {
               // Warn if a built-in or library function is being shadowed.
+
               if (! file_info_list.empty ())
                 {
                   file_info& old = file_info_list.front ();
-                  if (sys_path.find (old.dir_name) != std::string::npos)
-                  if (in_path_list (sys_path, old.dir_name))
+
+                  // FIXME -- do we need to be more careful about the
+                  // way we look for old.dir_name in sys_path to avoid
+                  // partial matches?
+
+                  if (sys_path.find (old.dir_name) != std::string::npos
+                      && in_path_list (sys_path, old.dir_name))
                     {
                       std::string fcn_path = file_ops::concat (dir_name, fname);
+
                       warning_with_id ("Octave:shadowed-function",
                                        "function %s shadows a core library function", 
                                        fcn_path.c_str ());