[ruby-core:28123] Re: [Bug #2709] $VERBOSE, $DEBUG and Kernel#sprintf

Yukihiro Matsumoto wrote:
> We don't accept any proposal with the POLS reason by default, since
> the background vary person to person.  So don't be so lazy.  Explain
> why YOU have assumed Kernel#sprintf not to change behavior according
> to the variables $VERBOSE and $DEBUG.

Take a look at [ruby-core:28032].  My current feeling is neutral ... I ha=
ve
been helped by printf's argument checks many time so it is useful for me =
at
least, but it seems the reporter does have needs to ignore it.

What about it:  raise exceptions as it is when arguments are too _few_, b=
ut not
for too _many_, even when $DEBUG is true.

Signed-off-by: Urabe, Shyuohei <[e-mail]>
---
 sprintf.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/sprintf.c b/sprintf.c
index b5cad94..ebb26eb 100644
--- a/sprintf.c
+++ b/sprintf.c
@@ -1057,7 +1057,7 @@ rb_str_format(int argc, const VALUE *argv, VALUE fm=
t)
      */
     if (posarg >=3D 0 && nextarg < argc) {
 	const char *mesg =3D "too many arguments for format string";
-	if (RTEST(ruby_debug)) rb_raise(rb_eArgError, "%s", mesg);
+	if (RTEST(ruby_debug)) rb_warn("%s", mesg);
 	if (RTEST(ruby_verbose)) rb_warn("%s", mesg);
     }
     rb_str_resize(result, blen);