Skip to content

Commit 7f05140

Browse files
committed
Fix conversion crash on 1.9
1 parent 2bcacc1 commit 7f05140

File tree

6 files changed

+9
-6
lines changed

6 files changed

+9
-6
lines changed

.travis.yml

+4
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,10 @@ language: ruby
44

55
# Specify which ruby versions you wish to run your tests on, each version will be used
66
rvm:
7+
- 1.8.7
8+
- 1.9.1
9+
- 1.9.2
10+
- 1.9.3
711
- 2.0.0
812
- 2.1
913
- 2.2

Rakefile

+2-2
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@ if defined?(Gem) and defined?(Gem::PackageTask)
9595
s.email = "flori@ping.de"
9696
s.homepage = "http://flori.github.com/#{PKG_NAME}"
9797
s.license = 'Ruby'
98-
s.required_ruby_version = '~>2.0'
98+
#s.required_ruby_version = '~>2.0'
9999
end
100100

101101
desc 'Creates a json_pure.gemspec file'
@@ -133,7 +133,7 @@ if defined?(Gem) and defined?(Gem::PackageTask)
133133
s.email = "flori@ping.de"
134134
s.homepage = "http://flori.github.com/#{PKG_NAME}"
135135
s.license = 'Ruby'
136-
s.required_ruby_version = '~>2.0'
136+
#s.required_ruby_version = '~>2.0'
137137
end
138138

139139
desc 'Creates a json.gemspec file'

ext/json/ext/parser/parser.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -1608,7 +1608,7 @@ static VALUE convert_encoding(VALUE source)
16081608
FORCE_UTF8(source);
16091609
}
16101610
} else {
1611-
source = rb_str_conv_enc(source, NULL, rb_utf8_encoding());
1611+
source = rb_str_conv_enc(source, rb_enc_get(source), rb_utf8_encoding());
16121612
}
16131613
}
16141614
#else

ext/json/ext/parser/parser.rl

+1-1
Original file line numberDiff line numberDiff line change
@@ -592,7 +592,7 @@ static VALUE convert_encoding(VALUE source)
592592
FORCE_UTF8(source);
593593
}
594594
} else {
595-
source = rb_str_conv_enc(source, NULL, rb_utf8_encoding());
595+
source = rb_str_conv_enc(source, rb_enc_get(source), rb_utf8_encoding());
596596
}
597597
}
598598
#else

json.gemspec

-66 Bytes
Binary file not shown.

json_pure.gemspec

