Martin Vermeer <[EMAIL PROTECTED]> writes:
| 323 } else {
| 324 FuncRequest cmd1 = cmd;
| 325 if (!bv->lockInset(this))
| 326 return DISPATCHED;
| 327 if (cmd.y <= button_dim.y2) {
| 328 cmd1.y = 0;
| 329 } else {
| 330 cmd1.y = ascent() + cmd.y - (height_collapsed() +
inset.ascent());
| 331 }
| 332 inset.localDispatch(cmd);
| 333 }
| 334 return DISPATCHED;
>
| Here is the BUG: inset.localDispatch uses cmd, not cmd1 (why was cmd1
| constructed in the first place?)
>
| So I fixed this and replaced the expression on line 330 by a call to
| the adjustCommand() method, which seems to be made for the purpose.
| And then I rearranged statements around the if...else.
Seems sound to me.
--
Lgb
