Description

radarek

opened on Jan 24, 2016

Issue body actions

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

   on Jan 24, 2016

   Collaborator

   More actions

   👍 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

   on Apr 27, 2018

   Contributor

   More actions

   👍 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

   on Oct 16, 2021

   Member

   More actions

   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

   Reacted by Keenan Brock

Issue actions