tree 5a33bf19b0c4bcd2b51c2076ccd6810f3ff0bf95
parent b4023c21958c8d654ed304a1a0eb628317a2d170
author Wes McKinney <wesm+git@apache.org> 1532908620 -0400
committer Wes McKinney <wesm+git@apache.org> 1532908620 -0400

PARQUET-1360: Use conforming API style, variable names in WriteFileMetaData functions

Per comments in PARQUET-1348. I also upgraded clang-format to 6.0 here so pardon the diff noise from that.

To summarize the style points:

* Don't pass `const std:shared_ptr<T>&` if `const T&` will do
* Don't pass `const std::shared_ptr<T>&` if `T*` will do
* I prefer to only use class static function members for alternate constructors
* Out arguments after in arguments
* `lower_with_underscores` for variable names

Author: Wes McKinney <wesm+git@apache.org>

Closes #482 from wesm/PARQUET-1360 and squashes the following commits:

375a442 [Wes McKinney] API conformity and changes per review comments in ARROW-1348