- Wine-Devel - list.winehq.org

mmtimer resolution question
by Robert Reif Feb. 11, 2005

Feb. 11, 2005

I'm trying to track down a problem with the direct sound regression tests when run in emulation mode using the wave api. I'm getting a lot of FIXMEs and poor quality playback. The dsound mixer callback routine expects to be called every 10 ms but is being called twice every 20 ms. Setting the timer resolution to 1 ms didn't help. I hacked the mmtimer to run at the requested 10 ms interval and the playback problems disappeared. Since the mmtimer thread uses WaitForSingleObject(sleep_time) for it's timer source, has something changed recently which would explain this behavior?

1 0

Problems with wine-patches
by Paul Vriens Feb. 10, 2005

Feb. 10, 2005

Hi, I'm not able to send (what I think is normal) email to wine-patches anymore. The Test mail worked. Any ideas? Cheers, Paul.

7 10

Text Alignment, how is it handled ?
by George Ginden Feb. 10, 2005

Feb. 10, 2005

Hi the following code part is not working in Wine as it work (as supposed) in Windows. HRight = GetDlgItem (hDlg,IDC_RIGHT); Edit_SetText(HRight, "Right"); How is text alignment, in a edit field, handled ? It looks like the Edit_SetText function is mapped to SetWindowText, which calls X11DRV_SetWindowText... But again, text alignment doesn't work. Can I do something ? PS: c source code is attacched Regards. /* ** Ditta..........:TeamSystem spa ** Programmatore..:Stefano Peraldini ** Nome programma.:Test Wine ** Data...........:04/02/2005 ** */ //INCLUDE #include <windows.h> #include <WindowsX.h> #include "resource.h" //VARIABILI GLOBALI HWND hUnaligned, HCenter, HRight; HDC hdc; //PROTOTIPI BOOL CALLBACK DlgProc (HWND, UINT, WPARAM, LPARAM); //MAIN int WINAPI WinMain (HINSTANCE hInstance, HINSTANCE hPrevInstance, LPSTR lpszCmdLine, int nCmdShow) { // Visualizzazione e controllo errore visualizzazione dialog if (DialogBox (hInstance, (LPCSTR) IDD_DIALOG1, NULL, DlgProc) == -1) MessageBox (NULL, "Errore visualizzazione dialog!!","Test Wine", MB_ICONQUESTION | MB_OK); return 0; } //DLGPROC della dialog1 BOOL CALLBACK DlgProc (HWND hDlg, UINT message, WPARAM wParam, LPARAM lParam) { //Switch per la gestione dei messaggi provenienti dalla dialog1 switch (message) { //Gestione del VM_CLOSE case WM_CLOSE: return TRUE; break; //Impostazione dialog1 case WM_INITDIALOG: { hUnaligned = GetDlgItem (hDlg,IDC_UNALIGNED); HCenter = GetDlgItem (hDlg,IDC_CENTER); HRight = GetDlgItem (hDlg,IDC_RIGHT); //Edit Box Unaligned Edit_SetText(hUnaligned, "Unaligned"); //Edit Box Center Edit_SetText(HCenter, "Center"); //Edit Box Right Edit_SetText(HRight, "Right"); return TRUE; } //Gestione messaggi dialog1 case WM_COMMAND: switch (LOWORD (wParam)) { case IDOK: EndDialog (hDlg, 0); return TRUE; break; // END WMCOMMAND } return FALSE; } return FALSE; }

4 4

Re: RESEND: dlls/comctl32/treeview.c: remove repainting
by Robert Shearman Feb. 10, 2005

Feb. 10, 2005

