comtctl32 : updown : Getter and Setter Tests for Updown control

Leslie Choong septikus at gmail.com
Sun Feb 11 12:39:30 CST 2007


Thanks for the input. UDM_SETRANGE has no return value according to
MSDN so I'll remove those assignments.

On 2/11/07, James Hawkins <truiken at gmail.com> wrote:
> On 2/10/07, Leslie Choong <septikus at gmail.com> wrote:
> > Hi there, My name is Leslie Choong and I am currently finishing up my
> > under graduate work at UCLA. This patch is being sent as part of the
> > coursework for CS 130 : Software Engineering. Please take a look and
> > let me know if you have any suggestions or comments for change.
> > Thanks!
> > -Leslie Choong
> >
>
> +    /* Set Range from 0 to 100 */
> +    r = SendMessage(updown, UDM_SETRANGE, 0 , MAKELONG(100,0) );
> +    r= SendMessage(updown, UDM_GETRANGE, 0,0);
>
> If you're not going to check the first return value, then take out the
> 'r =', else it seems like you're missing a test.  On the other hand,
> why don't you test the return value?
>
> --
> James Hawkins
>



More information about the wine-devel mailing list