Re: [perl-win32-gui-users] Documentation and default values [Was: BUG Followup[1092732 ]:Slider no longer supports setting position]

[Jez W. <je...@je...>]

> The XS code cannot provide a default in this case (due to the 
> optionality of the first argument), and so explicitly looks at the 
> number of arguements.   We have 4 ways to call this method, and for 
> consistency with what happens elsewhere this is how  I believe the 
> behaviour should be:
> (1) $obj->Pos()                  - 0 arguments, return current position 
> (TBM_GETPOS)
> (2) $obj->Pos($pos)          - 1 argument,  redraw defaults to 1 and 
> redraw is requested
> (3) $obj->Pos($pos, 1)      - 2 arguments, redraw explicitly set to 1, 
> redraw is requested
> (4) $obj->Pos($pos, 0)      - 2 arguments, redraw explicitly set to 0, 
> redraw is not requested
> 
> The "bug" I reported is with option (2), where the current code defaults 
> redraw to 0.  For consistency I think this should be changed.  Can we 
> reach an agreement on this?

Sounds good to me.

Cheers,

jez.