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.