Mercurial > octave

diff libinterp/parse-tree/bp-table.cc @ 26107:05dfcb24ef12

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Supply missing arguments to variable length *printf functions (bug #55046). * data.cc (Frem, Fmod): Change arg in format string from '%d' to '%s' because class name is a string. * graphics.in.h (double_radio_property::current_value): Supply name of property to match '%s' argument in error format string. * variables.cc (set_internal_variable): Supply name of variable to match '%s' argument in error format string. * bp-table.cc (bp_table::parse_dbfunction_params): Supply name of calling function to '%s' argument in error format string.
author Rik <rik@octave.org>
date Wed, 21 Nov 2018 10:16:04 -0800
parents 143007dad864
children 2db7e4e5d67f
line wrap: on
line diff
--- a/libinterp/parse-tree/bp-table.cc	Wed Nov 21 19:04:06 2018 +0100
+++ b/libinterp/parse-tree/bp-table.cc	Wed Nov 21 10:16:04 2018 -0800
@@ -332,7 +332,7 @@
               error ("%s: Only one 'at' clause is allowed -- %s",
                      who, args(pos).string_value ().c_str ());
             else if (seen_if)
-              error ("%s: line number must come before 'if' clause\n");
+              error ("%s: line number must come before 'if' clause\n", who);
             seen_at = true;
 
             if (! seen_in)
@@ -346,7 +346,7 @@
                 seen_in = true;
               }
             else if (seen_if)
-              error ("%s: line number must come before 'if' clause\n");
+              error ("%s: line number must come before 'if' clause\n", who);
 
             // Read a list of line numbers (or arrays thereof)
             for ( ; pos < nargin; pos++)
@@ -369,7 +369,7 @@
                   }
                 else
                   error ("%s: Invalid argument type %s",
-                         args(pos).type_name ().c_str ());
+                         who, args(pos).type_name ().c_str ());
               }
             break;