Fix CQS signal modernize-concat-nested-namespaces in xplat/js/react-native-github/packages/react-native/ReactCommon/cxxreact

Reviewed By: javache

Differential Revision: D77014011

packages/react-native/ReactCommon/cxxreact/CxxModule.h

@@ -20,9 +20,7 @@ class Instance;
 
 } // namespace facebook::react
 
-namespace facebook {
-namespace xplat {
-namespace module {
+namespace facebook::xplat::module {
 
 /**
  * Base class for Catalyst native modules whose implementations are
@@ -249,6 +247,4 @@ class CxxModule {
   std::weak_ptr<react::Instance> instance_;
 };
 
-} // namespace module
-} // namespace xplat
-} // namespace facebook
+} // namespace facebook::xplat::module

packages/react-native/ReactCommon/cxxreact/JsArgumentHelpers.h

@@ -20,8 +20,7 @@
 // jsArgAs... methods at the end simple to use should be most common, but any
 // non-detail method can be used when needed.
 
-namespace facebook {
-namespace xplat {
+namespace facebook::xplat {
 
 class JsArgumentException : public std::logic_error {
  public:
@@ -118,7 +117,6 @@ inline std::string jsArgAsString(const folly::dynamic& args, size_t n) {
   return jsArgN(args, n, &folly::dynamic::asString);
 }
 
-} // namespace xplat
-} // namespace facebook
+} // namespace facebook::xplat
 
 #include <cxxreact/JsArgumentHelpers-inl.h>

packages/react-native/ReactCommon/cxxreact/ReactMarker.cpp

@@ -8,8 +8,7 @@
 #include "ReactMarker.h"
 #include <cxxreact/JSExecutor.h>
 
-namespace facebook::react {
-namespace ReactMarker {
+namespace facebook::react::ReactMarker {
 
 #if __clang__
 #pragma clang diagnostic push
@@ -138,5 +137,4 @@ double StartupLogger::getAppStartupEndTime() {
   return appStartupEndTime;
 }
 
-} // namespace ReactMarker
-} // namespace facebook::react
+} // namespace facebook::react::ReactMarker

packages/react-native/ReactCommon/cxxreact/ReactMarker.h

@@ -14,8 +14,7 @@
 #include <functional>
 #endif
 
-namespace facebook::react {
-namespace ReactMarker {
+namespace facebook::react::ReactMarker {
 
 enum ReactMarkerId {
   APP_STARTUP_START,
@@ -106,5 +105,4 @@ extern RN_EXPORT void logMarkerDone(
     const ReactMarkerId markerId,
     double markerTime);
 
-} // namespace ReactMarker
-} // namespace facebook::react
+} // namespace facebook::react::ReactMarker

packages/react-native/ReactCommon/cxxreact/SampleCxxModule.cpp

@@ -15,9 +15,7 @@
 
 using namespace folly;
 
-namespace facebook {
-namespace xplat {
-namespace samples {
+namespace facebook::xplat::samples {
 
 std::string Sample::hello() {
   LOG(WARNING) << "glog: hello, world";
@@ -171,9 +169,7 @@ void SampleCxxModule::load([[maybe_unused]] folly::dynamic args, Callback cb) {
   cb({d});
 }
 
-} // namespace samples
-} // namespace xplat
-} // namespace facebook
+} // namespace facebook::xplat::samples
 
 // By convention, the function name should be the same as the class name.
 facebook::xplat::module::CxxModule* SampleCxxModule() {

packages/react-native/ReactCommon/cxxreact/SampleCxxModule.h

@@ -12,9 +12,7 @@
 
 #include <cxxreact/CxxModule.h>
 
-namespace facebook {
-namespace xplat {
-namespace samples {
+namespace facebook::xplat::samples {
 
 // In a less contrived example, Sample would be part of a traditional
 // C++ library.
@@ -52,8 +50,6 @@ class SampleCxxModule : public module::CxxModule {
   std::unique_ptr<Sample> sample_;
 };
 
-} // namespace samples
-} // namespace xplat
-} // namespace facebook
+} // namespace facebook::xplat::samples
 
 extern "C" facebook::xplat::module::CxxModule* SampleCxxModule();

packages/react-native/ReactCommon/cxxreact/SharedProxyCxxModule.h

@@ -11,9 +11,7 @@
 
 #include <cxxreact/CxxModule.h>
 
-namespace facebook {
-namespace xplat {
-namespace module {
+namespace facebook::xplat::module {
 
 // Allows a Cxx-module to be shared or reused across multiple React instances
 // Caveat: the setInstance call is not forwarded, so usages of getInstance
@@ -39,6 +37,4 @@ class SharedProxyCxxModule : public CxxModule {
   std::shared_ptr<CxxModule> shared_;
 };
 
-} // namespace module
-} // namespace xplat
-} // namespace facebook
+} // namespace facebook::xplat::module