Patch: VarWeekdayName

[James Hawkins]

On 8/2/07, Jiří Paleček <jpalecek at web.de> wrote:
> On Fri, 03 Aug 2007 01:43:05 +0200, James Hawkins <truiken at gmail.com>
> wrote:
>
> > On 8/2/07, Jiří Paleček <jpalecek at web.de> wrote:
> >> On Thu, 02 Aug 2007 16:06:44 +0200, Juan Lang <juan.lang at gmail.com>
> >> wrote:
> >>
> >> >> Thank you. Hope it will be better this time. Also, I'll send a test
> >> >> if this is OK.
> >> >
> >> > This one looks pretty good to me, but I have one error and a few nits
> >> > to correct:
> >> >
> >> > The error:
> >> > +  {
> >> > +    ERR("GetLocaleInfo of 0x%x failed in 2nd stage?!\n",
> >> localeValue);
> >> > +    SysFreeString(*pbstrOut);
> >> >
> >> > You should set *pbstrOut to NULL in this case.
> >>
> >> OK. However, note that this was copy&pasted, so there are more instances
> >> of this error in wine. It could be somehow automated (like having a
> >> macro
> >> for these disposing functions which will also set the pointer to NULL).
> >>
> >> > The nits:
> >> > You have a couple small errors in the comments:
> >> > - You misspell iWeekday
> >> > - The comment for iWeekday in the params section implies 0 is allowed,
> >> > but the first check in the function disallows it.
> >>
> >> This is the problem the misspelled comment is about, MSDN also suggest
> >> 0 for weekday is OK, however, disassemble showed it isn't on Win XP.
> >>
> >
> > Are you disassembling native windows binaries?
>
> Yes, is it a problem?
>

Yes it's not allowed in this project.  Assuming you only disassembled
oleaut32.dll, you will unfortunately not be able to contribute to that
dll anymore for the Wine project.

-- 
James Hawkins