Apply clang-format to folly/gen/ (partial: namespace)
authorYedidya Feldblum <yfeldblum@fb.com>
Sat, 3 Jun 2017 00:51:28 +0000 (17:51 -0700)
committerFacebook Github Bot <facebook-github-bot@users.noreply.github.com>
Sat, 3 Jun 2017 00:53:58 +0000 (17:53 -0700)
Summary:
[Folly] Apply `clang-format` to `folly/gen/` (partial: namespace).

With some manual rearrangement in the places where `clang-format` does awkward things, with the result that clang-format over `folly/gen/` becomes a no-op.

Reviewed By: Orvid

Differential Revision: D5177428

fbshipit-source-id: 17f51d00454a7be7f29dcf1a2ff3d0eaf1de72ab

15 files changed:
folly/gen/Base-inl.h
folly/gen/Base.h
folly/gen/Combine-inl.h
folly/gen/Combine.h
folly/gen/Core-inl.h
folly/gen/Core.h
folly/gen/File-inl.h
folly/gen/File.h
folly/gen/Parallel.h
folly/gen/ParallelMap-inl.h
folly/gen/ParallelMap.h
folly/gen/String-inl.h
folly/gen/String.h
folly/gen/test/BaseTest.cpp
folly/gen/test/StringBenchmark.cpp

index 66d2b92..959696e 100644 (file)
@@ -22,7 +22,8 @@
 #pragma GCC diagnostic push
 #pragma GCC diagnostic ignored "-Wshadow"
 
-namespace folly { namespace gen {
+namespace folly {
+namespace gen {
 
 /**
  * ArgumentReference - For determining ideal argument type to receive a value.
@@ -2336,7 +2337,7 @@ inline detail::Skip skip(size_t count) { return detail::Skip(count); }
 inline detail::Batch batch(size_t batchSize) {
   return detail::Batch(batchSize);
 }
-
-}} // folly::gen
+} // gen
+} // folly
 
 #pragma GCC diagnostic pop
index a5d4b05..cf3eca1 100644 (file)
  *
  * To learn more about this library, including the use of infinite generators,
  * see the examples in the comments, or the docs (coming soon).
-*/
+ */
 
-namespace folly { namespace gen {
+namespace folly {
+namespace gen {
 
 class Less {
 public:
@@ -814,7 +815,7 @@ template<class Fallback,
 UnwrapOr unwrapOr(Fallback&& fallback) {
   return UnwrapOr(std::forward<Fallback>(fallback));
 }
-
-}} // folly::gen
+} // gen
+} // folly
 
 #include <folly/gen/Base-inl.h>
index 58daf07..4e293d1 100644 (file)
@@ -201,7 +201,7 @@ class MergeTuples {
   }
 };
 
-}  // namespace detail
+} // namespace detail
 
 // TODO(mcurtiss): support zip() for N>1 operands. Because of variadic problems,
 // this might not be easily possible until gcc4.8 is available.
@@ -211,5 +211,5 @@ Zip zip(Source&& source) {
   return Zip(std::forward<Source>(source));
 }
 
-}  // namespace gen
-}  // namespace folly
+} // namespace gen
+} // namespace folly
index c665efc..42ca358 100644 (file)
@@ -29,7 +29,7 @@ class Interleave;
 template<class Container>
 class Zip;
 
-}  // namespace detail
+} // namespace detail
 
 template<class Source2,
          class Source2Decayed = typename std::decay<Source2>::type,
@@ -38,7 +38,7 @@ Interleave interleave(Source2&& source2) {
   return Interleave(std::forward<Source2>(source2));
 }
 
-}  // namespace gen
-}  // namespace folly
+} // namespace gen
+} // namespace folly
 
 #include <folly/gen/Combine-inl.h>
index 2868224..87891e3 100644 (file)
@@ -25,7 +25,8 @@
 #pragma GCC diagnostic push
 #pragma GCC diagnostic ignored "-Wshadow"
 
