Hey there, I've made the changes so that the return value is not stored unnecessarily. I've also added comments explaining why. Let me know what you think. -Leslie
On 2/11/07, Leslie Choong septikus@gmail.com wrote:
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@gmail.com wrote:
On 2/10/07, Leslie Choong septikus@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