[PATCH 4/5] riched20: Implement ITextRange::Collapse. (try 2)

Huw Davies huw at codeweavers.com
Wed Sep 17 03:27:18 CDT 2014


On Tue, Sep 16, 2014 at 06:39:32PM +0800, Jactry Zeng wrote:
> +static HRESULT range_Collapse(LONG bStart, LONG *start, LONG *end)
> +{
> +  BOOL isdege = !(*end - *start);

This is a strange way to test for an equality!

Also, isdegen or is_degen would be much better
choices for the variable name.

Huw.



More information about the wine-devel mailing list