Skip to content
New issue

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

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

Already on GitHub? # to your account

use StringScanner library for parsing #20

Merged
merged 1 commit into from
Jun 24, 2023
Merged

Conversation

gemmaro
Copy link
Collaborator

@gemmaro gemmaro commented Jun 24, 2023

before:

$ ruby bench.rb 
Warming up --------------------------------------
               parse     1.000  i/100ms
Calculating -------------------------------------
               parse      1.549  (± 0.0%) i/s -      8.000  in   5.164986s

after:

$ ruby bench.rb 
Warming up --------------------------------------
               parse     1.000  i/100ms
Calculating -------------------------------------
               parse      5.354  (± 0.0%) i/s -     27.000  in   5.043558s

* update changelog

* enable RuboCop's guard clause rule
@gemmaro gemmaro marked this pull request as ready for review June 24, 2023 07:29
@gemmaro gemmaro merged commit 6883808 into iyuuya:develop Jun 24, 2023
@gemmaro gemmaro deleted the re/strscan branch June 24, 2023 07:30
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant