On Thursday 29 November 2007 20:54:13 Will Coleda wrote:

> pynie currently doesn't run:
>
> $ ../../parrot pynie.pbc
> No such attribute 'PGE::OPTable
> current instr.: 'parrot;Pynie::Grammar;onload' pc 117653 (src/parser/
> indent.pir:20)
> called from Sub 'parrot;Pynie::Compiler;main' pc -1 ((unknown file):-1)
>
> This oafish patch avoids that issue, allow make test to pass some
> tests. I can only hope a more refined patch that actually set the
> attribute appropriately would pass all the tests; If not, perhaps
> this should be applied.
>
> $ svn diff
> Index: src/parser/indent.pir
> ===================================================================
> --- src/parser/indent.pir       (revision 23256)
> +++ src/parser/indent.pir       (working copy)
> @@ -17,7 +17,7 @@
>       .local pmc optable, ws
>       optable = get_global '$optable'
>       ws = get_global 'ws'
> -    setattribute optable, "PGE::OPTable\x0&!ws", ws
> +    setattribute optable, "&!ws", ws
>       .return ()
>   .end

Allison knows better than I do, but that patch looks correct.  I did something 
similar in a lot of code on the pdd15oo branch, way back when.

-- c

Reply via email to