parser: change order of the constructor optional arguments.
authorAlexei Sheplyakov <varg@theor.jinr.ru>
Sat, 13 Sep 2008 00:55:23 +0000 (04:55 +0400)
committerAlexei Sheplyakov <varg@theor.jinr.ru>
Fri, 19 Sep 2008 09:15:49 +0000 (13:15 +0400)
Functions/methods having multiple optional arguments are not very convenient
to use in C++ (to put it mildly). Shuffle parser ctor arguments so not so
frequently used argument is the last one.

ginac/parser/parser.cpp
ginac/parser/parser.hpp

index 1e905cf..0ee682a 100644 (file)
@@ -150,9 +150,9 @@ int parser::get_next_tok()
        return token;
 }
 
-parser::parser(const symtab& syms_, const prototype_table& funcs_,
-              const bool strict_) : strict(strict_), funcs(funcs_),
-       syms(syms_)
+parser::parser(const symtab& syms_, const bool strict_,
+              const prototype_table& funcs_) : strict(strict_),
+       funcs(funcs_), syms(syms_)
 {
        scanner = new lexer();
 }
index dbe60ee..eb50adc 100644 (file)
@@ -62,8 +62,8 @@ public:
         *        symbol is encountered.
         */
        parser(const symtab& syms_ = symtab(),
-               const prototype_table& funcs_ = get_default_reader(),
-               const bool strict_ = false);
+               const bool strict_ = false,
+               const prototype_table& funcs_ = get_default_reader());
        ~parser();
 
        /// parse the stream @a input