[2/7] dxdiagn: Fix return and output behavior of IDxDiagContainer::EnumChildContainerNames. (try 2)

Vitaliy Margolen wine-devel at kievinfo.com
Sun Mar 14 16:05:01 CDT 2010


On 03/14/2010 11:03 AM, Andrew Nguyen wrote:
> ---
>  dlls/dxdiagn/container.c       |   15 +++++------
>  dlls/dxdiagn/tests/container.c |   49 ++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 56 insertions(+), 8 deletions(-)
> 
> -  
> -  TRACE("(%p, %u, %s, %u)\n", iface, dwIndex, debugstr_w(pwszContainer), cchContainer);
>  
> -  if (NULL == pwszContainer) {
> +  TRACE("(%p, %u, %p, %u)\n", iface, dwIndex, pwszContainer, cchContainer);
> +
> +  if (NULL == pwszContainer || 0 == cchContainer) {
>      return E_INVALIDARG;
>    }
Please add printing of debugstr_w(pwszContainer) back, after sanity checks.
It's something really useful to know.




More information about the wine-devel mailing list