On Thu, Jan 31, 2013 at 3:26 PM, sigmundch <notifications@github.com> wrote:

> In test/visitor_test.dart:
>
> > +.foobar1 {
> > +}
> > +.xyzzy .foo #my-div {
> > +  color: #ff0000;
> > +}
> > +div.hello {
> > +  font: arial;
> > +}''');
> > +}
> > +
> > +class PolyfillEmitter extends CssPrinter {
> > +  final String _prefix;
> > +
> > +  PolyfillEmitter(this._prefix);
> > +
> > +  void visit(var tree, {bool pretty: false}) => super.visitStyleSheet(tree);
>
> Revisiting John's suggestion about using double-dispatch. We should remove
> this visit method here, and in ClassVisitor, instead, implement in Visitor:
>
> void visit(Node node) => node.visit(this);
>
> Ideally the 'visit' method in node should be called 'accept', but that's
> ok. node.visit will know which call to make, in particular, a StyleSheet
> node will will call visitStyleSheet.
>

I prefer "accept" too.