Re: [PATCH 0/1] MR4060: winegstreamer: Remove redundant null check before wcsdup in object_context_create.

[Zebediah Figura (＠zfigura)]

On Tue Oct 10 18:21:12 2023 +0000, Alex Henrie wrote:
> We have a [test for
> strdup](https://gitlab.winehq.org/wine/wine/-/blob/eb06f991725d21e1ce4e3353a377e8708...)
> that shows that strdup(NULL) == NULL on Windows, but it appears we don't
> have a similar test for wcsdup. I'd be happy to add that in a separate
> MR to reassure everyone that we can count on this behavior. It sounds
> like we should test the value of errno too.

Despite the test I'm still not convinced this is a good idea. If the code doesn't check for a NULL pointer and just passes directly to wcsdup(), it makes me think that it will never be NULL, which isn't the case.