[PATCH 3/4] msi: automation: Generalize list implementation.
Alexandre Julliard
julliard at winehq.org
Fri May 18 13:15:49 CDT 2007
Misha Koshelev <mk144210 at bcm.edu> writes:
> if (wFlags & DISPATCH_PROPERTYGET) {
> V_VT(pVarResult) = VT_I4;
> - V_I4(pVarResult) = data->iCount;
> + V_I4(pVarResult) = data->ulCount;
I committed this patch, but I can't resist pointing out that if you
had simply named the field "count" instead of using the (idiotic)
Hungarian notation, the code would be just as clear, and you wouldn't
need to rename it everywhere when making it unsigned...
--
Alexandre Julliard
julliard at winehq.org
More information about the wine-devel
mailing list