kudu-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From t...@apache.org
Subject [1/2] kudu git commit: KUDU-1303 Document code style guidelines for C++11 move semantics and rvalue references
Date Fri, 05 Aug 2016 04:52:20 GMT
Repository: kudu
Updated Branches:
  refs/heads/master 993a686ee -> cf009d431


KUDU-1303 Document code style guidelines for C++11 move semantics and rvalue references

Change-Id: Iad1580a0fdb46c25af3af4fa61e250a0904b376a
Reviewed-on: http://gerrit.cloudera.org:8080/3639
Tested-by: Kudu Jenkins
Reviewed-by: Todd Lipcon <todd@apache.org>


Project: http://git-wip-us.apache.org/repos/asf/kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/kudu/commit/65b18fa6
Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/65b18fa6
Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/65b18fa6

Branch: refs/heads/master
Commit: 65b18fa6fe27ad531dbdbbaa8f27c445fe11e4ca
Parents: 993a686
Author: Misty Stanley-Jones <misty@apache.org>
Authored: Wed Jul 13 15:20:51 2016 -0700
Committer: Todd Lipcon <todd@apache.org>
Committed: Fri Aug 5 04:49:46 2016 +0000

----------------------------------------------------------------------
 docs/contributing.adoc | 8 ++++++++
 1 file changed, 8 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/65b18fa6/docs/contributing.adoc
----------------------------------------------------------------------
diff --git a/docs/contributing.adoc b/docs/contributing.adoc
index 947b17b..2b6b217 100644
--- a/docs/contributing.adoc
+++ b/docs/contributing.adoc
@@ -146,6 +146,14 @@ In general, Kudu follows the
 link:https://google.github.io/styleguide/cppguide.html[Google {cpp} Style Guide],
 with the following exceptions:
 
+=== Notes on {cpp} 11
+
+Kudu uses {cpp} 11. Check out this handy guide to {cpp} 11 move semantics and rvalue
+references: https://www.chromium.org/rvalue-references
+
+We aim to follow most of the same guidelines, such as, where possible, migrating
+away from `foo.Pass()` in favor of `std::move(foo)`.
+
 === Limitations on `boost` Use
 
 `boost` classes from header-only libraries can be used in cases where a suitable


Mime
View raw message