Re: [5/6]wininet: handle cookie expiration

[Aric Stewart]

Wonderful, I was looking for a function JUST like that.

Reworked patch coming up shortly.
-aric

Hans Leidekker wrote:
> > > - LPCWSTR ptr,ptr2;
> > > - SYSTEMTIME tm;
> > > - int i;
> > > -
> > > - memset(&tm,0,sizeof(tm));
> > > -
> > > - ptr = expiry;
> > > - for (i = 0; i < 7; i++)
> > > - {
> > > -
> > >    if (strncmpiW(day[i],ptr,3)==0)
> > > -
> > >    {
> > > -
> > >        tm.wDayOfWeek = i;
> > > -
> > >        break;
> > > -
> > >    }
> > > - }
> > > - ptr += 5;
> > > - tm.wDay = strtolW(ptr,(LPWSTR*)&ptr2,10);
> > > - ptr = ptr2+1;
> >
> > You should handle the various possible parsing errors, not just assume
> > that you have a well-formatted string.
>
> Or use InternetTimeToSystemTimeW which does that already.
>
> -Hans