Mailing-List: contact stdcxx-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: stdcxx-dev@incubator.apache.org
Received-SPF: pass (athena.apache.org: domain of msebor@gmail.com designates
 209.85.198.186 as permitted sender)
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=beta;
        h=received:message-id:date:from:organization:user-agent:mime-version:to:subject:references:in-reply-to:content-type:content-transfer-encoding:sender;
        b=m9V5kS5qLnB6mZjebrW9PlSWvck2A1XSb7RwCArlt/ktdrFepqcqv28++ORtgQw3Knba6AbKCwmJSCQHNzc2QQsYYeZL0txxiRDwy+LpA4EkaRnGNAUp5hAOo0ITHoA4x7UJuelKMGi+FeAcp89MDl/FjP1JAMEAc7OrboFBAyY=
Message-ID: <46BF913F.8000403@roguewave.com>
Date: Sun, 12 Aug 2007 17:01:19 -0600
From: Martin Sebor <sebor@roguewave.com>
Organization: Rogue Wave Software
User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US;
 rv:1.8.0.12) Gecko/20070719 Fedora/1.0.9-2.fc6 pango-text SeaMonkey/1.0.9
MIME-Version: 1.0
To: stdcxx-dev@incubator.apache.org
Subject: Re: [PATCH] Update test 22.locale.time.put.mt.cpp to validate results
 [take 2]
References: <a648c5670708101402x2cf95594q7b70ed28091c8867@mail.gmail.com>
In-Reply-To: <a648c5670708101402x2cf95594q7b70ed28091c8867@mail.gmail.com>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Sender: Martin Sebor <msebor@gmail.com>

Travis Vitek wrote:
> Attached is a patch to enhance the time_put facet mt test. Threads
> verify that the values they put compare equal to those put in the
> primary thread.

Now that I've committed the patch... I have a question about the bit
below that I noticed too late:

[...]
> @@ -160,19 +214,117 @@
>  static int
>  run_test (int, char**)
[...]
> +    const char* const possible_locale_options[] = {
> +        locale_list, "C\0", 0
> +    };

Is the purpose of this code to exercise the "C" locale in addition
to all the named locales returned from rw_locales()?

If so, it's a valuable enhancement to the test since the base facet
(i.e., time_put as opposed to time_put_byname) wasn't necessarily
being exercised by the test before this change. Good catch!

That said, I'm not quite happy with how this solution is "grafted"
on to the current general mechanism we use to obtain the list of
locales to test. First, the list of locales the test says (in the
call to rw_info()) it exercises doesn't include this locale. Second,
there's no way to disable it using the --locales option. Finally,
it's a chunk of code that will likely end up being repeated in all
the locale tests and so it's a perfect candidate for an enhancement
to the test driver.

What do you think about this: let's change rw_locales() to always
return a list of names that starts with "C". That way callers that
don't want to exercise the "C" locale can simply skip past it while
others will be guaranteed to exercise the classic locale.

Martin