From: Moritz H. <ant...@gm...> - 2008-02-21 15:23:08
|
It seems like I was too fast: Committed revision 2371. Thanks, Moritz On Thu, Feb 21, 2008 at 3:51 PM, Moritz Hoffmann <ant...@gm...> wrote: > Sending rexxdoc/frontend.rex > Sending rexxdoc/rexxdoc.cls > Sending rexxdoc/style.css > Transmitting file data ... > Committed revision 2370. > > I'd be glad if you could try again! > Thank you, > Moritz > > > > On Thu, Feb 21, 2008 at 3:28 PM, Moritz Hoffmann <ant...@gm...> wrote: > > Hi Jon, > > that is very helpful. The first error seems to be my fault forgetting > > to parse some keyword. I'll have a look at it. The other errors are > > not that simple to analyze. Maybe you can supply me the complete lines > > the errors happen on? I can also modify the class to print them. That > > would be very helpful! > > One disadvantage I still have to overcome is the ability to remove > > comments on keyword instructions. It requires to parse them char by > > char as only comments outside of string literals are real comments. If > > I'd look for /* I would match them inside literals as well which is > > not intended. > > Maybe you have a solution for parsing such strings in a simple and > > performant way? > > > > This line for example: ::METHOD MyMethod /* CLASS */ > > At the moment if will complain about /* */ being unknown keywords, > > although they are actually comments. > > ::METHOD "My Method" is valid rexx but the rexxdoc will complain > > about Method" not being valid... I'd be glad for comments on solving > > this issue! > > > > The error appearing inside the frontend.rex should go away in the next revision. > > Moritz > > > > On Thu, Feb 21, 2008 at 10:35 AM, Sahananda (Jon) Wolfers > > > > > > <sah...@wi...> wrote: > > > Hi Moritz, > > > > > > well, that fixes that problem. When I give it my big class library it kicks > > > out lots of messages like this: > > > > > > oorexx\obrowser\obrowser.rex:5371 Invalid option public on routine > > > AccessDate2Date > > > > > > or > > > > > > oorexx\obrowser\obrowser.rex:5795 Keyword error /* method b Unguarded ) > > > oorexx\obrowser\obrowser.rex:5795 Keyword error BOLD method b Unguarded ) > > > oorexx\obrowser\obrowser.rex:5795 Keyword error */ method b Unguarded ) > > > > > > then finally dies with > > > > > > 66 *-* out~say('<strong>'r~getFile~name'</strong><br>') > > > Error 97 running D:\My Documents\ooRexx > > > Project\incubator\rexxdoc\frontend.rex line 66: Object method not found > > > Error 97.1: Object "The NIL object" does not understand message "NAME" > > > > > > However, not before it pumps out a very striking doc.html which Firefox > > > manages to render even though several of the tags are not closed. > > > I like the way you get the source as tooltips. However, several of the > > > methods source code is truncated. > > > > > > Once again, I hope that is helpful to you. > > > > > > Jon > > > > > > > > > > > > > > > On 21/02/2008, Moritz Hoffmann <ant...@gm...> wrote: > > > > Hi Jon, > > > > thank you for the tip, I just fixed it in revision 2369. I'd be glad > > > > if you could tell me if it works! The reason was that I added a state > > > > parameter to the sourceElement class, and the sourceRequires class, > > > > chich is a subclass to sourceElement was not updated accordingly... > > > > > > > > Moritz > > > > > > > > > > > > On Thu, Feb 21, 2008 at 6:43 AM, Sahananda (Jon) Wolfers > > > > <sah...@wi...> wrote: > > > > > Hi Moritz, > > > > > > > > > > Just having a play, trying to get the feel of the thing. It worked fine > > > on > > > > > a simple script, so I threw a larger class library at it, and it crashed > > > in > > > > > the sourceRequires init method as an unexpected arg (self~state) was > > > passed. > > > > > > > > > > I hope it is helpful to you to have the screen output: > > > > > > > > > > D:\My Documents\ooRexx Project\incubator\rexxdoc>rexx frontend > > > > > c:\oorexx\obrowser\obrowser.rex > > > > > 846 *-* use strict arg name, source, rxdoc, target, options > > > > > 567 *-* sourceRequires = .SourceRequires~new(name, source, rxdoc, > > > > > parent, self~parser~options, self~state) > > > > > 167 *-* rd = > > > > > .RequiresDirective~new(self)~~parse(line,source,self~getResetRxDoc, > > > tree) > > > > > 248 *-* self~parseLine(line,i) > > > > > 145 *-* repeat("DIRECTIVE") > > > > > 112 *-* self~parseLine(line~strip,line) > > > > > 48 *-* parser~parse > > > > > Error 93 running D:\My Documents\ooRexx > > > > > Project\incubator\rexxdoc\rexxdoc.cls line 846: Incorrect call to > > > method > > > > > Error 93.902: Too many arguments in invocation of method; 5 expected > > > > > > > > > > cheers, > > > > > > > > > > Jon > > > > > > > > > > > > > > > > > > > > > > ------------------------------------------------------------------------- > > > > > This SF.net email is sponsored by: Microsoft > > > > > Defy all challenges. Microsoft(R) Visual Studio 2008. > > > > > http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ > > > > > _______________________________________________ > > > > > Oorexx-devel mailing list > > > > > Oor...@li... > > > > > https://lists.sourceforge.net/lists/listinfo/oorexx-devel > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > Moritz Hoffmann; > > > > http://antiguru.de/ > > > > > > > > ------------------------------------------------------------------------- > > > > This SF.net email is sponsored by: Microsoft > > > > Defy all challenges. Microsoft(R) Visual Studio 2008. > > > > http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ > > > > _______________________________________________ > > > > Oorexx-devel mailing list > > > > Oor...@li... > > > > https://lists.sourceforge.net/lists/listinfo/oorexx-devel > > > > > > > > > > > > > ------------------------------------------------------------------------- > > > This SF.net email is sponsored by: Microsoft > > > Defy all challenges. Microsoft(R) Visual Studio 2008. > > > http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ > > > _______________________________________________ > > > Oorexx-devel mailing list > > > Oor...@li... > > > https://lists.sourceforge.net/lists/listinfo/oorexx-devel > > > > > > > > > > > > > > -- > > Moritz Hoffmann; > > http://antiguru.de/ > > > > > > -- > > > Moritz Hoffmann; > http://antiguru.de/ > -- Moritz Hoffmann; http://antiguru.de/ |