yuxdwa702(a)sneakemail.com wrote: > Changelog: > tinus <yuxdwa702(a)sneakemail.com> > Scroll instead of repainting when expanding/collapsing trees > Don't repaint on hover if 'hot tracking' isn't on > > > Index: dlls/comctl32/treeview.c > =================================================================== > RCS file: /home/wine/wine/dlls/comctl32/treeview.c,v > retrieving revision 1.164 > diff -u -p -r1.164 treeview.c > --- dlls/comctl32/treeview.c 14 Jan 2005 15:13:24 -0000 1.164 > +++ dlls/comctl32/treeview.c 31 Jan 2005 01:51:30 -0000 > @@ -631,7 +631,8 @@ TREEVIEW_SendCustomDrawItemNotify(TREEVI > uItemState |= CDIS_SELECTED; > if (wineItem == infoPtr->selectedItem) > uItemState |= CDIS_FOCUS; > - if (wineItem == infoPtr->hotItem) > + /* I'm not sure of this, needs to be compared to the real thing */ Usually you find this out before submitting a patch. > + if ((infoPtr->dwStyle & TVS_TRACKSELECT) && wineItem == > infoPtr->hotItem) > uItemState |= CDIS_HOT; > @@ -5037,11 +5107,13 @@ TREEVIEW_KeyDown(TREEVIEW_INFO *infoPtr, > static LRESULT > TREEVIEW_MouseLeave (TREEVIEW_INFO * infoPtr) > { > - if (infoPtr->hotItem) > + HTREEITEM orgItem = infoPtr->hotItem; > + infoPtr->hotItem = NULL; > + > + /* remove hot effect from item */ > + if (infoPtr->dwStyle & TVS_TRACKSELECT) > { > - /* remove hot effect from item */ > - InvalidateRect(infoPtr->hwnd, &infoPtr->hotItem->rect, TRUE); > - infoPtr->hotItem = NULL; > + InvalidateRect(infoPtr->hwnd, &orgItem->rect, TRUE); > } > return 0; > } If hot tracking isn't required when TVS_TRACKSELECT isn't set, then don't call TrackMouseEvent, instead of putting these exceptions everywhere. > @@ -5080,11 +5152,11 @@ TREEVIEW_MouseMove (TREEVIEW_INFO * info > if (item != infoPtr->hotItem) > { > /* redraw old hot item */ > - if (infoPtr->hotItem) > + if (infoPtr->hotItem && (infoPtr->dwStyle & TVS_TRACKSELECT) ) > InvalidateRect(infoPtr->hwnd, &infoPtr->hotItem->rect, > TRUE); > infoPtr->hotItem = item; > /* redraw new hot item */ > - if (infoPtr->hotItem) > + if (infoPtr->hotItem && (infoPtr->dwStyle & TVS_TRACKSELECT) ) > InvalidateRect(infoPtr->hwnd, &infoPtr->hotItem->rect, > TRUE); > Same here. Don't change the hot item if the hot item isn't going to be repainted anyway. Rob

1 0

Re: rsaenh: CPGetProvParam's PP_ENUMCONTAINERS
by Alexandre Julliard Feb. 9, 2005

Feb. 9, 2005

Michael Jung <mjung(a)iss.tu-darmstadt.de> writes: > Changelog: > Implemented CPGetProvParam's PP_ENUMCONTAINERS parameter type > Added corresponding test > Removed some tabs that slipped in with the last patch The test fails for me: rsaenh.c:1383: Test failed: 00000103 rsaenh.c:1391: Test failed: 0, 00000103 -- Alexandre Julliard julliard(a)winehq.org

2 2

Re: [Discuss] Recycle Bin problem with Preclick image editing software
by Mike Hearn Feb. 9, 2005

Feb. 9, 2005

On Wed, 2005-02-09 at 15:34 -0500, Don Feliciano wrote: > SUCCESS! Woo Hoo! You rock... now my wife can't tell me b*tch at me > for ditching Windows. Thanks a ton. Niceness Don. Of course, it'd rock more if this had worked out of the box. I expect we're missing some shell magic for the recycle bin. Wouldn't be hard to investigate but have a million other things to do so I'll CC this to wine-devel. Context is: app has a "send to recycle bin" option to delete photos. Doesn't work. Native shell32/shfolder fixes it. If anybody is up for what is probably an easy task, figure out why and fix it so c:\Recycled is used. For bonus points we should bridge it to KDE/GNOME so if an app recycles something it appears in your actual trash can :) thanks -mike

3 2

Re: Rearrange WINPROC_CallWndProc to give clearer tracing
by Alexandre Julliard Feb. 9, 2005

Feb. 9, 2005

Mike Hearn <mh(a)codeweavers.com> writes: > Yes, I realised the stack allocation might be a problem but only a while > after I sent the patch. Here's another that uses the heap. I don't think you want that either, it won't really make things less confusing if you are adding extra heap calls. You may want to look into using RelayFromExclude instead. -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: Rearrange WINPROC_CallWndProc to give clearer tracing
by Alexandre Julliard Feb. 9, 2005

Feb. 9, 2005

Mike Hearn <mh(a)codeweavers.com> writes: > ChangeLog: > Rearrange WINPROC_CallWndProc to give clearer tracing > > --- windows/winproc.c (revision 132) > +++ windows/winproc.c (local) > @@ -408,20 +408,25 @@ static LRESULT WINPROC_CallWndProc( WNDP > { > LRESULT retvalue; > int iWndsLocks; > + char msgname[1024]; You don't want to do that, window procs can be nested quite deeply. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: x86_64 multilib configure fix
by Alexandre Julliard Feb. 9, 2005

Feb. 9, 2005

Alex Woods <wine-devel(a)giblets.org> writes: > Changelog: > * configure.ac: Alex Woods <wine-devel(a)giblets.org> > Handle host_cpu of x86_64 like other x86s. This will break the 64-bit build. -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: More complete stubs for DuplicateTokenEx and NtAdjustPrivilegesToken
by Rob Shearman Feb. 9, 2005

Feb. 9, 2005

Dmitry Timoshkov wrote: >Hello, > >Changelog: > Dmitry Timoshkov <dmitry(a)codeweavers.com> > More complete stubs for DuplicateTokenEx and NtAdjustPrivilegesToken. > > I have an even more complete patch for this that implements the necessary server calls. Alexandre, please don't apply. Rob

1 0