search

meerkat-parser / Meerkat

Meerkat parsers
24 stars 10 forks source link

ScalaJs compatibility #1

Open omidb opened 9 years ago

omidb commented 9 years ago

Hi, is this lib ScalaJs compatible?

afroozeh commented 9 years ago

The short answer is that we don't know yet! This is something we definitely want. As we rely on Scala core libraries, there should not be a problem in principle. However, there may be two problems: we use some Java collections and also use Scala Macros.

July will be very busy and I don't think we will have time to investigate it. If you are interested, do you want to take the initiative and try to compile it to ScalaJS? We can work on the issues that you may encounter.

omidb commented 9 years ago

I would be happy to help but kinda too busy, as long as you don't reflection in your code you should be safe. Another not would be running your parsers on some benchmark for speed like these in "fastparse": https://github.com/lihaoyi/fastparse

afroozeh commented 9 years ago

We don't use reflection. Ok, I thought you are willing to try it soon, that's why I proposed you take a look. In any case, we will take a look into it. Thanks for the reference to the fastparse, we will consider that as well. We will announce on twitter if we added new functionality (You can follow @IAnastassija and @afruze or just watch the repo).