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.
