lupa icon indicating copy to clipboard operation
lupa copied to clipboard

Character classes/set bad conversion

Open mingodad opened this issue 10 years ago • 3 comments

Looking at how lupa is build and trying to update lpeg version to 0.12 I found that the conversion of character classes do not work properly if we look at how the rule rule_class converts "[\t ]" => "((Pattern.P("")+Pattern.P("t")+Pattern.P(" ")))" (lang.lua 730) it's clear that it's not taking in account the escaped chars.

By the way is this project dead ? I saw that you stopped working on it and now is working on nyanga.

Thanks in advance for your time, attention and great work.

mingodad avatar Mar 08 '14 12:03 mingodad

On 3/8/14 1:46 PM, Domingo Alvarez Duarte wrote:

Looking at how lupa is build and trying to update lpeg version to 0.12 I found that the conversion of character classes do not work properly if we look at how the rule rule_class converts "[\t ]" => "((Pattern.P("")+Pattern.P("t")+Pattern.P(" ")))" (lang.lua 730) it's clear that it's not taking in account the escaped chars.

Good catch. Thanks! It'll be a few days before I fix it, but I probably have the same issue with Nyanga.

By the way is this project dead ? I saw that you stopped working on it and now is working on nyanga.

I didn't think anyone was actually using it, so I haven't done much with it lately. It has some nice features but has a much higher complexity, and not many people showed any interest in the language. It's also missing a decent run-time, but what I'll probably do is either add it as a front-end to Nyanga, or pull in the Nyanga run-time and port Lupa to TvmJIT (as I've done with Nyanga).

Thanks in advance for your time, attention and great work.

— Reply to this email directly or view it on GitHub https://github.com/richardhundt/lupa/issues/4.

richardhundt avatar Mar 08 '14 16:03 richardhundt

Thanks for reply !

Probably it'll get more interest if announced on lua wiki because if for me it was hard to find it.

What's the advantage of using Tvmjit instead of luajit ? Been able to see the generated lua code to debug and check the correctness is a good option that probably will not exist using TvmJIT.

I'll update lua wiki pages inserting an entry for lupa, nyanga, and gaya.

The peg interface on nyanga is a lot more clear than on lupa.

Thanks for your time and attention !

On Sat, Mar 8, 2014 at 4:46 PM, Richard Hundt [email protected]:

On 3/8/14 1:46 PM, Domingo Alvarez Duarte wrote:

Looking at how lupa is build and trying to update lpeg version to 0.12 I found that the conversion of character classes do not work properly if we look at how the rule rule_class converts "[\t ]" => "((Pattern.P("")+Pattern.P("t")+Pattern.P(" ")))" (lang.lua 730) it's clear that it's not taking in account the escaped chars.

Good catch. Thanks! It'll be a few days before I fix it, but I probably have the same issue with Nyanga.

By the way is this project dead ? I saw that you stopped working on it and now is working on nyanga.

I didn't think anyone was actually using it, so I haven't done much with it lately. It has some nice features but has a much higher complexity, and not many people showed any interest in the language. It's also missing a decent run-time, but what I'll probably do is either add it as a front-end to Nyanga, or pull in the Nyanga run-time and port Lupa to TvmJIT (as I've done with Nyanga).

Thanks in advance for your time, attention and great work.

Reply to this email directly or view it on GitHub https://github.com/richardhundt/lupa/issues/4.

Reply to this email directly or view it on GitHubhttps://github.com/richardhundt/lupa/issues/4#issuecomment-37102547 .

mingodad avatar Mar 08 '14 19:03 mingodad

I added then to http://lua-users.org/wiki/LuaImplementations , you can edit then if you want a better description.

On Sat, Mar 8, 2014 at 7:13 PM, Domingo Alvarez Duarte [email protected]:

Thanks for reply !

Probably it'll get more interest if announced on lua wiki because if for me it was hard to find it.

What's the advantage of using Tvmjit instead of luajit ? Been able to see the generated lua code to debug and check the correctness is a good option that probably will not exist using TvmJIT.

I'll update lua wiki pages inserting an entry for lupa, nyanga, and gaya.

The peg interface on nyanga is a lot more clear than on lupa.

Thanks for your time and attention !

On Sat, Mar 8, 2014 at 4:46 PM, Richard Hundt [email protected]:

On 3/8/14 1:46 PM, Domingo Alvarez Duarte wrote:

Looking at how lupa is build and trying to update lpeg version to 0.12 I found that the conversion of character classes do not work properly if we look at how the rule rule_class converts "[\t ]" => "((Pattern.P("")+Pattern.P("t")+Pattern.P(" ")))" (lang.lua 730) it's clear that it's not taking in account the escaped chars.

Good catch. Thanks! It'll be a few days before I fix it, but I probably have the same issue with Nyanga.

By the way is this project dead ? I saw that you stopped working on it and now is working on nyanga.

I didn't think anyone was actually using it, so I haven't done much with it lately. It has some nice features but has a much higher complexity, and not many people showed any interest in the language. It's also missing a decent run-time, but what I'll probably do is either add it as a front-end to Nyanga, or pull in the Nyanga run-time and port Lupa to TvmJIT (as I've done with Nyanga).

Thanks in advance for your time, attention and great work.

Reply to this email directly or view it on GitHub https://github.com/richardhundt/lupa/issues/4.

Reply to this email directly or view it on GitHubhttps://github.com/richardhundt/lupa/issues/4#issuecomment-37102547 .

mingodad avatar Mar 08 '14 19:03 mingodad