Merge pull request #290 from Arthapz/master

Add C++20 module

.stylua.toml

@@ -0,0 +1 @@
+indent_type = "Spaces"

include/argparse/argparse.hpp

@@ -29,6 +29,8 @@ OUT OF OR IN CONNECTION WITH THE SOFTWARE  OR THE USE OR OTHER DEALINGS IN THE
 SOFTWARE.
 */
 #pragma once
+
+#ifndef ARGPARSE_MODULE_USE_STD_MODULE
 #include <algorithm>
 #include <any>
 #include <array>
@@ -53,6 +55,7 @@ SOFTWARE.
 #include <utility>
 #include <variant>
 #include <vector>
+#endif
 
 namespace argparse {
 
@@ -72,7 +75,7 @@ struct HasContainerTraits<
                    decltype(std::declval<T>().size())>> : std::true_type {};
 
 template <typename T>
-static constexpr bool IsContainer = HasContainerTraits<T>::value;
+inline constexpr bool IsContainer = HasContainerTraits<T>::value;
 
 template <typename T, typename = void>
 struct HasStreamableTraits : std::false_type {};
@@ -84,7 +87,7 @@ struct HasStreamableTraits<
     : std::true_type {};
 
 template <typename T>
-static constexpr bool IsStreamable = HasStreamableTraits<T>::value;
+inline constexpr bool IsStreamable = HasStreamableTraits<T>::value;
 
 constexpr std::size_t repr_max_container_size = 5;
 

module/argparse.cppm

@@ -0,0 +1,56 @@
+/*
+  __ _ _ __ __ _ _ __   __ _ _ __ ___  ___
+ / _` | '__/ _` | '_ \ / _` | '__/ __|/ _ \ Argument Parser for Modern C++
+| (_| | | | (_| | |_) | (_| | |  \__ \  __/ http://github.com/p-ranav/argparse
+ \__,_|_|  \__, | .__/ \__,_|_|  |___/\___|
+           |___/|_|
+
+Licensed under the MIT License <http://opensource.org/licenses/MIT>.
+SPDX-License-Identifier: MIT
+Copyright (c) 2019-2022 Pranav Srinivas Kumar <pranav.srinivas.kumar@gmail.com>
+and other contributors.
+
+Permission is hereby  granted, free of charge, to any  person obtaining a copy
+of this software and associated  documentation files (the "Software"), to deal
+in the Software  without restriction, including without  limitation the rights
+to  use, copy,  modify, merge,  publish, distribute,  sublicense, and/or  sell
+copies  of  the Software,  and  to  permit persons  to  whom  the Software  is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE  IS PROVIDED "AS  IS", WITHOUT WARRANTY  OF ANY KIND,  EXPRESS OR
+IMPLIED,  INCLUDING BUT  NOT  LIMITED TO  THE  WARRANTIES OF  MERCHANTABILITY,
+FITNESS FOR  A PARTICULAR PURPOSE AND  NONINFRINGEMENT. IN NO EVENT  SHALL THE
+AUTHORS  OR COPYRIGHT  HOLDERS  BE  LIABLE FOR  ANY  CLAIM,  DAMAGES OR  OTHER
+LIABILITY, WHETHER IN AN ACTION OF  CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE  OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.
+*/
+
+module;
+
+#ifndef ARGPARSE_MODULE_USE_STD_MODULE
+#include <argparse/argparse.hpp>
+#endif 
+
+export module argparse;
+
+#ifdef ARGPARSE_MODULE_USE_STD_MODULE
+import std;
+
+extern "C++" {
+#include <argparse/argparse.hpp>
+}
+#endif
+
+
+export namespace argparse {
+    using argparse::nargs_pattern;
+    using argparse::default_arguments;
+    using argparse::operator&;
+    using argparse::Argument;
+    using argparse::ArgumentParser;
+}
+

test/argparse_details.cppm

@@ -0,0 +1,10 @@
+module;
+
+#include <argparse/argparse.hpp>
+
+export module argparse.details;
+
+export namespace argparse::details {
+    using argparse::details::repr;
+}
+

test/test_actions.cpp

@@ -1,4 +1,8 @@
+#ifdef WITH_MODULE
+import argparse;
+#else
 #include <argparse/argparse.hpp>
+#endif
 #include <doctest.hpp>
 
 using doctest::test_suite;

test/test_append.cpp

@@ -1,6 +1,13 @@
+#ifdef WITH_MODULE
+import argparse;
+#else
 #include <argparse/argparse.hpp>
+#endif
 #include <doctest.hpp>
 
+#include <vector>
+#include <string>
+
 using doctest::test_suite;
 
 TEST_CASE("Simplest .append" * test_suite("append")) {

test/test_as_container.cpp

@@ -1,4 +1,8 @@
+#ifdef WITH_MODULE
+import argparse;
+#else
 #include <argparse/argparse.hpp>
+#endif
 #include <doctest.hpp>
 
 using doctest::test_suite;

test/test_bool_operator.cpp

@@ -1,4 +1,9 @@
+#ifdef WITH_MODULE
+import argparse;
+#else
 #include <argparse/argparse.hpp>
+#endif
+
 #include <doctest.hpp>
 
 using doctest::test_suite;

test/test_compound_arguments.cpp

@@ -1,4 +1,8 @@
+#ifdef WITH_MODULE
+import argparse;
+#else
 #include <argparse/argparse.hpp>
+#endif
 #include <doctest.hpp>
 #include <test_utility.hpp>
 

test/test_const_correct.cpp

@@ -1,4 +1,8 @@
+#ifdef WITH_MODULE
+import argparse;
+#else
 #include <argparse/argparse.hpp>
+#endif
 #include <doctest.hpp>
 
 using doctest::test_suite;

test/test_container_arguments.cpp

@@ -1,4 +1,8 @@
+#ifdef WITH_MODULE
+import argparse;
+#else
 #include <argparse/argparse.hpp>
+#endif
 #include <doctest.hpp>
 #include <test_utility.hpp>
 

test/test_default_args.cpp

@@ -1,7 +1,13 @@
+#ifdef WITH_MODULE
+import argparse;
+#else
 #include <argparse/argparse.hpp>
+#endif
 #include <doctest.hpp>
+
 #include <sstream>
 #include <streambuf>
+#include <iostream>
 
 using doctest::test_suite;
 

test/test_default_value.cpp

@@ -1,4 +1,8 @@
+#ifdef WITH_MODULE
+import argparse;
+#else
 #include <argparse/argparse.hpp>
+#endif
 #include <doctest.hpp>
 #include <string>
 

test/test_equals_form.cpp

@@ -1,6 +1,14 @@
+#ifdef WITH_MODULE
+import argparse;
+#else
 #include <argparse/argparse.hpp>
+#endif
 #include <doctest.hpp>
+
 #include <iostream>
+#include <vector>
+#include <string>
+
 using doctest::test_suite;
 
 TEST_CASE("Basic --value=value" * test_suite("equals_form")) {
@@ -36,4 +44,4 @@ TEST_CASE("Basic --value=value with nargs(2)" * test_suite("equals_form")) {
   parser.parse_args({"test", "--long=value1", "value2"});
   REQUIRE((parser.get<std::vector<std::string>>("--long") ==
            std::vector<std::string>{"value1", "value2"}));
-}
+}

test/test_get.cpp

@@ -1,6 +1,12 @@
+#ifdef WITH_MODULE
+import argparse;
+#else
 #include <argparse/argparse.hpp>
+#endif
 #include <doctest.hpp>
 
+#include <any>
+
 using doctest::test_suite;
 
 TEST_CASE("Getting a simple argument" * test_suite("ArgumentParser::get")) {

test/test_help.cpp

@@ -1,6 +1,12 @@
+#ifdef WITH_MODULE
+import argparse;
+#else
 #include <argparse/argparse.hpp>
+#endif
 #include <doctest.hpp>
+
 #include <sstream>
+#include <optional>
 
 using doctest::test_suite;
 

test/test_invalid_arguments.cpp

@@ -1,4 +1,8 @@
+#ifdef WITH_MODULE
+import argparse;
+#else
 #include <argparse/argparse.hpp>
+#endif
 #include <doctest.hpp>
 
 using doctest::test_suite;

test/test_is_used.cpp

@@ -1,4 +1,8 @@
+#ifdef WITH_MODULE
+import argparse;
+#else
 #include <argparse/argparse.hpp>
+#endif
 #include <doctest.hpp>
 
 using doctest::test_suite;

test/test_issue_37.cpp

@@ -1,4 +1,8 @@
+#ifdef WITH_MODULE
+import argparse;
+#else
 #include <argparse/argparse.hpp>
+#endif
 #include <doctest.hpp>
 
 using doctest::test_suite;

test/test_negative_numbers.cpp

@@ -1,4 +1,8 @@
+#ifdef WITH_MODULE
+import argparse;
+#else
 #include <argparse/argparse.hpp>
+#endif
 #include <doctest.hpp>
 
 using doctest::test_suite;

test/test_optional_arguments.cpp

@@ -1,4 +1,8 @@
+#ifdef WITH_MODULE
+import argparse;
+#else
 #include <argparse/argparse.hpp>
+#endif
 #include <doctest.hpp>
 
 using doctest::test_suite;

test/test_parent_parsers.cpp

@@ -1,4 +1,8 @@
+#ifdef WITH_MODULE
+import argparse;
+#else
 #include <argparse/argparse.hpp>
+#endif
 #include <doctest.hpp>
 
 using doctest::test_suite;

test/test_parse_args.cpp

@@ -1,6 +1,12 @@
+#ifdef WITH_MODULE
+import argparse;
+#else
 #include <argparse/argparse.hpp>
+#endif
 #include <doctest.hpp>
 
+#include <optional>
+
 using doctest::test_suite;
 
 TEST_CASE("Missing argument" * test_suite("parse_args")) {

test/test_parse_known_args.cpp

@@ -1,6 +1,13 @@
+#ifdef WITH_MODULE
+import argparse;
+#else
 #include <argparse/argparse.hpp>
+#endif
 #include <doctest.hpp>
 
+#include <vector>
+#include <string>
+
 using doctest::test_suite;
 
 TEST_CASE("Parse unknown optional and positional arguments without exceptions" *
@@ -79,4 +86,4 @@ TEST_CASE("Parse unknown optional and positional arguments in subparsers "
     REQUIRE((unknown_args == std::vector<std::string>{"--verbose", "FOO", "5",
                                                       "BAR", "-jn", "spam"}));
   }
-}
+}

test/test_positional_arguments.cpp

@@ -1,4 +1,8 @@
+#ifdef WITH_MODULE
+import argparse;
+#else
 #include <argparse/argparse.hpp>
+#endif
 #include <cmath>
 #include <doctest.hpp>
 

test/test_prefix_chars.cpp

@@ -1,4 +1,8 @@
+#ifdef WITH_MODULE
+import argparse;
+#else
 #include <argparse/argparse.hpp>
+#endif
 #include <cmath>
 #include <doctest.hpp>
 

test/test_repr.cpp

@@ -1,6 +1,14 @@
+#ifdef WITH_MODULE
+import argparse;
+import argparse.details;
+#else
 #include <argparse/argparse.hpp>
+#endif
 #include <doctest.hpp>
+
 #include <set>
+#include <list>
+#include <sstream>
 
 using doctest::test_suite;
 

test/test_required_arguments.cpp

@@ -1,4 +1,8 @@
+#ifdef WITH_MODULE
+import argparse;
+#else
 #include <argparse/argparse.hpp>
+#endif
 #include <doctest.hpp>
 
 using doctest::test_suite;

test/test_scan.cpp

@@ -1,4 +1,8 @@
+#ifdef WITH_MODULE
+import argparse;
+#else
 #include <argparse/argparse.hpp>
+#endif
 #include <doctest.hpp>
 #include <stdint.h>
 

test/test_subparsers.cpp

@@ -1,7 +1,14 @@
+#ifdef WITH_MODULE
+import argparse;
+#else
 #include <argparse/argparse.hpp>
-#include <cmath>
+#endif
 #include <doctest.hpp>
 
+#include <cmath>
+#include <vector>
+#include <string>
+
 using doctest::test_suite;
 
 TEST_CASE("Add subparsers" * test_suite("subparsers")) {
@@ -236,4 +243,4 @@ TEST_CASE("Check is_subcommand_used after parse" * test_suite("subparsers")) {
     REQUIRE(program.is_subcommand_used("clean") == false);
     REQUIRE(program.is_subcommand_used(command_2) == false);
   }
-}
+}

test/test_utility.hpp

@@ -1,6 +1,8 @@
 #ifndef ARGPARSE_TEST_UTILITY_HPP
 #define ARGPARSE_TEST_UTILITY_HPP
 
+#include <list>
+
 namespace testutility {
 // Get value at index from std::list
 template <typename T>

test/test_value_semantics.cpp

@@ -1,4 +1,8 @@
+#ifdef WITH_MODULE
+import argparse;
+#else
 #include <argparse/argparse.hpp>
+#endif
 #include <doctest.hpp>
 
 using doctest::test_suite;

test/test_version.cpp

@@ -1,4 +1,8 @@
+#ifdef WITH_MODULE
+import argparse;
+#else
 #include <argparse/argparse.hpp>
+#endif
 #include <doctest.hpp>
 #include <sstream>
 

xmake.lua

@@ -0,0 +1,89 @@
+set_xmakever("2.8.2")
+set_project("argparse")
+
+set_version("2.9.0", { build = "%Y%m%d%H%M" })
+
+option("enable_module")
+option("enable_std_import", { defines = "ARGPARSE_MODULE_USE_STD_MODULE" })
+option("enable_tests")
+option("enable_samples")
+
+add_cxxflags(
+    "-Wall",
+    "-Wno-long-long",
+    "-pedantic",
+    "-Wsign-conversion",
+    "-Wshadow",
+    "-Wconversion",
+    { toolsets = { "clang", "gcc" } }
+)
+add_cxxflags("cl::/W4")
+
+if is_plat("windows") then
+    add_defines("_CRT_SECURE_NO_WARNINGS")
+end
+
+target("argparse", function()
+    set_languages("c++17")
+    set_kind("headeronly")
+    if get_config("enable_module") then
+        set_languages("c++20")
+        set_kind("static") -- static atm because of a XMake bug, headeronly doesn't generate package module metadata
+    end
+
+    add_options("enable_std_import")
+
+    add_includedirs("include", { public = true })
+    add_headerfiles("include/argparse/argparse.hpp")
+    if get_config("enable_module") then
+        add_files("module/argparse.cppm", { install = true })
+    end
+end)
+
+if get_config("enable_tests") then
+    target("argparse_tests", function()
+        set_kind("binary")
+        set_languages("c++17")
+        set_basename("tests")
+
+        add_includedirs("test")
+
+        add_files("test/main.cpp", { defines = { "DOCTEST_CONFIG_IMPLEMENT_WITH_MAIN" } })
+        add_files("test/**.cpp")
+
+        add_deps("argparse")
+    end)
+
+    if get_config("enable_module") then
+        target("argparse_module_tests", function()
+            set_kind("binary")
+            set_languages("c++20")
+            set_basename("module_tests")
+
+            add_defines("WITH_MODULE")
+
+            add_includedirs("test")
+
+            add_files("test/main.cpp", { defines = { "DOCTEST_CONFIG_IMPLEMENT_WITH_MAIN" } })
+            add_files("test/**.cpp")
+            add_files("test/argparse_details.cppm")
+
+            add_deps("argparse")
+        end)
+    end
+end
+
+if get_config("enable_samples") then
+    for _, sample_file in ipairs(os.files("samples/*.cpp")) do
+        target(path.basename(sample_file), function()
+            set_kind("binary")
+            set_languages("c++17")
+
+            add_files(sample_file)
+
+            set_policy("build.c++.modules", false)
+
+            add_deps("argparse")
+        end)
+    end
+end