Date and time information is not spoken correctly in some languages because of small buffer size in winKernel.GetDateFormat and GetTimeFormat

[ragb]

In winkernel.GetDateFormat? (line 76 of current trunk winkernel.py) the created unicode buffer that will have the date string is sized to 32 characteres. At least in Portuguese (and some other languages I think) there are situations when Windows returns a 33, 34 or more characters string. The GetTimeFormat? may also suffer from the same bug.

There are to solutions to this problem. The Obvious one is to increase to buffer size to 64 or 128 chars, it must work for every language out there.
The other solution is partialy described here:
 http://msdn.microsoft.com/en-us/library/dd318086%28VS.85%29.aspx
and requires to calls to the kernel32.GetDateFormat?. One with the output buffer set to NULL and size to 0 to get the string size on the return value, and other to get the string itself for a allocated buffer with the previously got size value.

I've no way to compile NVDA from source right now so if someone could do something before 2009.1 it would be fine... If not I will try to write a patch but don't know when...

[aleksey_s]

Implemented the fix.

[aleksey_s]

Proposed patch

[pvagner]

Thanks Aleksey for the patch.
Also I have found out in the script_dateTime the call to the getThreadLocale function is redundant and we should just use the constant LOCALE_USER_DEFAULT.

[pvagner]

oops Sorry Aleksey, I've taken over the issue after you've accepted it. So I'am now returning it back to you and adding a patch with the little improvement I was tallking about.
Feel free to commit if you like.

[aleksey_s]

Fixed in r3365