summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authornobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2014-11-30 11:05:58 +0000
committernobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2014-11-30 11:05:58 +0000
commitd40aad1c09e509e585a8f23a2e713048ad5f3384 (patch)
tree7ca7450e19cec27e6bda08eaa3525002449c8088
parent1918ea16c788e6fcca468f4223bb24bb8b358f22 (diff)
parse.y: fix a typo
* parse.y (regexp_contents): fix a typo. pointed out by wanabe. [ruby-dev:48741] [Bug #10543] git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@48655 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r--ChangeLog5
-rw-r--r--parse.y2
-rw-r--r--test/ripper/test_sexp.rb20
3 files changed, 26 insertions, 1 deletions
diff --git a/ChangeLog b/ChangeLog
index 99bf2004c1..c2ed02967f 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+Sun Nov 30 20:05:55 2014 Nobuyoshi Nakada <nobu@ruby-lang.org>
+
+ * parse.y (regexp_contents): fix a typo. pointed out by wanabe.
+ [ruby-dev:48741] [Bug #10543]
+
Sun Nov 30 18:55:32 2014 Tanaka Akira <akr@fsij.org>
* test/ruby/test_process.rb (test_deadlock_by_signal_at_forking):
diff --git a/parse.y b/parse.y
index 0aacaa3713..8af2c9a189 100644
--- a/parse.y
+++ b/parse.y
@@ -4261,7 +4261,7 @@ regexp_contents: /* none */
/*%
VALUE s1 = 0, s2 = 0, n1 = $1, n2 = $2;
if (ripper_is_node_yylval(n1)) {
- s1 = RNODE(n2)->nd_cval;
+ s1 = RNODE(n1)->nd_cval;
n1 = RNODE(n1)->nd_rval;
}
if (ripper_is_node_yylval(n2)) {
diff --git a/test/ripper/test_sexp.rb b/test/ripper/test_sexp.rb
index 9be4aa7d41..adf8c46ec7 100644
--- a/test/ripper/test_sexp.rb
+++ b/test/ripper/test_sexp.rb
@@ -18,4 +18,24 @@ class TestRipper::Sexp < Test::Unit::TestCase
assert_nil Ripper.sexp("/*/")
assert_nil Ripper.sexp("/+/")
end
+
+ def test_regexp_content
+ sexp = Ripper.sexp('//')
+ assert_nil search_sexp(:@tstring_content, search_sexp(:regexp_literal, sexp))
+
+ sexp = Ripper.sexp('/foo/')
+ assert_equal 'foo', search_sexp(:@tstring_content, search_sexp(:regexp_literal, sexp))[1]
+
+ sexp = Ripper.sexp('/(?<n>a(b|\g<n>))/')
+ assert_equal '(?<n>a(b|\g<n>))', search_sexp(:@tstring_content, search_sexp(:regexp_literal, sexp))[1]
+ end
+
+ def search_sexp(sym, sexp)
+ return sexp if !sexp or sexp[0] == sym
+ sexp.find do |e|
+ if Array === e and e = search_sexp(sym, e)
+ return e
+ end
+ end
+ end
end if ripper_test