Skip to content

Remove "raise vs E2MM#Raise" benchmark from repository #95

Open

Description

@radarek

It's nice that someone found that this library is so slow but I think most Ruby devs don't know that e2mmap even exists (actually I learned about this library from this repo). IMHO it would be better to not tell other developers that it exists. I even grepped all gems from ~/.rbenv/ (more that 2000 gems) and only rdoc & yard gems use it (actually they include file ruby_lex.rb). All other benchmarks are more or less used in a real code but this one is completely impractical.

Activity

  1. nateberkopec commented on Jan 24, 2016

    @nateberkopec
    Collaborator

    👍 I've never heard of it, but I wasn't around when it was merged so I dunno the reasoning.

  2. kbrock commented on Apr 27, 2018

    @kbrock
    Contributor

    👍 does it make sense to just convert this into a PR - seems easier for the maintainers to just click merge.

    also noticed that e2mmap has no readme and last changed in 2013

  3. etagwerker commented on Oct 16, 2021

    @etagwerker
    Member

    At the moment that particular benchmark is failing (#196) with Ruby 3.0.2, so I went ahead and submitted a PR to remove it from this repo: #197

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions