Re: SVN: Products.Five/branches/regebro-traversal_refactor/ Removed last rests of Traversable usage. Deprecated Traversable and FiveTraversable.

On 4/28/06, Philipp von Weitershausen <philipp@xxxxxxxxxxxxxxxx> wrote:

> > +        if zope.deprecation.__show__():
>
> This is usage of the show switch doesn't make any sense, by the way,
> especially not when you're using warnings.warn afterwards.
> DeprecationWarnings are disabled via the Python command line and a
> global warning filter.

OK. I just did it to get rid of warnings in the unit tests. I can get
rid of them, no problemo.

> This deprecation warning doesn't make sense to me. IMO, the Traversable
> class should become a deprecated and *empty* class, just so that
> importing and subclassing it will still work. The rest of its
> functionality should be useless now.

Sure. I actually did this mostly because I didn't know how to issue a
deprecation warning for subclassing a class. :)

> By the way, don't forget to put your estimate on when you'll be able to
> merge your two branches into the Zope 2.10 wiki:
> http://www.zope.org/Wikis/DevSite/Projects/Zope2.10/Tasks.

Righto!

> Being-away-from-the-internet-for-the-weekend-ly,

Well, welcome back on Tuesday, when the branches are merged! ;)


--
Lennart Regebro, Nuxeo     http://www.nuxeo.com/
CPS Content Management     http://www.cps-project.org/