On Thu, 10 Oct 2002 04:27, Alexandre Julliard wrote:
> I still don't see much reason for renaming these things, but if you
> really want to do it, again a separate patch with just that change
> would be nicer.
will send this as a separate patch shortly
> Your patch
> breaks the scrollbars in notepad for me, but there are so many
> irrelevant differences that finding the bug is a lot more painful than
> it should be.
I assume that you mean that the tracking was not working correctly this was
due to the change
if ((info->fMask & SIF_TRACKPOS) && (info->cbSize == sizeof(*info)))
- info->nTrackPos = (SCROLL_TrackingWin == WIN_GetFullHandle(hwnd)) ?
SCROLL_TrackingVal : infoPtr->curVal;
+ info->nTrackPos = infoPtr->curVal;
It can be removed by patch 4 but not at this stage.
I apologise although I played with the whole set of changes for hours looking at all
sorts of corner cases,
when I split the patch up I simply checked that it compiled and then used notepad and
the file dialog in
particular to check that it looked ok and responded to mouse events etc.
