santagada at gmail.com
Wed Apr 25 05:22:24 CEST 2007
Em 24/04/2007, às 05:58, Carl Friedrich Bolz escreveu:
> Hi Leonardo!
> Leonardo Santagada wrote:
>> pythongrammar.txt file on parsing test dir.
> This is the point that does not really make sense to me already. You
> should base your grammar on the official (Mozilla) one, otherwise you
> will end up chasing and understanding strange corner cases. Also it
> makes updating the grammar much easier for future versions of the
> language. If you don't want to (or can't, for some reason) use the
> official grammar directly, you should use at least parts of it.
I've tried to start, converted the number literals, but somehow they
are not working, can you try it and help me to understand how do I
find and fix this kind of bugs (i'm a fast learner, antonio showed me
how to find then when translating rpython code and now I can find and
fix bugs miself).
>> The thing is still in its
>> infancy, but I just wanted to say that I am working on it, and would
>> love any help. It is living in users/santagada/newjsparser for now...
>> will move to js when it is at least kind of working.
> I looked quickly at the directory, my comment is that you should
> write tests. Otherwise you will get regressions when you change the
> grammar. As you know, PyPy is very strict about untested code, and an
> untested grammar is not really a good plan either. Also, the tests
> have told me what you expect to work, in the current state it is
> completely worthless to me since I have no clue what should work and
> what not.
Done man, now I will be writing tons of tests, I just need to get the
first ones to work. They are in test_parser.py
>> About the parsing lib I have some questions (should be asking cfbolz
>> but I don't seem to find him on irc lately):
> Sorry, I am enjoying my free time too much currently and don't want to
> be online consistently. Mail is fine for such a technical discussion
> anyway (and it has the additional benefit that I can answer you during
> this boring lecture I am sitting in now).
>> * ParsingError should print something like lineno: self.lineno+1
>> collum: self.collum+1 (I know there is a pretty error function, but
> There is no class called ParsingErro and ParseError (which exists)
> not have a lineno attribute, so what exactly do you mean? If it is
> a cosmetic change, just fix it yourself, I will look at it. Note that
> the line and column numbers of the parse errors are guesses only,
> is no way to pinpoint the exact error location with PEGs.
It has a sourcepos and that has a lineno... but it is all working ok
>> * IGNORE and friends, how do I make then work?
> What friends? IGNORE itself is a bit documented, so read the docs (and
> the tests). The idea is that IGNORE is a regular expression of things
> that should be ignored in the input (probably comments and
Read the documentation, I think that there is not much point in
separating pure regexes from the other actions (at least if I
understand correctly in real PEGs there isn't such separations). But
I am living with then, and man great docs!
>> * I don't know why when it recognizes something (or I think it
>> should) it just gives me list index out of bounds (I will check what
>> is happening, but if someone know why it would be better).
> Check in a failing test (skipped or not, it's your directory anyway),
> otherwise I have no clue what you are talking about.
There is some failing tests now
>> About the grammar:
>> * based on python grammar, so automatic semicolon insertion is OFF.
>> You have optional semicolons at the end of lines though
>> * No regex literals (ahh someday maybe, i'm not really interested in
> Regex literals will only be possible after refactoring the parser
> generator, I fear.
I don't like then anyway so thats ok.
>> * Lot's of work to be done still :)
> Carl Friedrich
Thanks and thanks in advance,
santagada at gmail.com
More information about the Pypy-dev