[Docutils-develop] Re: [Docutils-checkins] r3553 - in branches/blais_interrupt_render/docutils/docutils: core.py transforms/__init__.py

[David G. <go...@py...>]

[bl...@us...]
> Log:
> Minor changes to transform for running them in a separate phase.
>
> Modified: branches/blais_interrupt_render/docutils/docutils/core.py
> ===================================================================
> @@ -224,7 +224,7 @@
>          if self.settings.dump_transforms:
>              print >>sys.stderr, '\n::: Transforms applied:'
>              print >>sys.stderr, pprint.pformat(
> -                self.document.transformer.applied)
> +                self.document.transformer.applied[1:])

This doesn't do what you seem to intend.  I think you want
[a[1:] for a in self.document.transformer.applied].

> Modified: branches/blais_interrupt_render/docutils/docutils/transforms/__init__.py
> ===================================================================
> @@ -177,4 +177,4 @@
>              priority, transform_class, pending = self.transforms.pop()
>              transform = transform_class(self.document, startnode=pending)
>              transform.apply()
> -            self.applied.append((priority, transform_class, pending))
> +            self.applied.append((transform, priority, transform_class, pending))

Why are you adding the transform itself?

--
David Goodger <http://python.net/~goodger>