alexis_wilke — Tue, 17 Apr 2007 07:25:43 GMT

Logged In: YES
user_id=554061
Originator: YES
Okay! I got it! You have tons of fixes in your CVS and no new tarball for years... Argh!
To finish up the fix, change the following:
            if (found != -1)
            {
                vector<string> alts;
                // FIXME: the use of 'key' here might not
                // be the best approach.
                alts.push_back(m_options[found]->key(name));
                alts.push_back(m_options[i]->key(name));
                boost::throw_exception(ambiguous_option(name, alts));
            }
            else
            {
                found = i;
            }
with this:
   if(found != -1) found = -2; else found = i;
then right after the loop:
   if(found == -2) throw ...
That lets people enter their options in whatever order
(especially if they have multiple tables of options like me!)
Thank you.
Note: I'm removing the attachment since that's what's in the CVS:
http://boost.cvs.sourceforge.net/boost/boost/libs/program_options/src/options_description.cpp?revision=1.16.2.3&view=markup

status, resolution changed; severity set

Vladimir Prus — Wed, 11 Jul 2007 19:08:15 GMT

status assigned → closed
resolution None → fixed
severity → Showstopper

Fixed in CVS HEAD. I've used slightly different code solution, but thanks for the patch anyway!

Boost C++ Libraries: Ticket #898: [program_options] improper guess

status, resolution changed; severity set