From 0d868fdca8a7b0d5e1e9c7fd9d6fdc8c2ad1c9df Mon Sep 17 00:00:00 2001
From: Aayush Anand <aaayushanand5@gmail.com>
Date: Mon, 28 Feb 2022 13:20:08 -0700
Subject: [PATCH] Modify ArgumentParser::parse_args_validate() method to use a
 for loop instead of std::for_each()

Suggested-by: Aayush Anand <aaayushanand5@gmail.com>
[skrobinson: Updated for latest formatting and naming conventions]
Signed-off-by: Sean Robinson <sean.robinson@scottsdalecc.edu>
---
 include/argparse/argparse.hpp | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/include/argparse/argparse.hpp b/include/argparse/argparse.hpp
index 4cb17b8..6c8daf2 100644
--- a/include/argparse/argparse.hpp
+++ b/include/argparse/argparse.hpp
@@ -1139,11 +1139,9 @@ private:
    */
   void parse_args_validate() {
     // Check if all arguments are parsed
-    std::for_each(std::begin(m_argument_map), std::end(m_argument_map),
-                  [](const auto &pair) {
-                    const auto &argument = pair.second;
-                    argument->validate();
-                  });
+    for (const auto& [unused, argument] : m_argument_map) {
+      argument->validate();
+    }
   }
 
   // Used by print_help.