diff --git a/include/seqan3/argument_parser/argument_parser.hpp b/include/seqan3/argument_parser/argument_parser.hpp index c51d8220e55..2c451fab096 100644 --- a/include/seqan3/argument_parser/argument_parser.hpp +++ b/include/seqan3/argument_parser/argument_parser.hpp @@ -230,8 +230,8 @@ class argument_parser */ template > //!\cond - requires (argument_parser_compatible_option || - argument_parser_compatible_option) && + requires (argument_parser_compatible_option || + argument_parser_compatible_option) && std::invocable //!\endcond void add_option(option_type & value, @@ -292,9 +292,9 @@ class argument_parser */ template > //!\cond - requires (argument_parser_compatible_option || - argument_parser_compatible_option) && - std::invocable + requires (argument_parser_compatible_option || + argument_parser_compatible_option) && + std::invocable //!\endcond void add_positional_option(option_type & value, std::string const & desc, diff --git a/include/seqan3/argument_parser/auxiliary.hpp b/include/seqan3/argument_parser/auxiliary.hpp index 428ca9325e3..ac32a156eb7 100644 --- a/include/seqan3/argument_parser/auxiliary.hpp +++ b/include/seqan3/argument_parser/auxiliary.hpp @@ -38,7 +38,7 @@ namespace seqan3::custom * Please note that by default the `t const`, `t &` and `t const &` specialisations of this class inherit the * specialisation for `t` so you usually only need to provide a specialisation for `t`. * - * \note Only use this, if you cannot provide respective functions in your namespace. + * \note Only use this if you cannot provide respective functions in your namespace. */ template struct string_convertiblity @@ -100,7 +100,7 @@ namespace seqan3 /*!\brief Return the rank representation of a (semi-)alphabet object. * \tparam your_type Type of the argument. - * \param alph The (semi-)alphabet object. + * \param alph The (semi-)alphabet object. * \returns The rank representation; an integral type. * \ingroup alphabet * \details diff --git a/include/seqan3/argument_parser/detail/format_parse.hpp b/include/seqan3/argument_parser/detail/format_parse.hpp index 4ebb47a1ceb..85402e51bf4 100644 --- a/include/seqan3/argument_parser/detail/format_parse.hpp +++ b/include/seqan3/argument_parser/detail/format_parse.hpp @@ -322,7 +322,7 @@ class format_parse : public format_base auto it = seqan3::string_conversion_map(value).find(tmp); if (it == seqan3::string_conversion_map(value).end()) - throw type_conversion_failed("Argument " + in + " could not be casted to enum type " + + throw type_conversion_failed("Argument " + in + " could not be cast to enum type " + get_display_name_v.str() + "."); else value = it->second; diff --git a/test/snippet/argument_parser/string_conversion_map.cpp b/test/snippet/argument_parser/string_conversion_map.cpp index 04b0b9548ab..6fdf06a6a84 100644 --- a/test/snippet/argument_parser/string_conversion_map.cpp +++ b/test/snippet/argument_parser/string_conversion_map.cpp @@ -27,7 +27,7 @@ int main(int argc, char const * argv[]) // Because of the string_conversion_map function // you can now add an option that takes a value of type Foo: - parser.add_option(value, 'f', "foo", "Give me a value foo value."); + parser.add_option(value, 'f', "foo", "Give me a foo value."); try { diff --git a/test/snippet/argument_parser/string_convertible.cpp b/test/snippet/argument_parser/string_convertible.cpp index a64a5f83bdd..5da2eaf9bc7 100644 --- a/test/snippet/argument_parser/string_convertible.cpp +++ b/test/snippet/argument_parser/string_convertible.cpp @@ -3,11 +3,11 @@ namespace seqan3::custom { -// specialise the seqan3::custom::string_convertiblity data structure to enable parsing of std::errc. +// Specialise the seqan3::custom::string_convertiblity data structure to enable parsing of std::errc. template <> struct string_convertiblity { - // specialise a mapping from your values of type Foo to the respective string + // Specialise a mapping from your values of type Foo to the respective string. static auto string_conversion_map(std::errc) { return std::unordered_map @@ -29,8 +29,8 @@ int main(int argc, char const * argv[]) // Because of the string_convertiblity struct and // the static member function string_conversion_map - // you can now add an option that takes a value of type Foo: - parser.add_option(value, 'e', "errc", "Give me a value std::errc value."); + // you can now add an option that takes a value of type std::errc: + parser.add_option(value, 'e', "errc", "Give me a std::errc value."); try { diff --git a/test/unit/argument_parser/format_parse_test.cpp b/test/unit/argument_parser/format_parse_test.cpp index 3c1ac200ac1..0f6af1d101d 100644 --- a/test/unit/argument_parser/format_parse_test.cpp +++ b/test/unit/argument_parser/format_parse_test.cpp @@ -594,7 +594,7 @@ auto string_conversion_map(Foo) { return std::unordered_map{{{"one", Foo::one}, {"two", Foo::two}, {"three", Foo::three}}}; } -} // namespace bar +} // namespace Bar namespace Other { diff --git a/test/unit/core/debug_stream_test.cpp b/test/unit/core/debug_stream_test.cpp index 076f915f1ef..4329622edbc 100644 --- a/test/unit/core/debug_stream_test.cpp +++ b/test/unit/core/debug_stream_test.cpp @@ -238,7 +238,7 @@ TEST(debug_stream_test, string_convertible) o.flush(); EXPECT_EQ(o.str(), "one"); - fo = Foo::three; // unknown to the converion map + fo = Foo::three; // unknown to the conversion map my_stream << fo; o.flush();