Skip to content

fix: support named captures #44

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
kou merged 22 commits into from
Sep 18, 2022
+57 −0
Merged

fix: support named captures #44

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
16a4b46
feat: add test for named captures
ericgpks Jul 7, 2022
a3a7de3
fix: change test to be
ericgpks Jul 7, 2022
aecf835
feat: add empty method
ericgpks Jul 12, 2022
f9e9510
Update ext/strscan/strscan.c
ericgpks Jul 14, 2022
cd5c2c5
feat: change to use onig_foreach_name
ericgpks Aug 31, 2022
d8178c0
Merge branch 'master' into fix-support-named-captures
ericgpks Sep 13, 2022
1e1bf0f
fix: prepare another method to use main and change main logic
ericgpks Sep 13, 2022
666f22b
Merge branch 'fix-support-named-captures' of https://github.com/ericg…
ericgpks Sep 13, 2022
7380bda
feat: add document
ericgpks Sep 13, 2022
83d1655
Update ext/strscan/strscan.c
ericgpks Sep 14, 2022
617c3f9
Update test/strscan/test_stringscanner.rb
ericgpks Sep 14, 2022
83f042e
Update ext/strscan/strscan.c
ericgpks Sep 14, 2022
aecf844
Update test/strscan/test_stringscanner.rb
ericgpks Sep 14, 2022
30fb1b4
fix regex format in test
ericgpks Sep 14, 2022
f63639c
fix: change to use assert to check boolean
ericgpks Sep 14, 2022
aa60ea8
Update test/strscan/test_stringscanner.rb
ericgpks Sep 14, 2022
a139d4d
fix: change test to check length of a string
ericgpks Sep 16, 2022
4ed10e4
Merge branch 'fix-support-named-captures' of https://github.com/ericg…
ericgpks Sep 16, 2022
171a940
fix: change order to test expected and actual
ericgpks Sep 17, 2022
af71c09
fix: change expected value to use literal
ericgpks Sep 17, 2022
5b11f73
Fix style
kou Sep 18, 2022
5def9c2
Simplify
kou Sep 18, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 13 additions & 0 deletions ext/strscan/strscan.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1458,6 +1458,17 @@ strscan_fixed_anchor_p(VALUE self)
return p->fixed_anchor_p ? Qtrue : Qfalse;
}

/*
* call-seq:
* scanner.named_captures -> hash
*
*
*/
static VALUE
strscan_named_captures(VALUE self)
{
}
Comment thread
ericgpks marked this conversation as resolved.

/* =======================================================================
Ruby Interface
======================================================================= */
Expand Down Expand Up @@ -1652,4 +1663,6 @@ Init_strscan(void)
rb_define_method(StringScanner, "inspect", strscan_inspect, 0);

rb_define_method(StringScanner, "fixed_anchor?", strscan_fixed_anchor_p, 0);

rb_define_method(StringScanner, "named_captures", strscan_named_captures, 0);
}
8 changes: 8 additions & 0 deletions test/strscan/test_stringscanner.rb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -763,6 +763,14 @@ def test_fixed_anchor_false
assert_equal(false, StringScanner.new("a", fixed_anchor: nil).fixed_anchor?)
assert_equal(false, StringScanner.new("a", fixed_anchor: false).fixed_anchor?)
end

def test_named_captures
Comment thread
ericgpks marked this conversation as resolved.
s = "foobarbaz"
re = /(?<f>foo)(?<r>bar)(?<z>baz)/
scan = StringScanner.new(s)
scan.match? re

@kou kou Sep 13, 2022

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can use assert {...} for boolean expression to get better failure message:

Suggested change
scan.match? re
assert do
scan.match?(re)
end

@ericgpks ericgpks Sep 14, 2022

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it to be assert scan.match?(re) ?

@kou kou Sep 15, 2022

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In general, no. assert scan.match?(re) and assert {scan.match?(re)} is difference. And the latter is preferred for better message on failure. See also: https://github.com/ruby/power_assert

But we can't use assert {...} here. Sorry.
We can use assert {...} in ruby/strscan but we can't use assert {...} in ruby/ruby. Because ruby/ruby doesn't use test-unit gem.

So we should use assert_true(scan.match?(re)) here.

@ericgpks ericgpks Sep 16, 2022

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think that scan.match?(re) returns a length of matching. So can I change this test to check length of string?

@kou kou Sep 16, 2022

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow! I thought that it returns a boolean value because its name ends with ?.

You're right. We should use assert_equal(LENGTH, scan.match?(re)).

@ericgpks ericgpks Sep 17, 2022

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I fixed this test.

Comment thread
ericgpks marked this conversation as resolved.
Outdated
assert_equal({"f"=>"foo", "r"=>"bar", "z"=>"baz"}, scan.named_captures)
Comment thread
ericgpks marked this conversation as resolved.
Outdated
end
end

class TestStringScannerFixedAnchor < TestStringScanner
Expand Down