On 7/27/2012 06:00, Austin English wrote:
- if ( !locale_name.matches )
return FALSE;
- return TRUE;
Considering recent cleanup of such constructs please return like 'return locale_name.matches != 0;' or something like that.
+static void test_IsValidLocaleName(void) +{
- static const WCHAR enW[] = {'e','n',0};
- static const WCHAR enusW[] = {'e','n','-','U','S',0};
- static const WCHAR zzW[] = {'z','z',0};
- static const WCHAR zzzzW[] = {'z','z','-','Z','Z',0};
- BOOL ret;
Please iterate through test data so it could be easily extended in future.
Also it looks like duplication, please add some invalid locale names tests for LocaleNameToLCID and if return 0 for example, simply use it for validation (you'll get covered with LOCALE_NAME_USER_DEFAULT things for example).