diff options
author | Yusuke Endoh <mame@ruby-lang.org> | 2019-08-07 02:32:03 +0900 |
---|---|---|
committer | aycabta <aycabta@gmail.com> | 2019-08-16 06:02:45 +0900 |
commit | 723a37d0386bc20efedf516656c2ccafa889c89d (patch) | |
tree | f2d5bc94b781258f7090fb2644135f85a5e42c25 /test/rdoc | |
parent | 0a0760aa632f05bc04df395d0173580042d9f730 (diff) | |
download | ruby-723a37d0386bc20efedf516656c2ccafa889c89d.tar.gz |
Separate RDoc::TokenStream#add_tokens and #add_token
The old version of `add_tokens` accepts an array of tokens, and
multiple arguments of tokens by using `Array#flatten`.
And `add_token` was an alias to `add_tokens`.
I think it is unnecessarily flexible; in fact, all callsites of
`add_tokens` (except test) passes only an array of tokens.
And the code created a lot of temporal arrays.
This change makes `add_tokens` accept only one array of tokens,
and does `add_token` accept one token. It is a bit faster (about
1 second in Ruby's `make rdoc`), and it ls also cleaner in my point of
view.
Diffstat (limited to 'test/rdoc')
-rw-r--r-- | test/rdoc/test_rdoc_any_method.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/test/rdoc/test_rdoc_any_method.rb b/test/rdoc/test_rdoc_any_method.rb index 6dd46b0b46..615789dfb3 100644 --- a/test/rdoc/test_rdoc_any_method.rb +++ b/test/rdoc/test_rdoc_any_method.rb @@ -78,7 +78,7 @@ method(a, b) { |c, d| ... } ] @c2_a.collect_tokens - @c2_a.add_tokens(*tokens) + @c2_a.add_tokens(tokens) expected = '<span class="ruby-constant">CONSTANT</span>' @@ -96,7 +96,7 @@ method(a, b) { |c, d| ... } ] @c2_a.collect_tokens - @c2_a.add_tokens(*tokens) + @c2_a.add_tokens(tokens) assert_equal <<-EXPECTED.chomp, @c2_a.markup_code <span class="ruby-comment"># File xref_data.rb, line 1</span> |