oleaut32: be more explicit with locale dependent parsing of dates (try 2)

[Nikolay Sivov]

On 4/14/2010 23:56, André Hentschel wrote:
> Nikolay Sivov schrieb:
>    
>> On 4/14/2010 22:51, André Hentschel wrote:
>>      
>>>        case 0x3: /* TTT TTTDD TTTDDD */
>>> -      if (iDate&&   dp.dwCount == 3)
>>> +      if (dp.dwCount == 3&&   !strncmp(ShortDate,"dd.MM.yy",8))
>>>
>>>
>>>        
>>      
>>>        case 0x1B: /* localized DDDTTT */
>>> -      if (!iDate)
>>> +      if (strncmp(ShortDate,"dd.MM.yy",8))
>>>            {
>>>              hRet = DISP_E_TYPEMISMATCH;
>>>              break;
>>>
>>>        
>> This looks like a workaround. I don't think you should hard code it this
>> way, what's so special with this format?
>> And what is a initial problem with that?
>>
>>
>>      
> The date and time parsing is locale dependent of course, but the code is mainly written for english.
> There was a bug so i fixed it and introduced a regression and bug 22287 came up.
>    
Yes, but that doesn't mean it should be hard coded this way. Am I right 
it should use localized format and try to fit input in it?
If so, this should be fixed in a general way.