diff --git a/example/first.cpp b/example/first.cpp index 878db0a..8763fec 100644 --- a/example/first.cpp +++ b/example/first.cpp @@ -20,7 +20,7 @@ int main(int ac, char* av[]) po::options_description desc("Allowed options"); desc.add_options() ("help", "produce help message") - ("compression", po::value(), "set compression level") + ("compression", po::value(), "set compression level") ; po::variables_map vm; @@ -34,7 +34,7 @@ int main(int ac, char* av[]) if (vm.count("compression")) { cout << "Compression level was set to " - << vm["compression"].as() << ".\n"; + << vm["compression"].as() << ".\n"; } else { cout << "Compression level was not set.\n"; } diff --git a/include/boost/program_options/detail/cmdline.hpp b/include/boost/program_options/detail/cmdline.hpp index 7c43152..8e3bcc6 100644 --- a/include/boost/program_options/detail/cmdline.hpp +++ b/include/boost/program_options/detail/cmdline.hpp @@ -81,6 +81,18 @@ namespace boost { namespace program_options { namespace detail { cmdline(int argc, const char*const * argv); void style(int style); + + /** returns the canonical option prefix associated with the command_line_style + * In order of precedence: + * allow_long : allow_long + * allow_long_disguise : allow_long_disguise + * allow_dash_for_short : allow_short | allow_dash_for_short + * allow_slash_for_short: allow_short | allow_slash_for_short + * + * This is mainly used for the diagnostic messages in exceptions + */ + int get_canonical_option_prefix(); + void allow_unregistered(); void set_options_description(const options_description& desc); diff --git a/include/boost/program_options/detail/parsers.hpp b/include/boost/program_options/detail/parsers.hpp index a1124b2..af240c6 100644 --- a/include/boost/program_options/detail/parsers.hpp +++ b/include/boost/program_options/detail/parsers.hpp @@ -100,7 +100,11 @@ namespace boost { namespace program_options { basic_parsed_options basic_command_line_parser::run() { - parsed_options result(m_desc); + // save the canonical prefixes which were used by this cmdline parser + // eventually inside the parsed results + // This will be handy to format recognisable options + // for diagnostic messages if everything blows up much later on + parsed_options result(m_desc, detail::cmdline::get_canonical_option_prefix()); result.options = detail::cmdline::run(); // Presense of parsed_options -> wparsed_options conversion diff --git a/include/boost/program_options/errors.hpp b/include/boost/program_options/errors.hpp index ff6bc7f..d6dcf39 100644 --- a/include/boost/program_options/errors.hpp +++ b/include/boost/program_options/errors.hpp @@ -12,6 +12,8 @@ #include #include #include +#include + #if defined(BOOST_MSVC) # pragma warning (push) @@ -21,170 +23,17 @@ namespace boost { namespace program_options { + inline std::string strip_prefixes(const std::string& text) + { + return text.substr(text.find_last_of("-/") + 1); + } + /** Base class for all errors in the library. */ class BOOST_PROGRAM_OPTIONS_DECL error : public std::logic_error { public: error(const std::string& xwhat) : std::logic_error(xwhat) {} }; - class BOOST_PROGRAM_OPTIONS_DECL invalid_syntax : public error { - public: - enum kind_t { - long_not_allowed = 30, - long_adjacent_not_allowed, - short_adjacent_not_allowed, - empty_adjacent_parameter, - missing_parameter, - extra_parameter, - unrecognized_line - }; - - invalid_syntax(const std::string& tokens, kind_t kind); - - // gcc says that throw specification on dtor is loosened - // without this line - ~invalid_syntax() throw() {} - - kind_t kind() const; - - const std::string& tokens() const; - - protected: - /** Used to convert kind_t to a related error text */ - static std::string error_message(kind_t kind); - - private: - // TODO: copy ctor might throw - std::string m_tokens; - - kind_t m_kind; - }; - - /** Class thrown when option name is not recognized. */ - class BOOST_PROGRAM_OPTIONS_DECL unknown_option : public error { - public: - unknown_option(const std::string& name) - : error(std::string("unknown option ").append(name)), - m_option_name(name) - {} - - // gcc says that throw specification on dtor is loosened - // without this line - ~unknown_option() throw() {} - - const std::string& get_option_name() const throw(); - - private: - std::string m_option_name; - }; - - /** Class thrown when there's ambiguity amoung several possible options. */ - class BOOST_PROGRAM_OPTIONS_DECL ambiguous_option : public error { - public: - ambiguous_option(const std::string& name, - const std::vector& xalternatives) - : error(std::string("ambiguous option ").append(name)) - , m_alternatives(xalternatives) - , m_option_name(name) - {} - - ~ambiguous_option() throw() {} - - const std::string& get_option_name() const throw(); - - const std::vector& alternatives() const throw(); - - private: - // TODO: copy ctor might throw - std::vector m_alternatives; - std::string m_option_name; - }; - - /** Class thrown when there are several option values, but - user called a method which cannot return them all. */ - class BOOST_PROGRAM_OPTIONS_DECL multiple_values : public error { - public: - multiple_values() - : error("multiple values") - , m_option_name() {} - - ~multiple_values() throw() {} - - void set_option_name(const std::string& option); - - const std::string& get_option_name() const throw(); - - private: - std::string m_option_name; // The name of the option which - // caused the exception. - }; - - /** Class thrown when there are several occurrences of an - option, but user called a method which cannot return - them all. */ - class BOOST_PROGRAM_OPTIONS_DECL multiple_occurrences : public error { - public: - multiple_occurrences() - : error("multiple occurrences") - , m_option_name() {} - - ~multiple_occurrences() throw() {} - - void set_option_name(const std::string& option); - - const std::string& get_option_name() const throw(); - - private: - std::string m_option_name; // The name of the option which - // caused the exception. - }; - - /** Class thrown when value of option is incorrect. */ - class BOOST_PROGRAM_OPTIONS_DECL validation_error : public error { - public: - enum kind_t { - multiple_values_not_allowed = 30, - at_least_one_value_required, - invalid_bool_value, - invalid_option_value, - invalid_option - }; - - validation_error(kind_t kind, - const std::string& option_value = "", - const std::string& option_name = ""); - - ~validation_error() throw() {} - - void set_option_name(const std::string& option); - - const std::string& get_option_name() const throw(); - - const char* what() const throw(); - - protected: - /** Used to convert kind_t to a related error text */ - static std::string error_message(kind_t kind); - - private: - kind_t m_kind; - std::string m_option_name; // The name of the option which - // caused the exception. - std::string m_option_value; // Optional: value of the option m_options_name - mutable std::string m_message; // For on-demand formatting in 'what' - - }; - - /** Class thrown if there is an invalid option value givenn */ - class BOOST_PROGRAM_OPTIONS_DECL invalid_option_value - : public validation_error - { - public: - invalid_option_value(const std::string& value); -#ifndef BOOST_NO_STD_WSTRING - invalid_option_value(const std::wstring& value); -#endif - }; /** Class thrown when there are too many positional options. This is a programming error. @@ -192,16 +41,10 @@ namespace boost { namespace program_options { class BOOST_PROGRAM_OPTIONS_DECL too_many_positional_options_error : public error { public: too_many_positional_options_error() - : error("too many positional options") + : error("too many positional arguments have been specified on the command line") {} }; - /** Class thrown when there are syntax errors in given command line */ - class BOOST_PROGRAM_OPTIONS_DECL invalid_command_line_syntax : public invalid_syntax { - public: - invalid_command_line_syntax(const std::string& tokens, kind_t kind); - }; - /** Class thrown when there are programming error related to style */ class BOOST_PROGRAM_OPTIONS_DECL invalid_command_line_style : public error { public: @@ -214,26 +57,355 @@ namespace boost { namespace program_options { class BOOST_PROGRAM_OPTIONS_DECL reading_file : public error { public: reading_file(const char* filename) - : error(std::string("can not read file ").append(filename)) + : error(std::string("can not read options configuration file '").append(filename).append("'")) {} }; - - /** Class thrown when a required/mandatory option is missing */ - class BOOST_PROGRAM_OPTIONS_DECL required_option : public error { - public: - required_option(const std::string& name) - : error(std::string("missing required option ").append(name)) - , m_option_name(name) - {} - - ~required_option() throw() {} - const std::string& get_option_name() const throw(); + + /** Base class for most exceptions in the library. + * + * Substitutes the values for the parameter name + * placeholders in the template to create the human + * readable error message + * + * Placeholders are surrounded by % signs: %example% + * Poor man's version of boost::format + * + * If a parameter name is absent, perform default substitutions + * instead so ugly placeholders are never left in-place. + * + * Options are displayed in "canonical" form + * This is the most unambiguous form of the + * *parsed* option name and would correspond to + * option_description::format_name() + * i.e. what is shown by print_usage() + * + * The "canonical" form depends on whether the option is + * specified in short or long form, using dashes or slashes + * or without a prefix (from a configuration file) + * + * */ + class BOOST_PROGRAM_OPTIONS_DECL error_with_option_name : public error { + + protected: + /** can be + * 0 = no prefix (config file options) + * allow_long + * allow_dash_for_short + * allow_slash_for_short + * allow_long_disguise */ + int m_option_style; + + + /** substitutions + * from placeholders to values */ + std::map m_substitutions; + typedef std::pair string_pair; + std::map m_substitution_defaults; + + public: + /** template with placeholders */ + std::string m_error_template; + + error_with_option_name(const std::string& template_, + const std::string& option_name = "", + const std::string& original_token = "", + int option_style = 0); + + /** gcc says that throw specification on dtor is loosened + * without this line + * */ + ~error_with_option_name() throw() {} + + + //void dump() const + //{ + // std::cerr << "m_substitution_defaults:\n"; + // for (std::map::const_iterator iter = m_substitution_defaults.begin(); + // iter != m_substitution_defaults.end(); ++iter) + // std::cerr << "\t" << iter->first << ":" << iter->second.first << "=" << iter->second.second << "\n"; + // std::cerr << "m_substitutions:\n"; + // for (std::map::const_iterator iter = m_substitutions.begin(); + // iter != m_substitutions.end(); ++iter) + // std::cerr << "\t" << iter->first << "=" << iter->second << "\n"; + // std::cerr << "m_error_template:\n"; + // std::cerr << "\t" << m_error_template << "\n"; + // std::cerr << "canonical_option_prefix:[" << get_canonical_option_prefix() << "]\n"; + // std::cerr << "canonical_option_name:[" << get_canonical_option_name() <<"]\n"; + // std::cerr << "what:[" << what() << "]\n"; + //} + + /** Substitute + * parameter_name->value to create the error message from + * the error template */ + void set_substitute(const std::string& parameter_name, const std::string& value) + { m_substitutions[parameter_name] = value; } + + /** If the parameter is missing, then make the + * from->to substitution instead */ + void set_substitute_default(const std::string& parameter_name, + const std::string& from, + const std::string& to) + { + m_substitution_defaults[parameter_name] = std::make_pair(from, to); + } + + + /** Add context to an exception */ + void add_context(const std::string& option_name, + const std::string& original_token, + int option_style) + { + set_option_name(option_name); + set_original_token(original_token); + set_prefix(option_style); + } + + void set_prefix(int option_style) + { m_option_style = option_style;} + + /** Overridden in error_with_no_option_name */ + virtual void set_option_name(const std::string& option_name) + { set_substitute("option", option_name);} + + std::string get_option_name() const throw() + { return get_canonical_option_name(); } + + void set_original_token(const std::string& original_token) + { set_substitute("original_token", original_token);} + + + /** Creates the error_message on the fly + * Currently a thin wrapper for substitute_placeholders() */ + virtual const char* what() const throw(); + + protected: + /** Used to hold the error text returned by what() */ + mutable std::string m_message; // For on-demand formatting in 'what' + + /** Makes all substitutions using the template */ + virtual void substitute_placeholders(const std::string& error_template) const; + + // helper function for substitute_placeholders + void replace_token(const std::string& from, const std::string& to) const; + + /** Construct option name in accordance with the appropriate + * prefix style: i.e. long dash or short slash etc */ + std::string get_canonical_option_name() const; + std::string get_canonical_option_prefix() const; + }; + + + /** Class thrown when there are several option values, but + user called a method which cannot return them all. */ + class BOOST_PROGRAM_OPTIONS_DECL multiple_values : public error_with_option_name { + public: + multiple_values() + : error_with_option_name("option '%canonical_option%' only takes a single argument"){} + + ~multiple_values() throw() {} + }; + + /** Class thrown when there are several occurrences of an + option, but user called a method which cannot return + them all. */ + class BOOST_PROGRAM_OPTIONS_DECL multiple_occurrences : public error_with_option_name { + public: + multiple_occurrences() + : error_with_option_name("option '%canonical_option%' cannot be specified more than once"){} + + ~multiple_occurrences() throw() {} + + }; + + /** Class thrown when a required/mandatory option is missing */ + class BOOST_PROGRAM_OPTIONS_DECL required_option : public error_with_option_name { + public: + // option name is constructed by the option_descriptor and never on the fly + required_option(const std::string& option_name) + : error_with_option_name("the option '%canonical_option%' is required but missing", "", option_name) + { + } + + ~required_option() throw() {} + }; + + /** Base class of unparsable options, + * when the desired option cannot be identified. + * + * + * It makes no sense to have an option name, when we can't match an option to the + * parameter + * + * Having this a part of the error_with_option_name hierachy makes error handling + * a lot easier, even if the name indicates some sort of conceptual dissonance! + * + * */ + class BOOST_PROGRAM_OPTIONS_DECL error_with_no_option_name : public error_with_option_name { + public: + error_with_no_option_name(const std::string& template_, + const std::string& original_token = "") + : error_with_option_name(template_, "", original_token) + { + } + + /** Does NOT set option name, because no option name makes sense */ + virtual void set_option_name(const std::string& option_name){} + + ~error_with_no_option_name() throw() {} + }; + + + /** Class thrown when option name is not recognized. */ + class BOOST_PROGRAM_OPTIONS_DECL unknown_option : public error_with_no_option_name { + public: + unknown_option(const std::string& original_token = "") + : error_with_no_option_name("unrecognised option '%canonical_option%'", original_token) + { + } + + ~unknown_option() throw() {} + }; + + + + /** Class thrown when there's ambiguity amoung several possible options. */ + class BOOST_PROGRAM_OPTIONS_DECL ambiguous_option : public error_with_no_option_name { + public: + ambiguous_option(const std::vector& xalternatives) + : error_with_no_option_name("option '%canonical_option%' is ambiguous"), + m_alternatives(xalternatives) + {} + + ~ambiguous_option() throw() {} + + const std::vector& alternatives() const throw() {return m_alternatives;} + + protected: + /** Makes all substitutions using the template */ + virtual void substitute_placeholders(const std::string& error_template) const; + private: + // TODO: copy ctor might throw + std::vector m_alternatives; + }; + + + /** Class thrown when there's syntax error either for command + * line or config file options. See derived children for + * concrete classes. */ + class BOOST_PROGRAM_OPTIONS_DECL invalid_syntax : public error_with_option_name { + public: + enum kind_t { + long_not_allowed = 30, + long_adjacent_not_allowed, + short_adjacent_not_allowed, + empty_adjacent_parameter, + missing_parameter, + extra_parameter, + unrecognized_line + }; + + invalid_syntax(kind_t kind, + const std::string& option_name = "", + const std::string& original_token = "", + int option_style = 0): + error_with_option_name(get_template(kind), option_name, original_token, option_style), + m_kind(kind) + { + } + + ~invalid_syntax() throw() {} + + kind_t kind() const {return m_kind;} + + /** Convenience functions for backwards compatibility */ + virtual std::string tokens() const {return get_option_name(); } + protected: + /** Used to convert kind_t to a related error text */ + std::string get_template(kind_t kind); + kind_t m_kind; + }; + + class BOOST_PROGRAM_OPTIONS_DECL invalid_config_file_syntax : public invalid_syntax { + public: + invalid_config_file_syntax(const std::string& invalid_line, kind_t kind): + invalid_syntax(kind) + { + m_substitutions["invalid_line"] = invalid_line; + } + + ~invalid_config_file_syntax() throw() {} + + /** Convenience functions for backwards compatibility */ + virtual std::string tokens() const {return m_substitutions.find("invalid_line")->second; } + }; + + + /** Class thrown when there are syntax errors in given command line */ + class BOOST_PROGRAM_OPTIONS_DECL invalid_command_line_syntax : public invalid_syntax { + public: + invalid_command_line_syntax(kind_t kind, + const std::string& option_name = "", + const std::string& original_token = "", + int option_style = 0): + invalid_syntax(kind, option_name, original_token, option_style) {} + ~invalid_command_line_syntax() throw() {} + }; + + + /** Class thrown when value of option is incorrect. */ + class BOOST_PROGRAM_OPTIONS_DECL validation_error : public error_with_option_name { + public: + enum kind_t { + multiple_values_not_allowed = 30, + at_least_one_value_required, + invalid_bool_value, + invalid_option_value, + invalid_option + }; - private: - std::string m_option_name; // The name of the option which - // caused the exception. - }; + public: + validation_error(kind_t kind, + const std::string& option_name = "", + const std::string& original_token = "", + int option_style = 0): + error_with_option_name(get_template(kind), option_name, original_token, option_style) + { + } + + ~validation_error() throw() {} + + protected: + /** Used to convert kind_t to a related error text */ + std::string get_template(kind_t kind); + kind_t m_kind; + }; + + /** Class thrown if there is an invalid option value given */ + class BOOST_PROGRAM_OPTIONS_DECL invalid_option_value + : public validation_error + { + public: + invalid_option_value(const std::string& value); +#ifndef BOOST_NO_STD_WSTRING + invalid_option_value(const std::wstring& value); +#endif + }; + + /** Class thrown if there is an invalid bool value given */ + class BOOST_PROGRAM_OPTIONS_DECL invalid_bool_value + : public validation_error + { + public: + invalid_bool_value(const std::string& value); + }; + + + + + + + }} #if defined(BOOST_MSVC) diff --git a/include/boost/program_options/options_description.hpp b/include/boost/program_options/options_description.hpp index eff1f90..62530b2 100644 --- a/include/boost/program_options/options_description.hpp +++ b/include/boost/program_options/options_description.hpp @@ -102,6 +102,16 @@ namespace program_options { */ const std::string& key(const std::string& option) const; + + /** Returns the canonical name for the option description to enable the user to + recognised a matching option. + 1) For short options ('-', '/'), returns the short name prefixed. + 2) For long options ('--' / '-') returns the long name prefixed + 3) All other cases, returns the long name (if present) or the short name, + unprefixed. + */ + std::string canonical_display_name(int canonical_option_style = 0) const; + const std::string& long_name() const; /// Explanation of this option diff --git a/include/boost/program_options/parsers.hpp b/include/boost/program_options/parsers.hpp index cbd5e7d..96f38f2 100644 --- a/include/boost/program_options/parsers.hpp +++ b/include/boost/program_options/parsers.hpp @@ -36,8 +36,8 @@ namespace boost { namespace program_options { template class basic_parsed_options { public: - explicit basic_parsed_options(const options_description* xdescription) - : description(xdescription) {} + explicit basic_parsed_options(const options_description* xdescription, int options_prefix = 0) + : description(xdescription), m_options_prefix(options_prefix) {} /** Options found in the source. */ std::vector< basic_option > options; /** Options description that was used for parsing. @@ -46,6 +46,17 @@ namespace boost { namespace program_options { up to the caller. Can be NULL. */ const options_description* description; + + /** Mainly used for the diagnostic messages in exceptions. + * The canonical option prefix for the parser which generated these results, + * depending on the settings for basic_command_line_parser::style() or + * cmdline::style(). In order of precedence of command_line_style enums: + * allow_long + * allow_long_disguise + * allow_dash_for_short + * allow_slash_for_short + */ + int m_options_prefix; }; /** Specialization of basic_parsed_options which: @@ -64,6 +75,17 @@ namespace boost { namespace program_options { /** Stores UTF8 encoded options that were passed to constructor, to avoid reverse conversion in some cases. */ basic_parsed_options utf8_encoded_options; + + /** Mainly used for the diagnostic messages in exceptions. + * The canonical option prefix for the parser which generated these results, + * depending on the settings for basic_command_line_parser::style() or + * cmdline::style(). In order of precedence of command_line_style enums: + * allow_long + * allow_long_disguise + * allow_dash_for_short + * allow_slash_for_short + */ + int m_options_prefix; }; typedef basic_parsed_options parsed_options; diff --git a/include/boost/program_options/variables_map.hpp b/include/boost/program_options/variables_map.hpp index 7cb6621..be0a4b6 100644 --- a/include/boost/program_options/variables_map.hpp +++ b/include/boost/program_options/variables_map.hpp @@ -174,8 +174,10 @@ namespace boost { namespace program_options { bool utf8); /** Names of required options, filled by parser which has - access to options_description. */ - std::set m_required; + access to options_description. + The map values are the "canonical" names for each corresponding option. + This is useful in creating diagnostic messages when the option is absent. */ + std::map m_required; }; diff --git a/src/cmdline.cpp b/src/cmdline.cpp index 09e0256..6c1e582 100644 --- a/src/cmdline.cpp +++ b/src/cmdline.cpp @@ -34,64 +34,43 @@ namespace boost { namespace program_options { using namespace std; using namespace boost::program_options::command_line_style; - invalid_syntax:: - invalid_syntax(const string& tokens, kind_t kind) - : error(error_message(kind).append(" in '").append(tokens).append("'")) - , m_tokens(tokens) - , m_kind(kind) - {} string - invalid_syntax::error_message(kind_t kind) + invalid_syntax::get_template(kind_t kind) { // Initially, store the message in 'const char*' variable, // to avoid conversion to string in all cases. const char* msg; switch(kind) { - case long_not_allowed: - msg = "long options are not allowed"; - break; - case long_adjacent_not_allowed: - msg = "parameters adjacent to long options not allowed"; - break; - case short_adjacent_not_allowed: - msg = "parameters adjust to short options are not allowed"; - break; case empty_adjacent_parameter: - msg = "adjacent parameter is empty"; + msg = "the argument for option '%canonical_option%' should follow immediately after the equal sign"; break; case missing_parameter: - msg = "required parameter is missing"; - break; - case extra_parameter: - msg = "extra parameter"; + msg = "the required argument for option '%canonical_option%' is missing"; break; case unrecognized_line: - msg = "unrecognized line"; + msg = "the options configuration file contains an invalid line '%invalid_line%'"; + break; + // none of the following are currently used: + case long_not_allowed: + msg = "the unabbreviated option '%canonical_option%' is not valid"; + break; + case long_adjacent_not_allowed: + msg = "the unabbreviated option '%canonical_option%' does not take any arguments"; + break; + case short_adjacent_not_allowed: + msg = "the abbreviated option '%canonical_option%' does not take any arguments"; + break; + case extra_parameter: + msg = "option '%canonical_option%' does not take any arguments"; break; default: - msg = "unknown error"; + msg = "unknown command line syntax error for '%s'"; } return msg; } - invalid_syntax::kind_t - invalid_syntax::kind() const - { - return m_kind; - } - - const string& - invalid_syntax::tokens() const - { - return m_tokens; - } - - invalid_command_line_syntax:: - invalid_command_line_syntax(const string& tokens, kind_t kind) - : invalid_syntax(tokens, kind) - {} }} @@ -156,15 +135,26 @@ namespace boost { namespace program_options { namespace detail { const char* error = 0; if (allow_some_long && !(style & long_allow_adjacent) && !(style & long_allow_next)) - error = "style disallows parameters for long options"; + error = "boost::program_options misconfiguration: " + "choose one or other of 'command_line_style::long_allow_next' " + "(whitespace separated arguments) or " + "'command_line_style::long_allow_adjacent' ('=' separated arguments) for " + "long options."; if (!error && (style & allow_short) && !(style & short_allow_adjacent) && !(style & short_allow_next)) - error = "style disallows parameters for short options"; + error = "boost::program_options misconfiguration: " + "choose one or other of 'command_line_style::short_allow_next' " + "(whitespace separated arguments) or " + "'command_line_style::short_allow_adjacent' ('=' separated arguments) for " + "short options."; if (!error && (style & allow_short) && !(style & allow_dash_for_short) && !(style & allow_slash_for_short)) - error = "style disallows all characters for short options"; + error = "boost::program_options misconfiguration: " + "choose one or other of 'command_line_style::allow_slash_for_short' " + "(slashes) or 'command_line_style::allow_dash_for_short' (dashes) for " + "short options."; if (error) boost::throw_exception(invalid_command_line_style(error)); @@ -192,6 +182,23 @@ namespace boost { namespace program_options { namespace detail { m_positional = &positional; } + int + cmdline::get_canonical_option_prefix() + { + if (m_style & allow_long) + return allow_long; + + if (m_style & allow_long_disguise) + return allow_long_disguise; + + if ((m_style & allow_short) && (m_style & allow_dash_for_short)) + return allow_dash_for_short; + + if ((m_style & allow_short) && (m_style & allow_slash_for_short)) + return allow_slash_for_short; + + return 0; + } vector