Remove DeprecationWarning filtering from testserver.py

Remove DeprecationWarning filtering from testserver.py

BUG=none


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=164516

[M-A Ruel, 2012-09-24 23:20:41]

lgtm

Le lundi 24 septembre 2012,  <mattm@chromium.org> a écrit :
> Reviewers: Marc-Antoine Ruel,
>
> Description:
> Remove DeprecationWarning from testserver.py
>
> BUG=none
>
>
> Please review this at http://codereview.chromium.org/10983003/
>
> SVN Base: svn://svn.chromium.org/chrome/trunk/src
>
> Affected files:
>   M net/tools/testserver/testserver.py
>
>
> Index: net/tools/testserver/testserver.py
> diff --git a/net/tools/testserver/testserver.py
b/net/tools/testserver/testserver.py
> index
aeb0575118b6761feb3069ff64c8e5f4838f2974..33997e951ccb4496be3c70904caff391972626f1
100755
> --- a/net/tools/testserver/testserver.py
> +++ b/net/tools/testserver/testserver.py
> @@ -38,9 +38,6 @@ import urlparse
>  import warnings
>  import zlib
>
> -# Ignore deprecation warnings, they make our output more cluttered.
> -warnings.filterwarnings("ignore", category=DeprecationWarning)
> -
>  import echo_message
>  import pyftpdlib.ftpserver
>  import tlslite
>
>
>

[mattm, 2012-10-27 02:10:25]

Strange, I never got this email.

On 2012/09/24 23:20:41, Marc-Antoine Ruel wrote:
> lgtm
> 
> Le lundi 24 septembre 2012,  <mailto:mattm@chromium.org> a écrit :
> > Reviewers: Marc-Antoine Ruel,
> >
> > Description:
> > Remove DeprecationWarning from testserver.py
> >
> > BUG=none
> >
> >
> > Please review this at http://codereview.chromium.org/10983003/
> >
> > SVN Base: svn://svn.chromium.org/chrome/trunk/src
> >
> > Affected files:
> >   M net/tools/testserver/testserver.py
> >
> >
> > Index: net/tools/testserver/testserver.py
> > diff --git a/net/tools/testserver/testserver.py
> b/net/tools/testserver/testserver.py
> > index
>
aeb0575118b6761feb3069ff64c8e5f4838f2974..33997e951ccb4496be3c70904caff391972626f1
> 100755
> > --- a/net/tools/testserver/testserver.py
> > +++ b/net/tools/testserver/testserver.py
> > @@ -38,9 +38,6 @@ import urlparse
> >  import warnings
> >  import zlib
> >
> > -# Ignore deprecation warnings, they make our output more cluttered.
> > -warnings.filterwarnings("ignore", category=DeprecationWarning)
> > -
> >  import echo_message
> >  import pyftpdlib.ftpserver
> >  import tlslite
> >
> >
> >

[commit-bot: I haz the power, 2012-10-27 02:11:46]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mattm@chromium.org/10983003/10001

[commit-bot: I haz the power, 2012-10-27 02:11:48]

Presubmit check for 10983003-10001 failed and returned exit status 1.


Running presubmit commit checks ...

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    net/tools/testserver

[mattm, 2012-10-27 02:14:46]

+rtenneti: quick OWNERS approval

[ramant (doing other things), 2012-10-27 02:19:36]

lgtm

[commit-bot: I haz the power, 2012-10-27 02:20:55]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mattm@chromium.org/10983003/10001

[commit-bot: I haz the power, 2012-10-27 06:06:30]

Change committed as 164516