dplayx: Fix incorrect check in DirectPlayLobbyAImpl_EnumLocalApplications and IDirectPlayLobbyAImpl_EnumAddressTypes.

Marcus Meissner meissner at suse.de
Thu Feb 21 10:54:19 CST 2008


On Thu, Feb 21, 2008 at 04:44:34PM +0000, Robert Shearman wrote:
>
> Dereferencing a function pointer and checking it against NULL doesn't make 
> any sense, so remove the checks.


> -  if( !lpEnumAddressTypeCallback || !*lpEnumAddressTypeCallback )
> +  if( !lpEnumAddressTypeCallback )

It looks as if they were meant as &&, but that does not make much sense
either.

Ciao, Marcus



More information about the wine-devel mailing list