-namespace folly { namespace gen {
+namespace folly {
+namespace gen {
 
 /**
  * IsCompatibleSignature - Trait type for testing whether a given Functor
@@ -372,7 +373,7 @@ public:
 };
 
 } // detail
-
-}} // folly::gen
+} // gen
+} // folly
 
 #pragma GCC diagnostic pop
index 5181355..dfd93d6 100644 (file)
@@ -17,7 +17,8 @@
 #pragma once
 #define FOLLY_GEN_CORE_H_
 
-namespace folly { namespace gen {
+namespace folly {
+namespace gen {
 
 template<class Value, class Self>
 class GenImpl;
@@ -37,7 +38,7 @@ template<class Value, class First, class Second>
 class Chain;
 
 } // detail
-
-}} // folly::gen
+} // gen
+} // folly
 
 #include <folly/gen/Core-inl.h>
index 591ead3..d098dbd 100644 (file)
@@ -129,7 +129,7 @@ inline auto byLineImpl(File file, char delim, bool keepDelimiter)
     | resplit(delim, keepDelimiter);
 }
 
-}  // !detail
+} // !detail
 
 /**
  * Generator which reads lines from a file.
@@ -161,4 +161,5 @@ inline auto byLine(int fd, char delim = '\n')
 
 inline auto byLine(const char* f, char delim = '\n')
   -> decltype(byLine(File(f), delim)) { return byLine(File(f), delim); }
-}}  // !folly::gen
+} // !gen
+} // !folly
index b5dfc4b..4157a07 100644 (file)
@@ -27,7 +27,7 @@ namespace gen {
 namespace detail {
 class FileReader;
 class FileWriter;
-}  // namespace detail
+} // namespace detail
 
 /**
  * Generator that reads from a file with a buffer of the given size.
@@ -64,7 +64,7 @@ template <class S = detail::FileWriter>
 S toFile(File file, std::unique_ptr<IOBuf> buffer) {
   return S(std::move(file), std::move(buffer));
 }
-
-}}  // !folly::gen
+} // !gen
+} // !folly
 
 #include <folly/gen/File-inl.h>
index d4012d1..85e956e 100644 (file)
@@ -21,7 +21,8 @@
 
 #include <folly/gen/Base.h>
 
-namespace folly { namespace gen {
+namespace folly {
+namespace gen {
 namespace detail {
 
 template <class Ops>
@@ -102,7 +103,7 @@ template <class Sink, class Sub = detail::Sub<Sink>>
 Sub sub(Sink sink) {
   return Sub(std::move(sink));
 }
-
-}} // !namespace folly::gen
+} // !namespace gen
+} // !namespace folly
 
 #include <folly/gen/Parallel-inl.h>
index 47fd9bc..7975b21 100644 (file)
@@ -28,7 +28,9 @@
 #include <folly/MPMCPipeline.h>
 #include <folly/experimental/EventCount.h>
 
-namespace folly { namespace gen { namespace detail {
+namespace folly {
+namespace gen {
+namespace detail {
 
 /**
  * PMap - Map in parallel (using threads). For producing a sequence of
@@ -250,5 +252,6 @@ class PMap : public Operator<PMap<Predicate>> {
     return Gen(source.self(), pred_, nThreads_);
   }
 };
-
-}}}  // namespaces
+} // namespace detail
+} // namespace gen
+} // namespace folly
index 4e9da1e..e17acc9 100644 (file)
 
 #include <folly/gen/Core.h>
 
-namespace folly { namespace gen {
+namespace folly {
+namespace gen {
 
 namespace detail {
 
 template<class Predicate>
 class PMap;
 
-}  // namespace detail
+} // namespace detail
 
 /**
  * Run `pred` in parallel in nThreads. Results are returned in the
@@ -42,7 +43,7 @@ template<class Predicate,
   PMap pmap(Predicate pred = Predicate(), size_t nThreads = 0) {
   return PMap(std::move(pred), nThreads);
 }
-
-}}  // namespaces
+} // namespace gen
+} // namespace folly
 
 #include <folly/gen/ParallelMap-inl.h>
index be9f1d5..64e86f9 100644 (file)
@@ -412,7 +412,7 @@ class SplitTo {
 
 #pragma GCC diagnostic pop
 
-}  // namespace detail
+} // namespace detail
 
-}  // namespace gen
-}  // namespace folly
+} // namespace gen
+} // namespace folly
index fde954a..2b8b02c 100644 (file)
@@ -41,7 +41,7 @@ template<class TargetContainer,
          class... Targets>
 class SplitTo;
 
-}  // namespace detail
+} // namespace detail
 
 /**
  * Split the output from a generator into StringPiece "lines" delimited by
@@ -242,7 +242,7 @@ StreamSplitter<Callback> streamSplitter(char delimiter,
   return StreamSplitter<Callback>(delimiter, std::move(pieceCb), capacity);
 }
 
-}  // namespace gen
-}  // namespace folly
+} // namespace gen
+} // namespace folly
 
 #include <folly/gen/String-inl.h>
index 999d74b..3273471 100644 (file)
@@ -918,7 +918,7 @@ class TestIntSeq : public GenImpl<int, TestIntSeq> {
   TestIntSeq& operator=(const TestIntSeq&) = delete;
 };
 
-}  // namespace
+} // namespace
 
 TEST(Gen, NoGeneratorCopies) {
   EXPECT_EQ(15, TestIntSeq() | sum);
index cffe89c..c6de1ed 100644 (file)
@@ -61,7 +61,7 @@ void initStringResplitterBenchmark() {
 
 size_t len(folly::StringPiece s) { return s.size(); }
 
-}  // namespace
+} // namespace
 
 BENCHMARK(StringResplitter_Big, iters) {
   size_t s = 0;