c++11: Add discussion thread links, ban things that don't work with MSVS.

c++11: Add discussion thread links, ban things that don't work with MSVS.

Namely, ban __func__, constexpr, u"asdf", and char16_t.

BUG=none
R=jamesr@chromium.org

Committed: https://chromium.googlesource.com/chromium/src/+/cea20a82a4f0b48ede9543fe4a769ab8e78fb8e2

[Nico, 2014-09-25 01:59:23]

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

[Avi (use Gerrit), 2014-09-25 04:11:49]

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

https://codereview.chromium.org/603043002/diff/20001/styleguide/c++/c++11.htm...
styleguide/c++/c++11.html:124: <p>This section will lists C++11 features that
are not allowed in the Chromium
"This section will *list*", not "lists". Or "This section lists". Pick one.

[Nico, 2014-09-25 04:13:15]

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

https://codereview.chromium.org/603043002/diff/20001/styleguide/c++/c++11.htm...
styleguide/c++/c++11.html:124: <p>This section will lists C++11 features that
are not allowed in the Chromium
On 2014/09/25 04:11:49, Avi wrote:
> "This section will *list*", not "lists". Or "This section lists". Pick one.

porque no los dos? :-(

Done.

[jamesr, 2014-09-25 06:00:21]

On 2014/09/25 04:13:15, Nico (hiding) wrote:
> https://codereview.chromium.org/603043002/diff/20001/styleguide/c++/c++11.html
> File styleguide/c++/c++11.html (right):
> 
>
https://codereview.chromium.org/603043002/diff/20001/styleguide/c++/c++11.htm...
> styleguide/c++/c++11.html:124: <p>This section will lists C++11 features that
> are not allowed in the Chromium
> On 2014/09/25 04:11:49, Avi wrote:
> > "This section will *list*", not "lists". Or "This section lists". Pick one.
> 
> porque no los dos? :-(

por que no los dos (por que = why, porque = because)

> 
> Done.

[jamesr, 2014-09-25 06:01:52]

lgtm

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

https://codereview.chromium.org/603043002/diff/40001/styleguide/c++/c++11.htm...
styleguide/c++/c++11.html:143: <td>TODO: documentation link</td>
could we link to a crbug for these things? that way folks wanting to add
information can do so on the bug and check the status of the feature in preview
builds, etc

[Nico, 2014-09-25 06:02:13]

No hablo espanol. Yo soy un pollo.
On Sep 24, 2014 11:00 PM, <jamesr@chromium.org> wrote:

> On 2014/09/25 04:13:15, Nico (hiding) wrote:
>
>> https://codereview.chromium.org/603043002/diff/20001/
>> styleguide/c++/c++11.html
>> File styleguide/c++/c++11.html (right):
>>
>
>
> https://codereview.chromium.org/603043002/diff/20001/
> styleguide/c++/c++11.html#newcode124
>
>> styleguide/c++/c++11.html:124: <p>This section will lists C++11 features
>> that
>> are not allowed in the Chromium
>> On 2014/09/25 04:11:49, Avi wrote:
>> > "This section will *list*", not "lists". Or "This section lists". Pick
>> one.
>>
>
>  porque no los dos? :-(
>>
>
> por que no los dos (por que = why, porque = because)
>
>
>  Done.
>>
>
>
>
> https://codereview.chromium.org/603043002/
>

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, 2014-09-25 17:48:32]

Patchset 3 (id:??) landed as
https://crrev.com/cea20a82a4f0b48ede9543fe4a769ab8e78fb8e2
Cr-Commit-Position: refs/heads/master@{#296739}

[Nico, 2014-09-25 17:48:35]

Committed patchset #3 (id:40001) manually as cea20a8 (presubmit successful).