From bdbb40f928ea85a8d2c9b4f3f0dced267ff9d6b5 Mon Sep 17 00:00:00 2001
From: Marcus Holland-Moritz <github@mhxnet.de>
Date: Wed, 22 May 2024 15:31:11 +0200
Subject: [PATCH] refactor(fragment_order_parser): replace folly::gen with
 ranges

---
 src/dwarfs/fragment_order_parser.cpp | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/dwarfs/fragment_order_parser.cpp b/src/dwarfs/fragment_order_parser.cpp
index c94bf7ee..dc4cc6e1 100644
--- a/src/dwarfs/fragment_order_parser.cpp
+++ b/src/dwarfs/fragment_order_parser.cpp
@@ -25,7 +25,9 @@
 
 #include <fmt/format.h>
 
-#include <folly/gen/String.h>
+#include <range/v3/range/conversion.hpp>
+#include <range/v3/view/join.hpp>
+#include <range/v3/view/map.hpp>
 
 #include <dwarfs/fragment_order_parser.h>
 #include <dwarfs/option_map.h>
@@ -45,10 +47,8 @@ const std::map<std::string_view, file_order_mode> order_choices{
 } // namespace
 
 std::string fragment_order_parser::choices() {
-  // TODO: C++23
-  // auto tools = std::views::keys(order_choices) | std::views::join_with(", ");
-  using namespace folly::gen;
-  return from(order_choices) | get<0>() | unsplit<std::string>(", ");
+  return ranges::views::keys(order_choices) | ranges::views::join(", ") |
+         ranges::to<std::string>();
 }
 
 // TODO: find a common syntax for these options so we don't need