+1-2
Original file line numberDiff line numberDiff line change
@@ -8,15 +8,14 @@ Gem::Specification.new do |s|
88
s.required_rubygems_version = Gem::Requirement.new(">= 0".freeze) if s.respond_to? :required_rubygems_version=
99
s.require_paths = ["lib".freeze]
1010
s.authors = ["Florian Frank".freeze]
11-
s.date = "2017-01-11"
11+
s.date = "2017-01-12"
1212
s.description = "This is a JSON implementation in pure Ruby.".freeze
1313
s.email = "flori@ping.de".freeze
1414
s.extra_rdoc_files = ["README.md".freeze]
1515
s.files = ["./tests/test_json.rb".freeze, "./tests/test_json_addition.rb".freeze, "./tests/test_json_encoding.rb".freeze, "./tests/test_json_fixtures.rb".freeze, "./tests/test_json_generate.rb".freeze, "./tests/test_json_generic_object.rb".freeze, "./tests/test_json_string_matching.rb".freeze, "./tests/test_json_unicode.rb".freeze, ".gitignore".freeze, ".travis.yml".freeze, "CHANGES".freeze, "Gemfile".freeze, "README-json-jruby.markdown".freeze, "README.md".freeze, "Rakefile".freeze, "TODO".freeze, "VERSION".freeze, "data/example.json".freeze, "data/index.html".freeze, "data/prototype.js".freeze, "diagrams/.keep".freeze, "ext/json/ext/fbuffer/fbuffer.h".freeze, "ext/json/ext/generator/depend".freeze, "ext/json/ext/generator/extconf.rb".freeze, "ext/json/ext/generator/generator.c".freeze, "ext/json/ext/generator/generator.h".freeze, "ext/json/ext/parser/depend".freeze, "ext/json/ext/parser/extconf.rb".freeze, "ext/json/ext/parser/parser.c".freeze, "ext/json/ext/parser/parser.h".freeze, "ext/json/ext/parser/parser.rl".freeze, "ext/json/extconf.rb".freeze, "install.rb".freeze, "java/src/json/ext/ByteListTranscoder.java".freeze, "java/src/json/ext/Generator.java".freeze, "java/src/json/ext/GeneratorMethods.java".freeze, "java/src/json/ext/GeneratorService.java".freeze, "java/src/json/ext/GeneratorState.java".freeze, "java/src/json/ext/OptionsReader.java".freeze, "java/src/json/ext/Parser.java".freeze, "java/src/json/ext/Parser.rl".freeze, "java/src/json/ext/ParserService.java".freeze, "java/src/json/ext/RuntimeInfo.java".freeze, "java/src/json/ext/StringDecoder.java".freeze, "java/src/json/ext/StringEncoder.java".freeze, "java/src/json/ext/Utils.java".freeze, "json-java.gemspec".freeze, "json.gemspec".freeze, "json_pure.gemspec".freeze, "lib/json.rb".freeze, "lib/json/add/bigdecimal.rb".freeze, "lib/json/add/complex.rb".freeze, "lib/json/add/core.rb".freeze, "lib/json/add/date.rb".freeze, "lib/json/add/date_time.rb".freeze, "lib/json/add/exception.rb".freeze, "lib/json/add/ostruct.rb".freeze, "lib/json/add/range.rb".freeze, "lib/json/add/rational.rb".freeze, "lib/json/add/regexp.rb".freeze, "lib/json/add/struct.rb".freeze, "lib/json/add/symbol.rb".freeze, "lib/json/add/time.rb".freeze, "lib/json/common.rb".freeze, "lib/json/ext.rb".freeze, "lib/json/ext/.keep".freeze, "lib/json/generic_object.rb".freeze, "lib/json/pure.rb".freeze, "lib/json/pure/generator.rb".freeze, "lib/json/pure/parser.rb".freeze, "lib/json/version.rb".freeze, "tests/fixtures/fail1.json".freeze, "tests/fixtures/fail10.json".freeze, "tests/fixtures/fail11.json".freeze, "tests/fixtures/fail12.json".freeze, "tests/fixtures/fail13.json".freeze, "tests/fixtures/fail14.json".freeze, "tests/fixtures/fail18.json".freeze, "tests/fixtures/fail19.json".freeze, "tests/fixtures/fail2.json".freeze, "tests/fixtures/fail20.json".freeze, "tests/fixtures/fail21.json".freeze, "tests/fixtures/fail22.json".freeze, "tests/fixtures/fail23.json".freeze, "tests/fixtures/fail24.json".freeze, "tests/fixtures/fail25.json".freeze, "tests/fixtures/fail27.json".freeze, "tests/fixtures/fail28.json".freeze, "tests/fixtures/fail3.json".freeze, "tests/fixtures/fail4.json".freeze, "tests/fixtures/fail5.json".freeze, "tests/fixtures/fail6.json".freeze, "tests/fixtures/fail7.json".freeze, "tests/fixtures/fail8.json".freeze, "tests/fixtures/fail9.json".freeze, "tests/fixtures/pass1.json".freeze, "tests/fixtures/pass15.json".freeze, "tests/fixtures/pass16.json".freeze, "tests/fixtures/pass17.json".freeze, "tests/fixtures/pass2.json".freeze, "tests/fixtures/pass26.json".freeze, "tests/fixtures/pass3.json".freeze, "tests/setup_variant.rb".freeze, "tests/test_json.rb".freeze, "tests/test_json_addition.rb".freeze, "tests/test_json_encoding.rb".freeze, "tests/test_json_fixtures.rb".freeze, "tests/test_json_generate.rb".freeze, "tests/test_json_generic_object.rb".freeze, "tests/test_json_string_matching.rb".freeze, "tests/test_json_unicode.rb".freeze, "tools/diff.sh".freeze, "tools/fuzz.rb".freeze, "tools/server.rb".freeze]
1616
s.homepage = "http://flori.github.com/json".freeze
1717
s.licenses = ["Ruby".freeze]
1818
s.rdoc_options = ["--title".freeze, "JSON implemention for ruby".freeze, "--main".freeze, "README.md".freeze]
19-
s.required_ruby_version = Gem::Requirement.new("~> 2.0".freeze)
2019
s.rubygems_version = "2.6.8".freeze
2120
s.summary = "JSON Implementation for Ruby".freeze
2221
s.test_files = ["./tests/test_json.rb".freeze, "./tests/test_json_addition.rb".freeze, "./tests/test_json_encoding.rb".freeze, "./tests/test_json_fixtures.rb".freeze, "./tests/test_json_generate.rb".freeze, "./tests/test_json_generic_object.rb".freeze, "./tests/test_json_string_matching.rb".freeze, "./tests/test_json_unicode.rb".freeze]

0 commit comments

Comments
 (0)