Merge pull request #93 from cekc/nicer-message-for-required

nicer usage text for required arg
p-ranav · Dec 15, 2020 · 9903a22 · 9903a22
2 parents 68fb9f1 + ed06438
commit 9903a22
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/include/argparse/argparse.hpp b/include/argparse/argparse.hpp
@@ -529,12 +529,14 @@ class Argument {
     std::copy(std::begin(argument.mNames), std::end(argument.mNames),
               std::ostream_iterator<std::string>(nameStream, " "));
     stream << nameStream.str() << "\t" << argument.mHelp;
-    if (argument.mIsRequired && !argument.mDefaultValue.has_value())
-      stream << "[Required]";
     if (argument.mDefaultValue.has_value()) {
       if (!argument.mHelp.empty())
         stream << " ";
       stream << "[default: " << argument.mDefaultValueRepr << "]";
+    } else if (argument.mIsRequired) {
+      if (!argument.mHelp.empty())
+        stream << " ";
+      stream << "[required]";
     }
     stream << "\n";
     return stream;