emkornfield commented on a change in pull request #12661:
URL: https://github.com/apache/arrow/pull/12661#discussion_r829708009
##########
File path: docs/source/developers/cpp/development.rst
##########
@@ -85,6 +85,8 @@ This project follows `Google's C++ Style Guide
conveniences. For private headers the rules are relaxed further and structs
can be used where convenient for types that do not need access control even
though they may not be simple data containers.
+* We prefer pointers for non-optional output and input/output parameters (the
+ style guide recommends mutable references).
Review comment:
and by associative I meant distributive I think :) I think the wording
could be clarified
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
