mirror of
https://github.com/ruby/ruby.git
synced 2022-11-09 12:17:21 -05:00
* parse.y (rb_enc_symname2_p): :! is valid symbol. [Bug #5136]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_1_9_3@32809 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
This commit is contained in:
parent
a781b40242
commit
ceb5e60c81
3 changed files with 16 additions and 8 deletions
|
@ -1,3 +1,7 @@
|
|||
Tue Aug 2 11:42:15 2011 Nobuyoshi Nakada <nobu@ruby-lang.org>
|
||||
|
||||
* parse.y (rb_enc_symname2_p): :! is valid symbol. [Bug #5136]
|
||||
|
||||
Tue Aug 2 03:26:02 2011 NAKAMURA Usaku <usa@ruby-lang.org>
|
||||
|
||||
* test/rake/test_rake_directory_task.rb (TestRakeDirectoryTask#
|
||||
|
|
2
parse.y
2
parse.y
|
@ -9738,7 +9738,7 @@ rb_enc_symname2_p(const char *name, long len, rb_encoding *enc)
|
|||
break;
|
||||
|
||||
case '!':
|
||||
if (len == 1) return FALSE;
|
||||
if (len == 1) return TRUE;
|
||||
switch (*++m) {
|
||||
case '=': case '~': ++m; break;
|
||||
default: return FALSE;
|
||||
|
|
|
@ -3,21 +3,25 @@ require 'test/unit'
|
|||
class TestSymbol < Test::Unit::TestCase
|
||||
# [ruby-core:3573]
|
||||
|
||||
def assert_eval_inspected(sym)
|
||||
def assert_eval_inspected(sym, valid = true)
|
||||
n = sym.inspect
|
||||
if valid
|
||||
bug5136 = '[ruby-dev:44314]'
|
||||
assert_not_match(/\A:"/, n, bug5136)
|
||||
end
|
||||
assert_nothing_raised(SyntaxError) {assert_equal(sym, eval(n))}
|
||||
end
|
||||
|
||||
def test_inspect_invalid
|
||||
# 2) Symbol#inspect sometimes returns invalid symbol representations:
|
||||
assert_eval_inspected(:"!")
|
||||
assert_eval_inspected(:"=")
|
||||
assert_eval_inspected(:"0")
|
||||
assert_eval_inspected(:"=", false)
|
||||
assert_eval_inspected(:"0", false)
|
||||
assert_eval_inspected(:"$1")
|
||||
assert_eval_inspected(:"@1")
|
||||
assert_eval_inspected(:"@@1")
|
||||
assert_eval_inspected(:"@")
|
||||
assert_eval_inspected(:"@@")
|
||||
assert_eval_inspected(:"@1", false)
|
||||
assert_eval_inspected(:"@@1", false)
|
||||
assert_eval_inspected(:"@", false)
|
||||
assert_eval_inspected(:"@@", false)
|
||||
end
|
||||
|
||||
def assert_inspect_evaled(n)
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue