Remove build/c++11 from the set of linter rules ever used.

Remove build/c++11 from the set of linter rules ever used.

The checks are not reliable for Rvalue references, and only are
allowing default/deleted constructors. They are based on the google3
internal rules which do not exactly match our own c++11 rules, and
may diverge more over time.

Email thread that led to this change here: https://groups.google.com/a/chromium.org/forum/#!topic/cxx/9TZvt8vyeEM

R=agable@chromium.org

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

[danakj, 2016-01-08 23:56:49]

Actually I think we should be more aggressive than this. Some PRESUBMITs will
override the rules to remove the defaults (cc/ does this) cuz it's more
lint-clean than average.

We don't want broken rules anywhere though.

[danakj, 2016-01-09 00:04:35]

PTAL

[danakj, 2016-01-09 00:04:42]

The CQ bit was checked by danakj@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-01-09 00:04:52]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1573663003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1573663003/20001

[commit-bot: I haz the power, 2016-01-09 00:04:53]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-01-09 00:04:53]

Dry run: No L-G-T-M from a valid reviewer yet. Only full committers are
accepted.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[danakj, 2016-01-09 00:05:22]

Description was changed from

==========
Remove build/c++11 from the default set of linter rules.

The checks are not reliable for Rvalue references, and only are
allowing default/deleted constructors. They are based on the google3
internal rules which do not exactly match our own c++11 rules, and
may change slightly more over time.

R=agable@chromium.org
==========

to

==========
Remove build/c++11 from the set of linter rules ever used.

The checks are not reliable for Rvalue references, and only are
allowing default/deleted constructors. They are based on the google3
internal rules which do not exactly match our own c++11 rules, and
may change slightly more over time.

R=agable@chromium.org
==========

[danakj, 2016-01-09 00:05:39]

Description was changed from

==========
Remove build/c++11 from the set of linter rules ever used.

The checks are not reliable for Rvalue references, and only are
allowing default/deleted constructors. They are based on the google3
internal rules which do not exactly match our own c++11 rules, and
may change slightly more over time.

R=agable@chromium.org
==========

to

==========
Remove build/c++11 from the set of linter rules ever used.

The checks are not reliable for Rvalue references, and only are
allowing default/deleted constructors. They are based on the google3
internal rules which do not exactly match our own c++11 rules, and
may diverge more over time.

R=agable@chromium.org
==========

[Nico, 2016-01-10 01:21:53]

thakis@chromium.org changed reviewers:
+ thakis@chromium.org

[Nico, 2016-01-10 01:21:54]

non-owner lgtm, thanks!

Might want to add a link to the email discussion about this to the CL
description.

[agable, 2016-01-11 17:56:59]

owner LGTM!

[danakj, 2016-01-11 19:33:33]

Description was changed from

==========
Remove build/c++11 from the set of linter rules ever used.

The checks are not reliable for Rvalue references, and only are
allowing default/deleted constructors. They are based on the google3
internal rules which do not exactly match our own c++11 rules, and
may diverge more over time.

R=agable@chromium.org
==========

to

==========
Remove build/c++11 from the set of linter rules ever used.

The checks are not reliable for Rvalue references, and only are
allowing default/deleted constructors. They are based on the google3
internal rules which do not exactly match our own c++11 rules, and
may diverge more over time.

Email thread that led to this change here:
https://groups.google.com/a/chromium.org/forum/#!topic/cxx/9TZvt8vyeEM

R=agable@chromium.org
==========

[danakj, 2016-01-11 19:33:44]

On 2016/01/10 01:21:54, Nico wrote:
> non-owner lgtm, thanks!
> 
> Might want to add a link to the email discussion about this to the CL
> description.

Good idea, done!

[danakj, 2016-01-11 19:33:47]

The CQ bit was checked by danakj@chromium.org

[commit-bot: I haz the power, 2016-01-11 19:33:48]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1573663003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1573663003/20001

[commit-bot: I haz the power, 2016-01-11 19:37:15]

Description was changed from

==========
Remove build/c++11 from the set of linter rules ever used.

The checks are not reliable for Rvalue references, and only are
allowing default/deleted constructors. They are based on the google3
internal rules which do not exactly match our own c++11 rules, and
may diverge more over time.

Email thread that led to this change here:
https://groups.google.com/a/chromium.org/forum/#!topic/cxx/9TZvt8vyeEM

R=agable@chromium.org
==========

to

==========
Remove build/c++11 from the set of linter rules ever used.

The checks are not reliable for Rvalue references, and only are
allowing default/deleted constructors. They are based on the google3
internal rules which do not exactly match our own c++11 rules, and
may diverge more over time.

Email thread that led to this change here:
https://groups.google.com/a/chromium.org/forum/#!topic/cxx/9TZvt8vyeEM

R=agable@chromium.org

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

[commit-bot: I haz the power, 2016-01-11 19:37:15]

Committed patchset #2 (id:20001) as
http://src.chromium.org/viewvc/chrome?view=rev&revision=298202