[GR-16397] Implement rb_enc_sprintf.

PullRequest: truffleruby/887

Author: eregon

CHANGELOG.md

@@ -11,6 +11,7 @@ Compatibility:
 * `Integer#{ceil,floor,truncate}` now accept a precision and `Integer#round` accepts a rounding mode.
 * Added missing `Enumerable#filter` and `Enumerator::Lazy#filter` aliases to the respective `select` method (#1610).
 * Implement more `Ripper` methods as no-ops (#1694).
+* Implemented `rb_enc_sprintf` (#1702).
 
 # 20.0.0 beta 1
 

src/main/c/cext/ruby.c

@@ -1309,7 +1309,14 @@ void rb_enc_raise(rb_encoding *enc, VALUE exc, const char *fmt, ...) {
 }
 
 VALUE rb_enc_sprintf(rb_encoding *enc, const char *format, ...) {
-  rb_tr_error("rb_enc_sprintf not implemented");
+  VALUE result;
+  va_list ap;
+
+  va_start(ap, format);
+  result = rb_enc_vsprintf(enc, format, ap);
+  va_end(ap);
+
+  return result;
 }
 
 int rb_enc_to_index(rb_encoding *enc) {

test/mri/excludes/Test_SPrintf.rb

@@ -2,28 +2,16 @@
 exclude :"test_format_integer(% #+-00.d)", "needs investigation"
 exclude :"test_format_integer(% #+-020d)", "needs investigation"
 exclude :"test_format_integer(% #-00.d)", "needs investigation"
-exclude :"test_format_integer(% #-00d)", "needs investigation"
 exclude :"test_format_integer(% #-020.d)", "needs investigation"
 exclude :"test_format_integer(% #-020d)", "needs investigation"
 exclude :"test_format_integer(% #0.d)", "needs investigation"
-exclude :"test_format_integer(% #20d)", "needs investigation"
 exclude :"test_format_integer(% +-0.0d)", "needs investigation"
-exclude :"test_format_integer(% +-0d)", "needs investigation"
 exclude :"test_format_integer(% +-20.0d)", "needs investigation"
-exclude :"test_format_integer(% +-20d)", "needs investigation"
 exclude :"test_format_integer(%#+0.d)", "needs investigation"
 exclude :"test_format_integer(%#+05.d)", "needs investigation"
-exclude :"test_format_integer(%#+05d)", "needs investigation"
-exclude :"test_format_integer(%#+0d)", "needs investigation"
 exclude :"test_format_integer(%+-05.0d)", "needs investigation"
-exclude :"test_format_integer(%+-0d)", "needs investigation"
 exclude :"test_format_integer(%.0d)", "needs investigation"
 exclude :"test_format_integer(%5.0d)", "needs investigation"
-exclude :"test_format_integer(%5d)", "needs investigation"
-exclude :"test_format_integer(%d)", "needs investigation"
-exclude :test_inspect, "needs investigation"
 exclude :test_quote, "needs investigation"
-exclude :test_string_prec, "needs investigation"
 exclude :test_taint, "needs investigation"
-exclude :test_to_str, "needs investigation"
 exclude :test_snprintf_count, "needs investigation"

tool/parse_mri_errors.rb

@@ -1,11 +1,11 @@
-## Expirimental MRI test tagging script
-## Needs to be run from the jruby/test/mri directory
-## Error console output read from stdin or the first argument
+#!/usr/bin/env ruby
+## Experimental MRI test tagging script from a test run log
+## The input is read from stdin or the first argument
 
 # Usage:
-#     ruby tool/parse_mri_errors.rb output.txt
+#     tool/parse_mri_errors.rb output.txt
 # or
-#     jt test test/mri/tests/rdoc/test_rdoc_token_stream.rb | ruby tool/parse_mri_errors.rb
+#     jt test mri test/mri/tests/rdoc/test_rdoc_token_stream.rb | tool/parse_mri_errors.rb
 
 REASON = ENV.fetch('REASON', 'needs investigation')
 
@@ -47,7 +47,7 @@
 repo_root = File.expand_path("../..", __FILE__)
 excludes = "#{repo_root}/test/mri/excludes"
 
-t = /((?:\w+::)?\w+)#(\S+)(?:\s*\[.+\])?:$/
+t = /((?:\w+::)?\w+)#(.+?)(?:\s*\[(?:[^\]])+\])?:$/
 contents.scan(t) do |class_name, test_method, result|
   file = excludes + "/" + class_name.split("::").join('/') + ".rb"
   unless result == "."