Add Args::erase_last method

diff --git a/src/Args.cpp b/src/Args.cpp
index 92a1e41ba1cf249aefebe8ad7ed015fdeca09188..5270989d7dadf80ce94b01e867db432f0c092143 100644 (file)
--- a/src/Args.cpp
+++ b/src/Args.cpp
@@ -155,6 +155,15 @@ Args::to_string() const
   return result;
 }
 
+void
+Args::erase_last(string_view arg)
+{
+  const auto it = std::find(m_args.rbegin(), m_args.rend(), arg);
+  if (it != m_args.rend()) {
+    m_args.erase(std::next(it).base());
+  }
+}
+
 void
 Args::erase_with_prefix(string_view prefix)
 {

diff --git a/src/Args.hpp b/src/Args.hpp
index be917ec75dd4dc735d51455ff23dbaa1400550aa..ae428096a56b3643668fd57102c9e8ac8d184f16 100644 (file)
--- a/src/Args.hpp
+++ b/src/Args.hpp
@@ -60,6 +60,9 @@ public:
   // in arguments is performed.
   std::string to_string() const;
 
+  // Remove last argument equal to `arg`, if any.
+  void erase_last(nonstd::string_view arg);
+
   // Remove all arguments with prefix `prefix`.
   void erase_with_prefix(nonstd::string_view prefix);
 

diff --git a/unittest/test_Args.cpp b/unittest/test_Args.cpp
index 8c2b9cd94a82e91226a570b13aec8b8cc496a5c9..7d21b3dc48f4b34d81ba568aa2746320e46ef2b8 100644 (file)
--- a/unittest/test_Args.cpp
+++ b/unittest/test_Args.cpp
@@ -226,6 +226,23 @@ TEST_CASE("Args operations")
   Args more_args = Args::from_string("x y");
   Args no_args;
 
+  SUBCASE("erase_last")
+  {
+    Args repeated_args = Args::from_string("one two twotwo one two twotwo");
+
+    repeated_args.erase_last("three");
+    CHECK(repeated_args == Args::from_string("one two twotwo one two twotwo"));
+
+    repeated_args.erase_last("two");
+    CHECK(repeated_args == Args::from_string("one two twotwo one twotwo"));
+
+    repeated_args.erase_last("two");
+    CHECK(repeated_args == Args::from_string("one twotwo one twotwo"));
+
+    repeated_args.erase_last("two");
+    CHECK(repeated_args == Args::from_string("one twotwo one twotwo"));
+  }
+
   SUBCASE("erase_with_prefix")
   {
     args.erase_with_prefix("m");