pydrake: Commmit selected clang-format results

All of these changes were suggested by clang-format, though some other
suggested changes within these same files have been declined.

In a future commit, we will finish up the re-formatting work and enable
the clang-format idempotency linter.

bindings/bazel_workaround_4594_libdrake_py.cc

@@ -4,8 +4,9 @@ namespace drake {
 namespace pydrake {
 
 PYBIND11_MODULE(bazel_workaround_4594_libdrake, m) {
-  m.doc() = "Consolidated workaround for bazelbuild/bazel#4594 to load "
-            "libdrake.so, to be used by `pydrake/__init__.py`.";
+  m.doc() =
+      "Consolidated workaround for bazelbuild/bazel#4594 to load "
+      "libdrake.so, to be used by `pydrake/__init__.py`.";
 }
 
 }  // namespace pydrake

bindings/pydrake/common/cpp_param_pybind.cc

@@ -75,8 +75,7 @@ py::object GetPyParamScalarImpl(const std::type_info& tinfo) {
       // once simpler dependencies are used (or something else is used to
       // justify linking in `libdrake.so`).
       const std::string name = tinfo.name();
-      throw std::runtime_error(
-          "C++ type is not registered in pybind: " + name);
+      throw std::runtime_error("C++ type is not registered in pybind: " + name);
     }
     py::handle h(reinterpret_cast<PyObject*>(info->type));
     return py::reinterpret_borrow<py::object>(h);

bindings/pydrake/common/cpp_param_pybind.h

@@ -43,7 +43,7 @@ inline py::object GetPyParamScalarImpl(
 /// @returns Python tuple of canonical parameters.
 /// @throws std::runtime_error on the first type it encounters that is neither
 /// aliased nor registered in `pybind11`.
-template <typename ... Ts>
+template <typename... Ts>
 inline py::tuple GetPyParam(type_pack<Ts...> = {}) {
   return py::make_tuple(internal::GetPyParamScalarImpl(type_pack<Ts>{})...);
 }

bindings/pydrake/common/cpp_template_pybind.h

@@ -39,18 +39,15 @@ inline void AddInstantiation(
 // Gets name for a given instantiation.
 inline std::string GetInstantiationName(
     py::handle py_template, py::tuple param) {
-  return py::cast<std::string>(
-    py_template.attr("_instantiation_name")(param));
+  return py::cast<std::string>(py_template.attr("_instantiation_name")(param));
 }
 
 }  // namespace internal
 
-
 /// Provides a temporary, unique name for a class instantiation that
 /// will be passed to `AddTemplateClass`.
 template <typename T>
-std::string TemporaryClassName(
-    const std::string& name = "TemporaryName") {
+std::string TemporaryClassName(const std::string& name = "TemporaryName") {
   return "_" + name + "_" + typeid(T).name();
 }
 
@@ -100,8 +97,7 @@ py::class_<Class, Options...> DefineTemplateClassWithDefault(
 /// @param param Parameters for the instantiation.
 template <typename Func>
 py::object AddTemplateFunction(
-    py::handle scope, const std::string& name, Func&& func,
-    py::tuple param) {
+    py::handle scope, const std::string& name, Func&& func, py::tuple param) {
   // TODO(eric.cousineau): Use `py::sibling` if overloads are needed.
   py::object py_template =
       internal::GetOrInitTemplate(scope, name, "TemplateFunction");
@@ -121,9 +117,8 @@ template <typename Method>
 py::object AddTemplateMethod(
     py::handle scope, const std::string& name, Method&& method,
     py::tuple param) {
-  py::object py_template =
-      internal::GetOrInitTemplate(
-          scope, name, "TemplateMethod", py::make_tuple(scope));
+  py::object py_template = internal::GetOrInitTemplate(
+      scope, name, "TemplateMethod", py::make_tuple(scope));
   py::object py_func = py::cpp_function(
       std::forward<Method>(method),
       py::name(internal::GetInstantiationName(py_template, param).c_str()),

bindings/pydrake/common/deprecation_pybind.h

@@ -13,8 +13,7 @@ namespace pydrake {
 /// Deprecates an attribute `name` of a class `cls`.
 /// This *only* works with class attributes (unbound members or methods) as it
 /// is implemented with a Python property descriptor.
-inline void DeprecateAttribute(
-    py::object cls, py::str name, py::str message) {
+inline void DeprecateAttribute(py::object cls, py::str name, py::str message) {
   py::object deprecated =
       py::module::import("pydrake.common.deprecation").attr("deprecated");
   py::object original = cls.attr(name);

bindings/pydrake/common/drake_optional_pybind.h

@@ -15,12 +15,12 @@ namespace detail {
 // @see pybind11/stl.h, `optional_caster`.
 
 template <typename T>
-struct type_caster<stx::optional<T>> :
-    public optional_caster<stx::optional<T>> {};
+struct type_caster<stx::optional<T>>
+    : public optional_caster<stx::optional<T>> {};
 
-template<>
-struct type_caster<stx::nullopt_t> :
-    public void_caster<stx::nullopt_t> {};
+template <>
+struct type_caster<stx::nullopt_t>
+    : public void_caster<stx::nullopt_t> {};
 
 }  // namespace detail
 }  // namespace pybind11

bindings/pydrake/common/drake_variant_pybind.h

@@ -11,8 +11,8 @@ namespace detail {
 // @see pybind11/stl.h, `variant_caster`.
 
 template <typename... Types>
-struct type_caster<stx::variant<Types...>> :
-    public variant_caster<stx::variant<Types...>> {};
+struct type_caster<stx::variant<Types...>>
+    : public variant_caster<stx::variant<Types...>> {};
 
 }  // namespace detail
 }  // namespace pybind11