Fabric: Disabling custom ("better") containers in DEBUG mode

Summary:
Custom containers are only enabled in release mode.
Using custom stuff complicates debugging process because it breaks embedded into IDE introspections mechanisms.

Reviewed By: mdvacca

Differential Revision: D14508299

fbshipit-source-id: d2dbe87764b17d75ccd544c0a4baf03dd9e0cd0b

Author: shergin

ReactCommon/better/better.h

@@ -49,9 +49,14 @@ namespace better {
 
 /*
  * Enables using Folly containers instead of standard ones (such as map, vector,
- * string, optional and etc.)
+ * small_vector, optional and etc.)
+ * Custom containers are only enabled in release mode. Using custom stuff
+ * complicates debugging process because it breaks embedded into IDE
+ * introspections mechanisms.
  */
+#ifndef DEBUG
 #define BETTER_USE_FOLLY_CONTAINERS
+#endif
 
 } // namespace better
 } // namespace facebook

ReactCommon/better/optional.h

@@ -9,7 +9,7 @@
 
 #include <better/better.h>
 
-#ifdef BETTER_USE_FOLLY_CONTAINERS
+#if defined(BETTER_USE_FOLLY_CONTAINERS) || __cplusplus < 202000L
 
 #include <folly/Optional.h>
 
@@ -22,7 +22,7 @@
 namespace facebook {
 namespace better {
 
-#ifdef BETTER_USE_FOLLY_CONTAINERS
+#if defined(BETTER_USE_FOLLY_CONTAINERS) || __cplusplus < 202000L
 
 template <typename Value>
 using optional = folly::Optional<Value>;