Merge pull request #4146 from eggrobin/sourcery

Sourcery for functions

functions/accurate_table_generator_test.cpp

@@ -15,7 +15,7 @@
 #include "mathematica/logger.hpp"
 #include "mathematica/mathematica.hpp"
 #include "numerics/combinatorics.hpp"
-#include "quantities/numbers.hpp"
+#include "quantities/numbers.hpp"  // 🧙 For π.
 #include "testing_utilities/approximate_quantity.hpp"
 #include "testing_utilities/is_near.hpp"
 #include "testing_utilities/matchers.hpp"

functions/core_math_accuracy_test.cpp

@@ -8,15 +8,15 @@
 #include "glog/logging.h"
 #include "gmock/gmock.h"
 #include "gtest/gtest.h"
-#include "quantities/numbers.hpp"
+#include "quantities/numbers.hpp"  // 🧙 For π.
 #include "testing_utilities/approximate_quantity.hpp"
 #include "testing_utilities/is_near.hpp"
 
 namespace principia {
 namespace functions {
-namespace _multiprecision {
 
 using namespace boost::multiprecision;
+using namespace principia::functions::_multiprecision;
 using namespace principia::testing_utilities::_approximate_quantity;
 using namespace principia::testing_utilities::_is_near;
 
@@ -74,6 +74,5 @@ TEST_F(CoreMathAccuracyTest, SinCos) {
 
 #endif
 
-}  // namespace _multiprecision
 }  // namespace functions
 }  // namespace principia

functions/sin_cos_test.cpp

@@ -9,16 +9,22 @@
 #include "glog/logging.h"
 #include "gtest/gtest.h"
 #include "numerics/next.hpp"
-#include "quantities/numbers.hpp"
+#include "quantities/numbers.hpp"  // 🧙 For π.
 #include "testing_utilities/almost_equals.hpp"
 
 // This test lives in `functions` to avoid pulling `boost` into `numerics`.
+// It uses neither the `functions` nor the `numerics` namespace so that the Sin
+// and Cos from both (`principia::functions::_multiprecision` and
+// `principia::numerics::_sin_cos`) are made visible by the using directives
+// below.
+
 namespace principia {
-namespace numerics {
-namespace _sin_cos {
+namespace functions_test {
 
 using namespace boost::multiprecision;
+using namespace principia::functions::_multiprecision;
 using namespace principia::numerics::_next;
+using namespace principia::numerics::_sin_cos;
 using namespace principia::testing_utilities::_almost_equals;
 
 class SinCosTest : public ::testing::Test {};
@@ -44,8 +50,7 @@ TEST_F(SinCosTest, Random) {
     double const principia_argument = uniformly_at(random);
     auto const boost_argument = cpp_rational(principia_argument);
     {
-      auto const boost_sin =
-          functions::_multiprecision::Sin(boost_argument);
+      auto const boost_sin = Sin(boost_argument);
       double const principia_sin = Sin(principia_argument);
       auto const sin_error =
           abs(boost_sin - static_cast<cpp_bin_float_50>(principia_sin));
@@ -61,8 +66,7 @@ TEST_F(SinCosTest, Random) {
       }
     }
     {
-      auto const boost_cos =
-          functions::_multiprecision::Cos(boost_argument);
+      auto const boost_cos = Cos(boost_argument);
       double const principia_cos = Cos(principia_argument);
       auto const cos_error =
           abs(boost_cos - static_cast<cpp_bin_float_50>(principia_cos));
@@ -143,6 +147,5 @@ TEST_F(SinCosTest, HardReduction) {
               AlmostEquals(-4.687165924254627611122582801963884e-19, 0));
 }
 
-}  // namespace _sin_cos
-}  // namespace numerics
+}  // namespace functions_test
 }  // namespace principia

functions/std_accuracy_test.cpp

@@ -15,10 +15,10 @@
 
 namespace principia {
 namespace functions {
-namespace _multiprecision {
 
 using ::testing::AnyOf;
 using namespace boost::multiprecision;
+using namespace principia::functions::_multiprecision;
 using namespace principia::quantities::_si;
 using namespace principia::testing_utilities::_almost_equals;
 using namespace principia::testing_utilities::_approximate_quantity;
@@ -85,6 +85,5 @@ TEST_F(StdAccuracyTest, SinCos) {
 
 #endif
 
-}  // namespace _multiprecision
 }  // namespace functions
 }  // namespace principia

geometry/serialization_body.hpp

@@ -6,6 +6,7 @@
 #include "geometry/grassmann.hpp"
 #include "geometry/point.hpp"
 #include "quantities/quantities.hpp"
+#include "quantities/serialization.hpp"
 
 namespace principia {
 namespace geometry {

include_what_you_using_all_the_things.ps1

@@ -4,13 +4,14 @@
     --project:astronomy `
     --project:base `
     --project:benchmarks `
+    --project:functions `
     --project:geometry `
     --project:integrators `
     --project:journal `
-    --project:nanobenchmarks `
     --project:ksp_plugin `
     --project:ksp_plugin_test `
     --project:mathematica `
+    --project:nanobenchmarks `
     --project:numerics `
     --project:physics `
     --project:quantities `
@@ -24,4 +25,5 @@
     --special_own_header:ksp_plugin/interface_*.cpp=ksp_plugin/interface.hpp `
     --special_own_header:ksp_plugin_test/interface_*.cpp=ksp_plugin/interface.hpp `
     --special_own_header:ksp_plugin_test/benchmark.cpp=ksp_plugin/plugin.hpp `
-    --special_own_header:ksp_plugin_test/plugin*_test.cpp=ksp_plugin/plugin.hpp
+    --special_own_header:ksp_plugin_test/plugin*_test.cpp=ksp_plugin/plugin.hpp `
+    --special_own_header:functions/sin_cos_test.cpp=numerics/sin_cos.hpp

sourcerer/include_what_you_using.cs

@@ -186,7 +186,10 @@ private static void FixIncludes(Parser.File file,
     foreach (string ns in using_namespaces) {
       var segments = ns.Split("::", StringSplitOptions.RemoveEmptyEntries);
       var include_path = Array.Empty<string>();
-      bool skip = false;
+      if (ns == file.file_namespace_full_name) {
+        // Don't add an include for our own header.  This matters for tests.
+        continue;
+      }
       foreach (string segment in segments) {
         if (segment == "principia") {
           continue;
@@ -195,24 +198,17 @@ private static void FixIncludes(Parser.File file,
                    segment == "std") {
           // We have using directives for namespaces in absl or std, don't emit
           // an include for them.
-          skip = true;
-          continue;
+          goto next_using_directive;
         } else if (segment[0] == '_') {
-          // Don't add an include for our own header.  This matters for tests.
-          if (segment == file.file_namespace_simple_name) {
-            skip = true;
-          } else {
-            string header_filename = Regex.Replace(segment, @"^_", "");
-            include_path = include_path.Append(header_filename).ToArray();
-          }
+          string header_filename = Regex.Replace(segment, @"^_", "");
+          include_path = include_path.Append(header_filename).ToArray();
           break;
         } else {
           include_path = include_path.Append(segment).ToArray();
         }
       }
-      if (!skip) {
-        new_include_paths.Add(include_path);
-      }
+      new_include_paths.Add(include_path);
+      next_using_directive: {}
     }
 
     // Extract the includes.