styleguide: ban inherited constructors

styleguide: ban inherited constructors

Committed: https://crrev.com/33bafda741776ba2595c4adb56a4851a51156339
Cr-Commit-Position: refs/heads/master@{#302188}

[mdempsky, 2014-10-30 21:05:32]

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

[Nico, 2014-10-30 22:11:04]

lgtm, thanks!

https://codereview.chromium.org/691803004/diff/1/styleguide/c++/c++11.html
File styleguide/c++/c++11.html (right):

https://codereview.chromium.org/691803004/diff/1/styleguide/c++/c++11.html#ne...
styleguide/c++/c++11.html:200: <p>This section will list C++11 features that are
not allowed in the Chromium
nit: If you're close to the checkout housing this CL, maybe s/will list/lists/
here while you're here. If you're on your phone, ignore this and just click cq+

https://codereview.chromium.org/691803004/diff/1/styleguide/c++/c++11.html#ne...
styleguide/c++/c++11.html:251: <td><a
href="http://en.cppreference.com/w/cpp/language/using_declaration">Using-declaration</a></td>
(it's kinda hidden on that page, i wonder if there's a better link)

[mdempsky_google, 2014-10-30 22:19:10]

mdempsky@google.com changed reviewers:
+ mdempsky@google.com

[mdempsky_google, 2014-10-30 22:19:11]

https://codereview.chromium.org/691803004/diff/1/styleguide/c++/c++11.html
File styleguide/c++/c++11.html (right):

https://codereview.chromium.org/691803004/diff/1/styleguide/c++/c++11.html#ne...
styleguide/c++/c++11.html:200: <p>This section will list C++11 features that are
not allowed in the Chromium
On 2014/10/30 22:11:04, Nico wrote:
> nit: If you're close to the checkout housing this CL, maybe s/will list/lists/
> here while you're here. If you're on your phone, ignore this and just click
cq+

Done.

https://codereview.chromium.org/691803004/diff/1/styleguide/c++/c++11.html#ne...
styleguide/c++/c++11.html:251: <td><a
href="http://en.cppreference.com/w/cpp/language/using_declaration">Using-declaration</a></td>
On 2014/10/30 22:11:04, Nico wrote:
> (it's kinda hidden on that page, i wonder if there's a better link)

Yeah, I was hoping there would be an anchor within the page that I could link
to, but no such luck.

[mdempsky, 2014-10-30 22:19:22]

The CQ bit was checked by mdempsky@chromium.org

[commit-bot: I haz the power, 2014-10-30 22:20:14]

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

[commit-bot: I haz the power, 2014-10-30 23:15:02]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-10-30 23:15:43]

Patchset 2 (id:??) landed as
https://crrev.com/33bafda741776ba2595c4adb56a4851a51156339
Cr-Commit-Position: refs/heads/master@{#302188}