Skip to content
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.

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

Support nested calls #398

Closed
wants to merge 4 commits into from
Closed

Conversation

maximeallanic
Copy link

@maximeallanic maximeallanic commented Apr 27, 2020

Fixes for #341

Benchmark:
From this Pull Request:

      26,884,850 op/s » 1 style
      24,148,076 op/s » 2 styles
      21,742,104 op/s » 3 styles
      26,739,202 op/s » cached: 1 style
      24,135,040 op/s » cached: 2 styles
      22,669,978 op/s » cached: 3 styles
      10,143,420 op/s » cached: 1 style with newline
       5,015,684 op/s » cached: 1 style nested intersecting
      13,039,434 op/s » cached: 1 style nested non-intersecting


  Suites:  1
  Benches: 9
  Elapsed: 7,992.41 ms

From master:

      24,860,725 op/s » 1 style
      23,484,116 op/s » 2 styles
      20,689,530 op/s » 3 styles
      26,039,018 op/s » cached: 1 style
      23,091,189 op/s » cached: 2 styles
      21,367,378 op/s » cached: 3 styles
       9,357,187 op/s » cached: 1 style with newline
       5,068,170 op/s » cached: 1 style nested intersecting
      13,006,792 op/s » cached: 1 style nested non-intersecting


  Suites:  1
  Benches: 9
  Elapsed: 7,984.43 ms

IssueHunt Summary

Referenced issues

This pull request has been submitted to:


@sindresorhus
Copy link
Member

This is a duplicate of #392.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants