[Qt-creator] CaseStatementAST visitor

[Marek Jasovsky]

thanks, I will update this locally..

I have noticed some more empty visitors... should I send you a list?

btw... NewTypeIdAST has zero set for lastToken(). (my code displays -1 in
that case...) can you have a look at it too?

thanks

Marek

2009/2/10 Roberto Raggi <roberto.raggi at trolltech.com>

> Hi Marek,
>
> That's wrong! good catch. The code should look like
>
> if (visitor->visit(this)) {
>     accept(expression, visitor);
>     accept(statement, visitor);
> }
>
> I push a fix in our internal repository, it will be available tomorrow
> in the public git repo.
>
> Thanks for the report.
> ciao robe
>
>
> On Feb 9, 2009, at 10:45 PM, Marek Jasovsky wrote:
>
> > Hi
> >
> > in AST:cpp
> >
> > void CaseStatementAST::accept0(ASTVisitor *visitor)
> > {
> >     if (visitor->visit(this)) {
> >     }
> > }
> >
> > is this correct? my tools now works for all nodes, but it did not
> > expand CaseStatementAST.
> >
> > if I replace above pasted code with content of
> > PrettyPrinter::visit(CaseStatementAST) then it seems to be ok and
> > node gets expanded
> >
> > thanks
> >
> > Marek
> >
> >
> > _______________________________________________
> > Qt-creator mailing list
> > Qt-creator at trolltech.com
> > http://lists.trolltech.com/mailman/listinfo/qt-creator
>
> _______________________________________________
> Qt-creator mailing list
> Qt-creator at trolltech.com
> http://lists.trolltech.com/mailman/listinfo/qt-creator
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.qt-project.org/pipermail/qt-creator-old/attachments/20090210/85afd242/attachment.html