Remove -Wno-deprecated-warnings from cflags for clang

Remove -Wno-deprecated-warnings from cflags for clang

third-party libraries that use auto_ptr or other deprecated APIs
should either define this locally or be updated.

BUG=324299
R=thakis@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=237971

[Nico, 2013-11-29 15:49:03]

Lgtm
On Nov 29, 2013 4:04 AM, <jochen@chromium.org> wrote:

> Reviewers: Nico,
>
> Description:
> Remove -Wno-deprecated-warnings from cflags for clang
>
> third-party libraries that use auto_ptr or other deprecated APIs
> should either define this locally or be updated.
>
> BUG=324299
> R=thakis@chromium.org
>
> Please review this at https://codereview.chromium.org/96833003/
> Index: build/common.gypi
> diff --git a/build/common.gypi b/build/common.gypi
> index e7cccb04381bfb54c382e1b8693b0671392cf4f2..
> 025b0753fc4efdbede85f7d32bde6535b5661219 100644
> --- a/build/common.gypi
> +++ b/build/common.gypi
> @@ -3277,9 +3277,6 @@
>                # code generated by flex (used in angle) contains that
> keyword.
>                # http://crbug.com/255186
>                '-Wno-deprecated-register',
> -
> -              # This warns about auto_ptr<>, used in third-party code.
> -              '-Wno-deprecated-declarations',
>              ],
>              'cflags!': [
>                # Clang doesn't seem to know know this flag.
> @@ -4001,9 +3998,6 @@
>                  # code generated by flex (used in angle) contains that
> keyword.
>                  # http://crbug.com/255186
>                  '-Wno-deprecated-register',
> -
> -                # This warns about auto_ptr<>, used in third-party code.
> -                '-Wno-deprecated-declarations',
>                ],
>              }],
>              ['clang==1 and clang_use_chrome_plugins==1', {
>
>
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[commit-bot: I haz the power, 2013-11-29 20:24:23]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jochen@chromium.org/96833003/1

[commit-bot: I haz the power, 2013-11-30 05:15:28]

Change committed as 237971