On 25 February 2010 17:12, Andrew Bennetts
<email address hidden> wrote:
> Review: Approve uifactory
> On IRC Martin asked me to look at the UIFactory changes, i.e. the new show_user_warning/suppressed_warnings API. It looks fine to me. I like the paranoia when looking up the warning string and trying to format it. I think the API to temporarily suppress a particular warning is a bit cumbersome, but good enough for now (perhaps we'll think of improvements later if we use it more heavily). So, for that aspect of the patch, +1.
I was planning to build on addCleanup etc, but this isn't in 2.0 where
this is targeted.
It might be nice if there was a contextmanager-like object for "with a
warning suppressed".
On 25 February 2010 17:12, Andrew Bennetts warning/ suppressed_ warnings API. It looks fine to me. I like the paranoia when looking up the warning string and trying to format it. I think the API to temporarily suppress a particular warning is a bit cumbersome, but good enough for now (perhaps we'll think of improvements later if we use it more heavily). So, for that aspect of the patch, +1.
<email address hidden> wrote:
> Review: Approve uifactory
> On IRC Martin asked me to look at the UIFactory changes, i.e. the new show_user_
I was planning to build on addCleanup etc, but this isn't in 2.0 where
this is targeted.
It might be nice if there was a contextmanager-like object for "with a
warning suppressed".
-- launchpad. net/~mbp/>
Martin <http://