Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(147)

Issue 589413003: Add a document describing which C++11 features we allow. (Closed)

Created:
6 years, 3 months ago by Nico
Modified:
6 years, 3 months ago
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Project:
chromium
Visibility:
Public.

Description

Add a document describing which C++11 features we allow. For now, this allows only a very limited set. Also describes the process we're going to follow to allow more features. The idea is that we're gradually going to allow more features over time. Also included is the source of a tiny appengine app running at chromium-cpp.appspot.com that will show the contents of the document in this CL once it's submitted. BUG=360096 NOTRY=true R=ben@chromium.org, inferno@chromium.org, jamesr@chromium.org Committed: https://chromium.googlesource.com/chromium/src/+/019d40fa03d9135e7db9d04c2c132af5a05e3dc2

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1775 lines, -4 lines) Patch
A styleguide/c++/OWNERS View 1 chunk +3 lines, -0 lines 0 comments Download
A styleguide/c++/c++11.css View 1 chunk +742 lines, -0 lines 0 comments Download
A styleguide/c++/c++11.html View 1 chunk +963 lines, -0 lines 0 comments Download
A styleguide/c++/chromium-cpp/app.yaml View 1 chunk +17 lines, -0 lines 0 comments Download
A styleguide/c++/chromium-cpp/favicon.ico View Binary file 0 comments Download
A + styleguide/c++/chromium-cpp/index.yaml View 2 chunks +1 line, -4 lines 0 comments Download
A styleguide/c++/chromium-cpp/main.py View 1 chunk +49 lines, -0 lines 0 comments Download

Messages

Total messages: 15 (5 generated)
Nico
jamesr, ajwong: Please look at c++11.html tsepez: Can you give the appenine main.py a short, ...
6 years, 3 months ago (2014-09-23 18:49:17 UTC) #2
Tom Sepez
> tsepez: Can you give the appenine main.py a short, security-conscious look? Let's bounce this ...
6 years, 3 months ago (2014-09-23 18:55:48 UTC) #4
awong
C++ lgtm On 23 Sep 2014 11:55, <tsepez@chromium.org> wrote: > tsepez: Can you give the ...
6 years, 3 months ago (2014-09-23 19:05:35 UTC) #5
aarya
On 2014/09/23 18:55:48, Tom Sepez wrote: > > tsepez: Can you give the appenine main.py ...
6 years, 3 months ago (2014-09-23 19:07:35 UTC) #6
inferno
On 2014/09/23 19:07:35, aarya wrote: > On 2014/09/23 18:55:48, Tom Sepez wrote: > > > ...
6 years, 3 months ago (2014-09-23 19:08:06 UTC) #7
Nico
darin: Can I get toplevel OWNERS lgtm for the new directory, please?
6 years, 3 months ago (2014-09-23 19:37:58 UTC) #9
jamesr
lgtm
6 years, 3 months ago (2014-09-23 20:05:39 UTC) #10
Ben Goodger (Google)
lgtm
6 years, 3 months ago (2014-09-23 20:17:55 UTC) #12
commit-bot: I haz the power
Patchset 1 (id:??) landed as https://crrev.com/019d40fa03d9135e7db9d04c2c132af5a05e3dc2 Cr-Commit-Position: refs/heads/master@{#296214}
6 years, 3 months ago (2014-09-23 20:23:07 UTC) #14
Nico
6 years, 3 months ago (2014-09-23 20:24:00 UTC) #15
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as 019d40f (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698