changeset 8703:c953a6977be6

dispatch.cc: comment out troublesome tests
author John W. Eaton <jwe@octave.org>
date Mon, 09 Feb 2009 12:33:47 -0500
parents 3c4628550318
children 236ff50db90f
files src/ChangeLog src/DLD-FUNCTIONS/dispatch.cc
diffstat 2 files changed, 44 insertions(+), 34 deletions(-) [+]
line wrap: on
line diff
--- a/src/ChangeLog	Mon Feb 09 12:29:30 2009 -0500
+++ b/src/ChangeLog	Mon Feb 09 12:33:47 2009 -0500
@@ -1,5 +1,7 @@
 2009-02-09  John W. Eaton  <jwe@octave.org>
 
+	* DLD-FUNCTIONS/dispatch.cc: Comment out troublesome tests.
+
 	* DLD-FUNCTIONS/eigs.cc: Increase tolerance to 1e-11 on all tests.
 
 	* lex.l (lexical_feedback::looking_at_decl_list): New data member.
--- a/src/DLD-FUNCTIONS/dispatch.cc	Mon Feb 09 12:29:30 2009 -0500
+++ b/src/DLD-FUNCTIONS/dispatch.cc	Mon Feb 09 12:33:47 2009 -0500
@@ -227,40 +227,48 @@
 %! evalin('base','assert(hanning(1),1)');
 %! dispatch('hanning','string');
 
-FIXME I would rather not create dispatch_x/dispatch_y
-in the current directory!  I don't want them installed accidentally.
-
-%!function echo_to_file (str, name)
-%!  fid = fopen (name, 'w');
-%!  if (fid != -1)
-%!    fprintf (fid, str);
-%!    fprintf (fid, '\n');
-%!    fclose (fid);
-%!  endif
-
-%!test # replace base m-file
-%! echo_to_file ('function a=dispatch_x(a)', "dispatch_x.m");
-%! dispatch('dispatch_x','length','string')
-%! assert(dispatch_x(3),3)
-%! assert(dispatch_x("a"),1)
-%! sleep (2);
-%! echo_to_file ('function a=dispatch_x(a),++a;', "dispatch_x.m");
-%! rehash();
-%! assert(dispatch_x(3),4)
-%! assert(dispatch_x("a"),1)
-%!test 
-%! unlink("dispatch_x.m");
-
-%!test # replace dispatch m-file
-%! echo_to_file ('function a=dispatch_y(a)', "dispatch_y.m");
-%! dispatch('hello','dispatch_y','complex scalar')
-%! assert(hello(3i),3i)
-%! sleep (2);
-%! echo_to_file ('function a=dispatch_y(a),++a;', "dispatch_y.m");
-%! rehash();
-%! assert(hello(3i),1+3i)
-%!test 
-%! unlink("dispatch_y.m");
+## The following tests have been disabled because creating functions
+## on the fly causes trouble (filesystem timestamp resolution?) and so
+## leads people to complain about the failed tests when the dispatch
+## mechanism is working fine, but it is really the creation of the
+## functions that is failing.  And anyway, this method of function
+## dispatch should be considered obsolete and probably removed from
+## Octave now that we have classes.
+##
+## FIXME I would rather not create dispatch_x/dispatch_y
+## in the current directory!  I don't want them installed accidentally.
+## 
+## %!function echo_to_file (str, name)
+## %!  fid = fopen (name, 'w');
+## %!  if (fid != -1)
+## %!    fprintf (fid, str);
+## %!    fprintf (fid, '\n');
+## %!    fclose (fid);
+## %!  endif
+## 
+## %!test # replace base m-file
+## %! echo_to_file ('function a=dispatch_x(a)', "dispatch_x.m");
+## %! dispatch('dispatch_x','length','string')
+## %! assert(dispatch_x(3),3)
+## %! assert(dispatch_x("a"),1)
+## %! sleep (2);
+## %! echo_to_file ('function a=dispatch_x(a),++a;', "dispatch_x.m");
+## %! rehash();
+## %! assert(dispatch_x(3),4)
+## %! assert(dispatch_x("a"),1)
+## %!test 
+## %! unlink("dispatch_x.m");
+## 
+## %!test # replace dispatch m-file
+## %! echo_to_file ('function a=dispatch_y(a)', "dispatch_y.m");
+## %! dispatch('hello','dispatch_y','complex scalar')
+## %! assert(hello(3i),3i)
+## %! sleep (2);
+## %! echo_to_file ('function a=dispatch_y(a),++a;', "dispatch_y.m");
+## %! rehash();
+## %! assert(hello(3i),1+3i)
+## %!test 
+## %! unlink("dispatch_y.m");
 
 FIXME add tests for preservation of mark_as_command status.