From c854fde373bf5f08c8a906e2bad00b6ed4d80ffc Mon Sep 17 00:00:00 2001 From: nobu Date: Fri, 26 May 2017 06:39:49 +0000 Subject: [PATCH] sprintf.c: remove redundant condition * sprintf.c (rb_str_format): when `t + 1 == end` (or `t < end`), `*t == '%'` is always true. [ruby-core:80153] [Bug #13315] git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@58898 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- sprintf.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/sprintf.c b/sprintf.c index 8488d58cbd..74229cc2d7 100644 --- a/sprintf.c +++ b/sprintf.c @@ -521,8 +521,7 @@ rb_str_format(int argc, const VALUE *argv, VALUE fmt) for (t = p; t < end && *t != '%'; t++) ; if (t + 1 == end) { - if (*t == '%') rb_raise(rb_eArgError, "incomplete format specifier"); - ++t; + rb_raise(rb_eArgError, "incomplete format specifier"); } PUSH(p, t - p); if (coderange != ENC_CODERANGE_BROKEN && scanned < blen) {