December 2006 - Wine-Devel - list.winehq.org

What happened to the Fedora packages?
by Neal Gompa Aug. 2, 2014

Aug. 2, 2014

What happened to the Fedora packages? They have not been updated since 0.9.2!!!! Right now it is at 0.9.10!!! Nearly every other Linux distro supported has the up to date packages!!! And why does the Red Hat packages site not go to the SourceForge site as it does for SUSE packages and the others?? I have not really had the guts to ask until now, because I thought that maybe there was a slump, but now, its getting annoying!! And Fedora just released Fedora Core 5 yesterday!!! Please tell me new packages will be ready soon!!! Compiling WINE always crashes my computer, so I prefer to use the RPMs...

10 18

winspool: ERROR_INVALID_NAME in GetDefaultPrinter() ?
by Detlef Riekenberg Oct. 9, 2013

Oct. 9, 2013

Hi. >From which configuration does the "ERROR_INVALID_NAME" came from, when calling GetDefaultPrinter(NULL, &size) and no Printer is installed? This Test is Present in the current "dlls/winspool/tests/info.c". MSDN told us, that we receive an "ERROR_FILE_NOT_FOUND", if no Printer is installed: http://msdn.microsoft.com/library/en-us/gdi/prntspol_0hma.asp I get the "ERROR_FILE_NOT_FOUND" on win98se, winme, w2k and win2003 in this Situation. -- By By ... ... Detlef

3 3

The status of the wintab dll
by Robert North Dec. 24, 2007

Dec. 24, 2007

-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 I think after the 10 or more patches to the Wintab dll that I submitted last month, I should say something about it's status... And of course thank Alexandre for applying those patches. P.S. I won't be available from the 15th for about a week or so. So, if you have any questions, I'm afraid you might have to be patient. Hope this is of interest to someone -Rob. ******Applications: Current status*********** ***In Painter 5 *Cursor pressure works. (Therefore is usable by most) *Cursor orientation is a little odd: The orientation maths needs to be re-done. *No eraser. Haven't yet cracked what enables the eraser. *Doesn't work in desktop mode: Need to map to desktop coordinates. *Repeats windows bugs where cursor looses pressure/orientation info almost bug for bug (Is this a feature? ;-) *Cannot detect pressure/orientation int the "Brush Tracking" window: The tablet context is attached to the main window, so no events get to the popup, even if they overlap. This is not how windows wintab functions. ***In Photoshop 6. *Can only get tablet data in desktop mode: This is because the tablet context is attached to the desktop. Which generates/receives no wine events outside desktop mode. * Eraser and pen pressure working. *But* to get them working, you must have 3 XInput devices listed in your XF86Config file, They need to be the last entries in the "ServerLayout" section and the following order: eraser, tablet mouse. This is a far from ideal way of specifying the devices Wintab should use :-/ I'll document this if someone can point me to a good place to put the docs. *******To Do************* 1. Look at X11 errors. There appear to be some errors that deny some users the ability to access Wintab enabled apps. (I think I know how to fix this) 2. Improve orientation data. Orientation comes in as X-Y coords (Implicit Z), and has to leave as spherical coords. This calculation needs to be re-done. 3. When tablet context is on top, let it read XInput events from all the app's top-level windows. (This simulates the fact that the context is usually designed to cover the whole screen) 4. When tablet context is attached to desktop, read XInput events from all the app's top-level windows. 5. Tests ~ --My current philosophy on tests is... ~ Use Photoshop & Painter, any formal tests ~ can be written if anyone else gets involved in patching Wintab, to avoid regressions, and conflict. **********Long term to do (Anyone interested?):********* There's a lot of work that could be done here, but what gets done and who does it probably depends upon whether anyone finds an app that needs these features. I'd love to implement these, but realistically, I don't foresee doing this unless someone hires me to do so ;) 1. Improve configuration of wintab. Wintab could probably do with some information entered into the config file, to avoid the user having to hack their Xfree86cfg file. 2. Handle Z-Order of context properly. This entails *sharing Z-Order between apps. *Working out exactly what role windows have in determining tablet context z-orders. *Allow tablet contexts that don't cover the whole screen/tablet. *Handle inter-application clipping of tablet contexts *Allow all application's windows to receive tablet events when tablet context is on top 3. Implement non-system tablet contexts (Where system cursor not moved by pen or mouse) 4. Unicodify 5. Implement various wintab extensions. 6. Implement wintab manager functions. 7. Tests. *********Unknowns********* 1. How are wintab contexts are raised lowered? Contexts have their own z-order independent of windows, and their own viewport concept, based upon the tablet's coordinate system, not that of the OS. It appears that entering, or clicking on the window the tablet context is attached to will raise/lower the context. But I haven't done much testing on this. In particular, what happens if more than one app request their tablet context is attached to the desktop? 2. How Painter detects the eraser. Have 3 possibilities i. Windows can detect an eraser, and sends specific messages. (I'm sure I've seen this, but can't work out where!) ii. Only works if tablet and cursors are named correctly. (Probably linked to wacom tablets only). iii. I've missed something 3. Requirements of other applications -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.2-rc1-SuSE (GNU/Linux) Comment: Using GnuPG with Thunderbird - http://enigmail.mozdev.org iD8DBQFCC8zI2vfwxdLxwWYRAsisAJ4q2gAYTgRc6f9wDI+Ruv943eDxOQCfcl3s /ZKMUGwQOuw/SIIbOkIUbd0= =R4M7 -----END PGP SIGNATURE-----

2 1

Updated version of Wine-0.9.18 RPMs with ntoskrnl for SuSE 10.1
by Vitaliy Margolen Feb. 19, 2007

Feb. 19, 2007

Same as before. Just recompiled with new version. http://www.kievinfo.com/wine-0.9.18-1.2.i586.rpm http://www.kievinfo.com/wine-0.9.18-1.2.src.rpm -- Best regards, Vitaliy

3 3

Font issue - workaround found
by Timm Feb. 18, 2007

Feb. 18, 2007

My one win app that makes me a devotee of WINE is TablEdit (demo at www.tabledit.com/download/tabled32.exe) has a custom font , tef260.ttf, that has failed to display since somewhere after ver 9.6. The program installs the font into c:\windows\font, but can not display it within the program, instead displaying for the most part, empty squares. I tried putting/removing tef260.ttf into /usr/local/share/wine/fonts, /usr/local/share/fonts, and the wine windows/fonts directories in all permutations. The only way I got it to display was to remove ALL fonts from these three directories and install tef260.ttf into an X11 font directory along with other win ttf fonts. This forces WINE to build a cachedmetrics file like it used to, and the font is displayed properly in the program. Hope this is a clue for some developer, as I merely lurk here, looking for clues about wine behavior. I can't do a regression back to ver 9.6 (the version I was using that last worked) because my upgrades to freetype cause it to complain of missing functions. current (pertinent ? ) specs libfreetype.so.6.3.7 ] built from source libfontconfig.so.1.0.4 ] built from source fontforge 20060715 ] built from source WINE 0.9.20 ] built from source kernel 2.6.17.11 Slack current (10.2, with upgrades from "current" tree) -- Timm

3 2

Re: Bug 50
by Pedro Araújo Chaves Jr. Feb. 13, 2007

Feb. 13, 2007

I've been trying to get the lpDx array to be properly calculated, but unfortunately to no avail. I did find, though, that GetTextExtentExPointW is never called with a maxExt value other than zero, and thus nfit is not calculated. Regarding that, would any of you know where I should look for the proper method of calculating the maximum width (maxExt) of a formatted string? I'm somewhat puzzled... neither lprect nor dc->BoundsRect seem to hold the proper values for getting that value [1], or (more likely, I guess) I'm just looking at it the wrong way. Btw, I'd add that I've been testing this in Lotus Notes R5, just in case it helps, and no, I haven't found a link to download a trial. [2] [3] Cheers, Pedro. [1] I'm investigating the SetTextJustification, ExtTextOutW, GetTextExtentPointW, GetTextExtentPoint32W, GetTextExtentExPointW, and WineEngGetTextExtentExPoint functions from gdi32 (the latter in freetype.c, the rest in font.c). I am yet to look into their *I counterparts, where applicable. [2] http://appdb.winehq.org/viewbugs.php?bug_id=50 [3] http://bugs.winehq.org/show_bug.cgi?id=7016

2 3

Re: configure: add ncursesw detecting [try 2]
by Alexandre Julliard Jan. 12, 2007

Jan. 12, 2007

Vitaly Lipatov <lav(a)etersoft.ru> writes: > Try 2, with ncursesw/ncurses detecting > In order to build wineconsole run properly in utf8 locale, we use ncursesw, > which can handle wchar strings. Why would you need wchar strings if you use utf-8? Have you configured your locale properly? -- Alexandre Julliard julliard(a)winehq.org

4 14

Looking for sound testers
by Maarten Lankhorst Jan. 11, 2007

Jan. 11, 2007

Hi all, I've forward ported the old patches of Davin McCall (dsound.patch). With them I have no more sound underruns etc, I'm therefore looking for other people to test them as well. I'm welcoming comments on how it works and possible issues with it. I also attached my patch to alsa, I'm aware of the things I need to fix for that, but I thought it might be good to test dsound with winealsa, to see if other things still need to be fixed. Cheers, Maarten

6 5

appdb rating inflation
by Dan Kegel Jan. 6, 2007

Jan. 6, 2007

The appdb says "Only applications which install and run flawlessly on an out-of-the-box Wine installation make it to the Platinum list" But several platinum-rated apps seem to deserve a silver or bronze rating. For instance, Call to Duty, http://appdb.winehq.org/appview.php?iVersionId=3603 has all sorts of caveats: - Punkbuster enabled Multiplayer will not work - Sound out of sync. - Runs without crashes, but only if installed as root - requires whacky third-party Loki installer; real installer fails on cd change (http://bugs.winehq.org/show_bug.cgi?id=6594) - "with alsa, the sound lags ~ 1 second behind the game.... if i choose oss i got the 166 sound files missing error" And Diablo II, http://appdb.winehq.org/appview.php?iVersionId=49, also has lots of complaints / caveats / complicated howto's. Should we impose some standards on appdb ratings? - Dan -- Wine for Windows ISVs: http://kegel.com/wine/isv

19 39

Re: systray icon questions
by Kirill K. Smirnov Jan. 2, 2007

Jan. 2, 2007

> Unfortunately, yes. The way the helper window is currently implemented, > if you remove the WS_CAPTION style, the icon won't be centered when it > gets docked (in GNOME at least.) I tried that when I put together the > systray tooltips patch. Ultimately it caused more problems than it > solved, so I gave up on it. But, it would be really great if you could > figure it out. :) I've tried kde, fluxbox and xfce4 (and fvwm, but it lacks systray). Now I can see that xfce4 suffers from it. I did not noticed that before. So, I suppose a new patch (attached). Does it work correct with GNOME? In xfce is OK. Thanks for pointing the path! -- Kirill K. Smirnov

2 1

Reason for some process failures in kernel32 tests
by Paul Vriens Jan. 1, 2007

Jan. 1, 2007

Hi, I've been trying to tackle some of the issues we have when trying to run some process tests (through winetest.exe, see also http://test.winehq.org/data) process.c:775: Test failed: CreateProcess (./kernel32_test.exe tests/process.c C:\TEMP\wt44.tmp "a\"b\\" c\" d) failed : 2 process.c:777: Test failed: Child process termination process.c:781: Test failed: Arguments:argvA0 expected './kernel32_test.exe', got '(null)' process.c:790: Test failed: CreateProcess (.\kernel32_test.exe tests/process.c C:\TEMP\wt45.tmp "a\"b\\" c\" d) failed : 2 process.c:792: Test failed: Child process termination process.c:796: Test failed: Arguments:argvA0 expected '.\kernel32_test.exe', got '(null)' process.c:810: Test failed: CreateProcess (..\wct12/kernel32_test.exe tests/process.c C:\TEMP\wt46.tmp "a\"b\\" c\" d) failed : 2 process.c:812: Test failed: Child process termination process.c:816: Test failed: Arguments:argvA0 expected '..\wct12/kernel32_test.exe', got '(null)' The reason for this particular one is that I'm running winetest.exe started from a command-file on my desktop. This means that the current directory is the desktop directory. The winetest gui shows that the working directory is (in this case) c:\temp\wct12. This value is however not passed to the tests when CreateProcess is run: main.c: 291 if (!CreateProcessA (NULL, cmd, NULL, NULL, TRUE, 0, 292 NULL, NULL, &si, &pi)) { The above means that the kernel32_test.exe has a working directory that is not c:\temp\wct12 (in this case again) and thus cannot find the executable when referenced relatively. All the test-results that are shown on http://test.winehq.org show more-or-less the same behavior. How can I retrieve the correct working directory (as shown in the winetest gui) and pass that to the single tests? (I think that would solve these particular issues). Cheers and thanks for any pointers, Paul.

3 5

OpenGL as child windows
by Chris Robinson Dec. 31, 2006

Dec. 31, 2006

The topic of getting OpenGL to show and use more visuals/pixel formats than just the one the main window uses came up on IRC, and I was told to make a post here. One of the ideas was to create a child window of the main X window (which can be given a different visual) and have OpenGL use that for drawing. A simple test program I wrote (using X) seems to behave well. The child window stays relative to the parent, it remains fully connected (even with the wobbly effects of compositors like Beryl), and it can use a different visual. Since wgl is now a part of x11drv, it should be possible to make an X11 child window. Are there any problems with using this approach in Wine?

7 9

Re: Resend: user32: fix return value in DialogBoxParamA if no resourcecan be found + simple test (try 2)
by Dmitry Timoshkov Dec. 31, 2006

Dec. 31, 2006

"Louis. Lenders" <xerox_xerox2000(a)yahoo.co.uk> wrote: > As Dmitry Timoshkov (thanks for that) the SetLastError was not necessary. > Here's updated patch. Probably the problem is that you need also handle both FindResourceW and LoadResource failures the same way by returning -1, and change 'return -1;' later to 'return 0;' (for invalid parent) as MSDN suggests. All that needs to be tested as well. Also DialogBoxParamW needs to be fixed as well. Probably other dialaog APIs (like DialogBoxIndirectParamAorW) need a review regarding that. -- Dmitry.

1 0

Re: ntdll: Map ESPIPE to STATUS_ILLEGAL_FUNCTION
by Andreas Mohr Dec. 31, 2006

Dec. 31, 2006

Hi, On Sun, Dec 31, 2006 at 05:28:23PM +0800, Dmitry Timoshkov wrote: > Hello, > > Running RedAlert demo spits a lot of FIXMEs about converting errno 29 > (ESPIPE) to STATUS_UNSUCCESSFUL. In Linux /usr/include/asm/errno.h has > a comment /* Illegal seek */ for ESPIPE, so STATUS_ILLEGAL_FUNCTION seems > to be the best choice. > > Changelog: > ntdll: Map ESPIPE to STATUS_ILLEGAL_FUNCTION. The STATUS_ILLEGAL_FUNCTION name was rather misleading to me since it doesn't indicate at all that it's supposed to be pipe-related (however it's within the numeric range of many other pipe status codes!). So, given that this is mainly a pipe status code after all I'd say that's the best we can achieve right now. Of course, what really matters is what the receiving Win32 side *usually* expects after we encountered ESPIPE, and this might turn out to differ from your conversion ;). (is there any indication that RedAlert checks for a specific status code??) Andreas Mohr

2 1

Re: X11DRV: remove DesktopDoubleBuffered option
by Dmitry Timoshkov Dec. 31, 2006

Dec. 31, 2006

"Roderick Colenbrander" <thunderbird2k(a)gmx.net> wrote: > This patch removes the DesktopDoubleBuffered option. Support for this option should be removed from winecfg simultaneously with this change. -- Dmitry.

1 0

problems with installed wine
by Corey McClymonds Dec. 31, 2006

Dec. 31, 2006

for almost a week now, wine has been acting weird. I compile wine and using it from the wine source tree, everything works perfectly. But if I make install it, then try to use it, all I get is a black screen that I have to alt-tab to get out of. On a note, it works fine if I run as root. I have blown away my .wine directory, it doesn't help. I compile from git. Corey McClymonds

2 1

Re: Resend: msxml.dll: nextNode and reset functions
by James Hawkins Dec. 30, 2006

Dec. 30, 2006

On 12/28/06, Colin Pitrat <colin.pitrat(a)bull.net> wrote: > Hi, > > here is a patch I sent a few weeks ago implementing two functions in > msxml.dll, and the corresponding test. > I suggest sending the tests in one patch first, then send each function in a separate patch afterwards. -- James Hawkins

1 0

Re: Vijay Kiran Kamuju : comctl32: MonthCalendar - Fix highlighting of current date.
by Duane Clark Dec. 30, 2006

Dec. 30, 2006

Alexandre Julliard wrote: > Module: wine > Branch: master > Commit: c5b8df481f2bb0362b57188a8c7df423030ba8b0 > URL: http://source.winehq.org/git/wine.git/?a=commit;h=c5b8df481f2bb0362b57188a8… > > Author: Vijay Kiran Kamuju <infyquest(a)gmail.com> > Date: Sun Dec 24 13:31:56 2006 +0530 > > comctl32: MonthCalendar - Fix highlighting of current date. That does highlight the current date on startup. But I notice that when I click another date, the current date doesn't get unhighlighted.

5 6

Re: riched20: block recursive notification
by Matt Finnicum Dec. 30, 2006

Dec. 30, 2006

On 12/30/06, Clinton Stimpson <cjstimpson(a)utwire.net> wrote: > > This patch blocks riched20 from recursivly sending an EN_CHANGE > notification. > Fixes bug 6985. > > Thanks, > Clinton Stimpson > > ChangeLog > Block recursive EN_CHANGE notifications. If you're adding a new variable to the ME_TextEditor structure then you should probably also initialize it in ME_MakeEditor. Naming the variable "notify" is somewhat ambiguous, and there seems to be an established convention of naming int's with the 'n' prefix, like nNotifiy. You should probably comment to explain what the variable is for or attempt to name it something more self evident. Maybe a BOOL would be more appropriate, unless you're intending to extend this to other notify calls as well. Lastly, I don't see why this patch is needed - it only prevents the ME_SendOldNotify (which just calls SendMessageA) from triggering another call of that same EN_CHANGE notification. I don't see why calling SendMessageA would cause ME_UpdateRepaint to be called. Maybe the application you're working with, in processing the EN_CHANGE message, is sending another message to which we wrongly are sending an EN_CHANGE notification - so thus we should fix our message handling of that. A quick look at MSDN shows that "The EN_CHANGE notification message is sent when the user has taken an action that may have altered text in an edit control." - it seems that we may be sending them whenever any action (user or not) has altered text in the control. We really could use some tests to see what should generate EN_CHANGE messages. Another possibility is that we don't handle the case where (again, from MSDN) "The EN_CHANGE notification is not sent when the ES_MULTILINE style is used and the text is sent through WM_SETTEXT." - perhaps your application has ES_MULTILINE set and uses a WM_SETTEXT in response to the EN_CHANGE notification? Thanks for your help with richedit, --Matt Finnicum

1 0

Re: wine-patch OleIconToCursor function
by Marcus Meissner Dec. 30, 2006

Dec. 30, 2006

On Sat, Dec 30, 2006 at 04:49:27AM +0100, Bernard Ladenthin wrote: > Hello, > in the attachment is a small patch for the OleIconToCursor function. > The function works with this. I hope i can help with this and get a > response mail that this mail have received. I haven't any experiences > with git so it's a simply patch. > > I have tested it with the "Lexware Buchhalter 2005". > You can download it from http://server.ladenthin.net/bh2005.zip to see > that it works. Start with wine the "LxStart.exe" and turn over the > Text and Icon with the mouse, the icon is now the cursor. > > Best regards from Germany, > Bernard Ramon Ladenthin In general you do no need to edit Changelog yourself. Just add a note above the patch. Otherwise it looks fine to my eyes (and perhaps Alexandres too), thanks! Ciao, Marcus > diff --git a/ChangeLog b/ChangeLog > index 3b536b5..9723401 100644 > --- a/ChangeLog > +++ b/ChangeLog > @@ -1,3 +1,8 @@ > +2006-12-30 Bernard Ramon Ladenthin <bernard(a)ladenthin.net> > + > + * dlls/oleaut32/oleaut.c, dlls/oleaut32/stubs.c: > + oleaut32: Implement the function OleIconToCursor and remove the stub. > + > 2006-12-22 Dmitry Timoshkov <dmitry(a)codeweavers.com> > > * dlls/user32/tests/msg.c, dlls/winex11.drv/winpos.c: > diff --git a/dlls/oleaut32/oleaut.c b/dlls/oleaut32/oleaut.c > index d6a08a9..65a4174 100644 > --- a/dlls/oleaut32/oleaut.c > +++ b/dlls/oleaut32/oleaut.c > @@ -837,3 +837,19 @@ BOOL WINAPI DllMain(HINSTANCE hInstDll, > > return TRUE; > } > + > +/*********************************************************************** > + * OleIconToCursor (OLEAUT32.415) > + */ > +HCURSOR WINAPI OleIconToCursor( HINSTANCE hinstExe, HICON hIcon) > +{ > + FIXME("(%p,%p), (olepro32.dll), partially implemented.\n",hinstExe,hIcon); > + if (hIcon) > + return CopyCursor(hIcon); > + else > + return NULL; > + > + /* FIXME > + make a extended conversation from HICON to HCURSOR > + */ > +} > diff --git a/dlls/oleaut32/stubs.c b/dlls/oleaut32/stubs.c > index ac2d494..d87e76e 100644 > --- a/dlls/oleaut32/stubs.c > +++ b/dlls/oleaut32/stubs.c > @@ -33,15 +33,6 @@ > WINE_DEFAULT_DEBUG_CHANNEL(ole); > > /*********************************************************************** > - * OleIconToCursor (OLEAUT32.415) > - */ > -HCURSOR WINAPI OleIconToCursor( HINSTANCE hinstExe, HICON hicon) > -{ > - FIXME("(%p,%p), not implemented (olepro32.dll)\n",hinstExe,hicon); > - return S_OK; > -} > - > -/*********************************************************************** > * OleCreatePropertyFrameIndirect (OLEAUT32.416) > */ > HRESULT WINAPI OleCreatePropertyFrameIndirect( LPOCPFIPARAMS lpParams) >

2 1

Re: USB
by Damjan Jovanovic Dec. 30, 2006

Dec. 30, 2006

On 12/30/06, Andrew Neil Ramage <nrsc16850(a)blueyonder.co.uk> wrote: > A franklin eBookMan using the provided Windows Software. (By the way, it's bad practice to top-post bottom-posted emails.) What interface does the device provide? Is it a mass-storage device? Or some vendor-specific protocol? Do "lsusb -v" and see the class and subclass -if they're 0xFF (vendor-specific), you're probably out of luck, because: If the Windows software includes .SYS files, wine can't load those yet (although wine's NTOSKRNL.EXE is coming soon, right?) Since wine is largely designed for running Windows applications on Linux, a lot of the hardware support infrastructure (SETUPAPI, CFGMGR, ...) is still missing. At present, a Windows app cannot directly access the hardware, only indirectly (for example, it can save a file to a mass-storage device supported by Linux, or communicate through a USB ethernet controller supported by Linux). Now in Windows, applications don't access USB directly either (Windows only allows USB control packets from user-space, not bulk, interrupt or isochronous, and no app I know uses that interface anyway). What apps usually do is open a .SYS device driver (eg. \\.\USBSCAN0\2) and use ReadFile to send bulk data, WriteFile to receive bulk data, and DeviceIoControl for everything else. So in theory if you figure out the interface between the application and the driver in Windows, you can write a kernel module for Linux that does the same, and your app can communicate with the hardware. Wine currently doesn't do this at all, although I've written a semi-complete implementation of STI and USBSCAN.SYS this way, and it works with native TWAIN drivers. Damjan > Andrew > > > On 12/30/06, Andrew Neil Ramage <nrsc16850(a)blueyonder.co.uk> wrote: > >> How do you get a Windows program access to the USB port ? > > > > What are you trying to do? What device are you accessing? > > > >> Andrew > > > > Damjan > > > > > > > -- > Andrew Ramage > > >

1 0

USB
by Andrew Neil Ramage Dec. 30, 2006

Dec. 30, 2006

How do you get a Windows program access to the USB port ? -- Andrew "Oh, bother," said the Borg. "We've assimilated Pooh."

2 1

windows web server app fails to send more than 11680 bytes !?
by [IDC]Dragon Dec. 30, 2006

Dec. 30, 2006

Hello, I'm testing a windows app I'd very much like to run under Wine (using v0.9.22), being in touch with the author. This is a small web server app based on a Microsoft MFC sample app. It happily works under Windows, but that'd be the wrong thing for running it 24/7. ;-) It's not yet working under Wine, fails to serve larger files, but we've pretty much narrowed down the problem by adding logs to the app and using ethereal. For example, the app tries to write 25017 bytes to the socket, but the socket only allows 11680 bytes at a time, and the TCP stack fragments this in 8 packets of 1460 bytes, presumably because it only has 8 buffers reserved. So far so good. We see them being sent on the Ethernet and the ACK's are coming in, but after these 8 packets there should be a signal to trigger the OnSend function of the app so it can send the next part of data, but this never happens... The reselection of the socket did not return an error. Given the reselection is ok, and at the Ethernet level messages are consistently sent and ACK's received, the conclusion is that there might be a bug in Wine. So here we are stuck. Given that Wine successfully runs real big application such as MS Internet Explorer, I can't imagine it to have such a fundamental problem. Is there something I overlooked, like a configuration issue? If it really gets interesting, I can ask the developer for a more detailed report, including the Ethereal log snippet, in case somebody wants to actually look into it. First I'd like to find out if this is the right place, or if I should "bury" it into bugzilla. Thank you for your time and help, Jörg -- GMX DSL-Flatrate 0,- Euro* - Überall, wo DSL verfügbar ist! NEU: Jetzt bis zu 16.000 kBit/s! http://www.gmx.net/de/go/dsl

5 9

Budgetdedicated mirror down
by Kai Blin Dec. 30, 2006

Dec. 30, 2006

Hi folks, at the time of writing, I'm unable to get a page back from the budgetdedicated mirror. Could someone who has access to that box take a look at that? Cheers, Kai -- Kai Blin, <kai Dot blin At gmail Dot com> WorldForge developer http://www.worldforge.org/ Wine developer http://wiki.winehq.org/KaiBlin/ -- Will code for cotton.

1 0

Re: [user32][msgbox] Disable IDCANCEL for some MessageBoxs
by Dmitry Timoshkov Dec. 30, 2006

Dec. 30, 2006

"Anatoly Lyutin" <vostok(a)etersoft.ru> wrote: > @@ -129,9 +132,11 @@ static HFONT MSGBOX_OnInit(HWND hwnd, LP > ShowWindow(GetDlgItem(hwnd, IDCANCEL), SW_HIDE); > ShowWindow(GetDlgItem(hwnd, IDYES), SW_HIDE); > ShowWindow(GetDlgItem(hwnd, IDNO), SW_HIDE); > + bUseCancel = FALSE; > break; > case MB_YESNO: > ShowWindow(GetDlgItem(hwnd, IDCANCEL), SW_HIDE); > + bUseCancel = FALSE; It would be much simpler to use IsWindowVisible(GetDlgItem(hwnd, IDCANCEL)) where appropriate instead of introducing bUseCancel. > /* fall through */ > case MB_YESNOCANCEL: > ShowWindow(GetDlgItem(hwnd, IDOK), SW_HIDE); > @@ -147,6 +152,11 @@ static HFONT MSGBOX_OnInit(HWND hwnd, LP > ShowWindow(GetDlgItem(hwnd, IDNO), SW_HIDE); > break; > } > + if(!bUseCancel) > + { > + SetWindowLongW(hwnd,GWL_STYLE,GetWindowLongW(hwnd,GWL_STYLE)&~(WS_SYSMENU)); > + } > + SetPropA(hwnd, "WINE_MSGBOX_BUSECANCEL", (HANDLE)bUseCancel); What is the purpose of removing WS_SYSMENU? In any case this change requires a test case. -- Dmitry.

1 0

Building a full win32 winetest.exe
by Paul Vriens Dec. 29, 2006

Dec. 29, 2006

Hi, any pointers how to build a winetest.exe I can run on Windows? All tests (but crypt32 and hlink) can be cross compiled (MinGW) but only give me the single test executable for that dll. Cheers, Paul.

3 4

Removing unused functions and variables
by Andrew Talbot Dec. 29, 2006

Dec. 29, 2006

Hi, I am intending to send in a few patches to remove some unused Wine variables and functions from the dlls. Please accept that my good faith is tempered by a degree of ignorance, and advise me of suitable markup if any seemingly redundant piece of code I seek to remove must remain for a legitimate reason. Thanks, -- Andy.

1 0

[4/4] wined3d: Create samplers for *all* 1.x instructions that sample
by Christoph Bumiller Dec. 29, 2006

Dec. 29, 2006

Note that WINED3DSIO_TEXDP3TEX, WINED3DSIO_TEXM3x3VSPEC, WINED3DSIO_TEXM3x3SPEC are already handled in the 2 else if branches below, they should probably be removed somewhere. > @@ -295,8 +295,15 @@ HRESULT shader_get_registers_used( > if > (WINED3DSHADER_VERSION_MAJOR(This->baseShader.hex_version) == 1 && > (WINED3DSIO_TEX == curOpcode->opcode || > WINED3DSIO_TEXBEM == curOpcode->opcode || > + WINED3DSIO_TEXBEML == curOpcode->opcode || > + WINED3DSIO_TEXDP3TEX == curOpcode->opcode || > WINED3DSIO_TEXM3x2TEX == curOpcode->opcode || > - WINED3DSIO_TEXM3x3TEX == curOpcode->opcode)) { > + WINED3DSIO_TEXM3x3SPEC == curOpcode->opcode || > + WINED3DSIO_TEXM3x3TEX == curOpcode->opcode || > + WINED3DSIO_TEXM3x3VSPEC == curOpcode->opcode || ... } else if (WINED3DSHADER_VERSION_MAJOR(This->baseShader.hex_version) == 1 && (WINED3DSIO_TEXM3x3SPEC == curOpcode->opcode || WINED3DSIO_TEXM3x3VSPEC == curOpcode->opcode)) {

2 1

Solution to Bug 3023
by Michael Gardiner Dec. 29, 2006

Dec. 29, 2006

http://bugs.winehq.org/show_bug.cgi?id=3023 I have been fiddling with Orcad 9.2 Capture and have found the cause of the problem behind its inability to place parts, text etc. It is caused by the window that the object is to be placed in temporarily losing focus (or maybe activation) after the dialog box for entering the object's properties is closed. This temporary loss of focus causes the window to "lose its grip" on the object being placed. Removing the line that calls WINPOS_ActivateOtherWindow in dlls/user32/dialog.c in the EndDialog function fixes this problem - the focus seems to immediately return to the correct window and the object can be placed. The removal of this WINPOS_ActivateOtherWindow call does not seem to have any visible side-effects in this particular instance - does it have a purpose? (In this instance I traced its call to see what it activated - it does not find a valid window to activate, so reaches the end and calls SetActiveWindow(0)).

1 0

Re: sane.ds: Remove unused WINE_DEFAULT_DEBUG_CHANNEL
by Marcus Meissner Dec. 28, 2006

Dec. 28, 2006

On Thu, Dec 28, 2006 at 09:08:51PM +0000, Andrew Talbot wrote: > Changelog: > sane.ds: Remove unused WINE_DEFAULT_DEBUG_CHANNEL. > > diff -urN a/dlls/sane.ds/ui.c b/dlls/sane.ds/ui.c > --- a/dlls/sane.ds/ui.c 2006-05-23 13:48:34.000000000 +0100 > +++ b/dlls/sane.ds/ui.c 2006-12-28 20:49:16.000000000 +0000 > @@ -37,8 +37,6 @@ > #include "wine/debug.h" > #include "resource.h" > > -WINE_DEFAULT_DEBUG_CHANNEL(twain); > - > #ifdef HAVE_SANE NAK, this code has 1 TRACE() some way below. Just move it below #ifdef HAVE_SANE. Ciao, Marcus

1 0

systray icon questions
by Kirill K. Smirnov Dec. 28, 2006

Dec. 28, 2006

Hi, all! I was playing around systray icons in various modes 1) "allow the window manager to control the windows"="on" "emulate a virtual desktop"="off" As result, systray icon is OK. 2) In other combination of these parameters, the icon is drawn with window title. I've tried to investigate the problem and created the patch (attached). But it solves the problem only in non-desktop modes. In desktop modes only window title is drawn. So the questions: 1) Does systray icon helper window really need to be created with WS_CAPTION style? 2) What does "managed" mode mean? 3) When I select "desktop" mode, should we disable the "managed" checkbox? Thanks in advance, -- Kirill K. Smirnov

2 1

Re: Building a full win32 winetest.exe
by Hans Leidekker Dec. 28, 2006

Dec. 28, 2006

On Monday 25 December 2006 11:22, Paul Vriens wrote: > These are the MinGW packages I'm currently using: > > mingw-binutils-2.16.91-9hl > mingw-w32api-3.6-18hl > mingw-gcc-core-3.4.5-13hl > mingw-runtime-3.9-18hl I think I'll get around to building a new set of RPMs this week. Lets go for a Wine 1.0 release without any failing test on XP. -Hans

1 1

Re: [1/4] wined3d: Where possible, avoid using D3DCOLORTOGLFLOAT4
by Stefan Dösinger Dec. 28, 2006

Dec. 28, 2006

Am 28.12.2006 um 03:12 schrieb H. Verbeet: > Most of the places that use D3DCOLORTOGLFLOAT4 can handle GLint's > just as well. > > Changelog: > - Where possible, avoid using D3DCOLORTOGLFLOAT4 > <wined3d_d3dcolor.diff.txt> > This patch causes a regression in Battlefield 1942, a lot of geometry is missing(ships, planes, parts of trees).

1 0

re: comctl32: MonthCalendar - Fixhighlightingof current date.
by Dan Kegel Dec. 28, 2006

Dec. 28, 2006

Dmitry wrote: > "Vijay Kiran Kamuju" <infyquest at gmail.com> wrote: >> What do we need to setup, if we want a test framework for messages in >> comctl32 similar to user32? > >Some kind of a message queue is needed and a mechanism to check what >has been gathered with an expected sequence. I'd suggest to not invent >a wheel and copy existing parts from dlls/user32/tests/msg.c. Yeah. Also I think James Hawkins is doing this now for comctl, might check with him first. - Dan -- Wine for Windows ISVs: http://kegel.com/wine/isv

1 0

Re: WGL: fix wglGetProcAddress bug
by Roderick Colenbrander Dec. 28, 2006

Dec. 28, 2006

Vitaly just pointed out that the previous version added some extra spaces. This one fixes that. Roderick > Hi, > > This patch fixes a nasty wglGetProcAddress bug. In case the name of an > unknown GL extension (one which is not in wine yet) was passed to > wglGetProcAddress it was assumed that it was a WGL extension. > > WGL extensions are in the end looked up in winex11.drv. The > wglGetProcAddress function in winex11.drv can return WGL extensions but it can also > return native GL functions. This ability is used in order to fill the opengl32 > thunks (win32 GL -> native GL). > > The problem was that unknown extensions were also passed to the > winex11.drv code. This resulted that code to look up a native GL functions (the > winex11.drv code returns wgl extensions if the function name starts with 'wgl' > and else it calls glXGetProcAddress). > > In the end this patch makes sure that only functions which start with a > 'w' are passed to the winex11.drv code in the other cases a warning is > printed that the function wasn't found. > > Regards, > Roderick Colenbrander -- Der GMX SmartSurfer hilft bis zu 70% Ihrer Onlinekosten zu sparen! Ideal für Modem und ISDN: http://www.gmx.net/de/go/smartsurfer

1 0

function_grep.pl - idutils
by Claudio Fontana Dec. 27, 2006

Dec. 27, 2006

Hello, I noticed that you are using function_grep.pl to search for a certain function. Have you considered GNU idutils? They can do that and more (and faster): http://www.gnu.org/software/idutils It is as simple as $ cd $WINE_SRC_PATH $ mkid (generate ID database, takes about 4 seconds on my machine for wine) $ gid -r PATTERN Claudio

1 0

kernel32 console bug?
by Robert Reif Dec. 27, 2006

Dec. 27, 2006

I get an error when running a windows console program on wine. It appears that in wine, GetStdHandle(STD_INPUT_HANDLE) doesn't return a handle to a console so GetConsoleMode() on that handle fails. The actual programs that fail are at: http://simh.trailing-edge.com/ Here is a simple patch to kernel32/tests/console.c to demonstrate the problem: diff -p -u -r1.2 console.c --- dlls/kernel32/tests/console.c 10 Oct 2006 18:19:57 -0000 1.2 +++ dlls/kernel32/tests/console.c 20 Dec 2006 07:20:06 -0000 @@ -552,12 +552,31 @@ static void testCtrlHandler(void) ok(GetLastError() == ERROR_INVALID_PARAMETER, "Bad error %u\n", GetLastError()); } +static void test(void) +{ + HANDLE std_input; + DWORD saved_mode; + + std_input = GetStdHandle(STD_INPUT_HANDLE); + ok(std_input != INVALID_HANDLE_VALUE, + "GetStdHandle(STD_INPUT_HANDLE) returned INVALID_HANDLE_VALUE\n"); + + if (std_input != INVALID_HANDLE_VALUE) + { + BOOL status; + status = GetConsoleMode(std_input, &saved_mode); + ok(status != FALSE, "GetConsoleMode() failed: %08x\n", GetLastError()); + } +} + START_TEST(console) { HANDLE hConIn, hConOut; BOOL ret; CONSOLE_SCREEN_BUFFER_INFO sbi; + test(); + /* be sure we have a clean console (and that's our own) * FIXME: this will make the test fail (currently) if we don't run * under X11

4 6

adding a data point to ALSA vs Wine sound discussion
by Molle Bestefich Dec. 27, 2006

Dec. 27, 2006

Adding a data point to the ALSA and Wine sound discussion. ALSA has a general problem with: * Doubling volume * Playing sound bites two times when it doesn't receive data fast enough from an application. I tried playing a video file over a network using a non-wine player, then paused the network stream. The sound paused. Then restarted the network stream. The volume now doubles for the next sound bite (always), and the next sound bite is played twice (sometimes). So, just in case anyone is hearing the above two effects, they may actually be bugs in ALSA occuring when samples are delivered slower than expected, rather than bugs in Wine.

2 1

make test failure
by Kai Blin Dec. 27, 2006

Dec. 27, 2006

Hi folks, over christmas, I decided to give make test another try. I'm running this on an openSUSE 10.1 box, ati radeon mobility x300 with open source drivers. (Probably unimportant for the errors I'm getting) My procedure to get the tests setup were: export WINEPREFIX=~/.wine-test rm -rf $WINEPREFIX wineprefixcreate winecfg (set to emulate virtual desktop, nothing else touched) make test This fails in user32, with the attached error message: Any thoughts? Cheers, Kai -- Kai Blin, <kai Dot blin At gmail Dot com> WorldForge developer http://www.worldforge.org/ Wine developer http://wiki.winehq.org/KaiBlin/ -- Will code for cotton.

2 3

Re: ole32: Marshal the ORPCTHAT structure prefixed to the server data.
by Dmitry Timoshkov Dec. 27, 2006

Dec. 27, 2006

"Robert Shearman" <rob(a)codeweavers.com> wrote: > + *hook_count = 0; > + *extension_count = 0; > + > + EnterCriticalSection(&csChannelHook); > + > + LIST_FOR_EACH_ENTRY(entry, &channel_hooks, struct channel_hook_entry, entry) > + (*hook_count)++; > + > + if (hook_count) > + *data = HeapAlloc(GetProcessHeap(), 0, *hook_count * sizeof(struct channel_hook_buffer_data)); > + else > + *data = NULL; Perhaps 'if (hook_count)' should be 'if (*hook_count)' ? Is that a typo? -- Dmitry.

2 1

Re: oleaut32: Fix failing VarMod tests in WinXP, fix VarMod(VT_DECIMAL) in Wine
by Robert Shearman Dec. 27, 2006

Dec. 27, 2006

a_villacis(a)palosanto.com wrote: > In a preparation for a possible implementation of IDispatch > tests in oleaut32, I decided to clean up the failing tests in > Windows XP: > Great, thanks for fixing this. I was about to do this myself. It's about time we get the number of failing tests on XP as close to zero as possible. -- Rob Shearman

3 2

Re: [Winmm/winealsa] Don't use asynchronous callbacks in dsound any more
by Eric Pouech Dec. 26, 2006

Dec. 26, 2006

Maarten, nice work on Alsa (which is definitively needed !!) a couple of comments: - I know the winmm code (and drivers) is crippled by bad synchronisation tricks (like cleaning a field in a structure to signal a thread). This is bad (TM). So I'd suggest using here a real synchronisation object. - I wonder if it's a good idea to create another thread for this... but before merging all existing drivers' threads into a single one, I'd suggest using the fd oriented functions & poll (see snd_pcm_poll_descriptors and friends) instead of snd_pcm_wait, that'll ease up the future work. Maarten Lankhorst a écrit : > Instead of using asynchronous callbacks that uses signals, use a > seperate thread that can be cancelled, this prevents deadlock issues. > > Basically we use snd_pcm_wait() that tells us when enough room is free > to commit another buffer, then we commit the previous buffer and make > the next buffer ready. > > Since snd_pcm_wait() uses poll(), we don't have signals in winealsa > any more. > ------------------------------------------------------------------------ > > diff --git a/dlls/winmm/winealsa/audio.c b/dlls/winmm/winealsa/audio.c > index 6043680..e1676ed 100644 > --- a/dlls/winmm/winealsa/audio.c > +++ b/dlls/winmm/winealsa/audio.c > @@ -6,6 +6,7 @@ > * Copyright 2002 Eric Pouech > * 2002 Marco Pietrobono > * 2003 Christian Costa : WaveIn support > + * 2006 Maarten Lankhorst > * > * This library is free software; you can redistribute it and/or > * modify it under the terms of the GNU Lesser General Public > @@ -70,11 +71,6 @@ WINE_DEFAULT_DEBUG_CHANNEL(wave); > > #ifdef HAVE_ALSA > > -/* internal ALSALIB functions */ > -/* FIXME: we shouldn't be using internal functions... */ > -snd_pcm_uframes_t _snd_pcm_mmap_hw_ptr(snd_pcm_t *pcm); > - > - > /* state diagram for waveOut writing: > * > * +---------+-------------+---------------+---------------------------------+ > @@ -2627,7 +2623,6 @@ #undef EXIT_ON_ERROR > snd_pcm_hw_params_free(wwo->hw_params); > wwo->hw_params = hw_params; > > - > return wodNotifyClient(wwo, WOM_OPEN, 0L, 0L); > > errexit: > @@ -3045,11 +3040,8 @@ struct IDsDriverBufferImpl > DWORD mmap_buflen_bytes; > snd_pcm_uframes_t mmap_buflen_frames; > snd_pcm_channel_area_t * mmap_areas; > - snd_async_handler_t * mmap_async_handler; > snd_pcm_uframes_t mmap_ppos; /* play position */ > - > - /* Do we have a direct hardware buffer - SND_PCM_TYPE_HW? */ > - int mmap_mode; > + HANDLE mmap_thread; > }; > > static void DSDB_CheckXRUN(IDsDriverBufferImpl* pdbi) > @@ -3076,71 +3068,66 @@ static void DSDB_CheckXRUN(IDsDriverBuff > } > } > > -static void DSDB_MMAPCopy(IDsDriverBufferImpl* pdbi, int mul) > +/** > + * The helper thread for DirectSound > + * > + * Basically it does an infinite loop until it is told to die > + * > + * snd_pcm_wait() is a call that polls the sound buffer and waits > + * until there is at least 1 period free before it returns. > + * > + * We then commit the buffer filled by the owner of this > + * IDSDriverBuffer */ > +static DWORD CALLBACK DBSB_MMAPStart(LPVOID data) > { > + IDsDriverBufferImpl* pdbi = (IDsDriverBufferImpl*)data; > WINE_WAVEDEV * wwo = &(WOutDev[pdbi->drv->wDevID]); > - snd_pcm_uframes_t period_size; > - snd_pcm_sframes_t avail; > - int err; > - int dir=0; > - > + snd_pcm_uframes_t frames, wanted, ofs; > const snd_pcm_channel_area_t *areas; > - snd_pcm_uframes_t ofs; > - snd_pcm_uframes_t frames; > - snd_pcm_uframes_t wanted; > > - if ( !pdbi->mmap_buffer || !wwo->hw_params || !wwo->pcm) > - return; > - > - err = snd_pcm_hw_params_get_period_size(wwo->hw_params, &period_size, &dir); > - avail = snd_pcm_avail_update(wwo->pcm); > + TRACE("\n"); > > - DSDB_CheckXRUN(pdbi); > - > - TRACE("avail=%d, mul=%d\n", (int)avail, mul); > - > - frames = pdbi->mmap_buflen_frames; > - > - EnterCriticalSection(&pdbi->mmap_crst); > + if (areas != pdbi->mmap_areas || areas->addr != pdbi->mmap_areas->addr) > + FIXME("Can't access sound driver's buffer directly.\n"); > > - /* we want to commit the given number of periods, or the whole lot */ > - wanted = mul == 0 ? frames : period_size * 2; > + while (pdbi->mmap_thread) > + { > + snd_pcm_wait(wwo->pcm, -1); > + wanted = frames = pdbi->mmap_buflen_frames; > + DSDB_CheckXRUN(pdbi); > > - snd_pcm_mmap_begin(wwo->pcm, &areas, &ofs, &frames); > - if (areas != pdbi->mmap_areas || areas->addr != pdbi->mmap_areas->addr) > - FIXME("Can't access sound driver's buffer directly.\n"); > - > - /* mark our current play position */ > - pdbi->mmap_ppos = ofs; > - > - if (frames > wanted) > - frames = wanted; > - > - err = snd_pcm_mmap_commit(wwo->pcm, ofs, frames); > - > - /* Check to make sure we committed all we want to commit. ALSA > - * only gives a contiguous linear region, so we need to check this > - * in case we've reached the end of the buffer, in which case we > - * can wrap around back to the beginning. */ > - if (frames < wanted) { > - frames = wanted -= frames; > + EnterCriticalSection(&pdbi->mmap_crst); > snd_pcm_mmap_begin(wwo->pcm, &areas, &ofs, &frames); > snd_pcm_mmap_commit(wwo->pcm, ofs, frames); > - } > > - LeaveCriticalSection(&pdbi->mmap_crst); > + /* mark our current play position */ > + pdbi->mmap_ppos = ofs; > + > + /* Check to make sure we committed all we want to commit. ALSA > + * only gives a contiguous linear region, so we need to check this > + * in case we've reached the end of the buffer, in which case we > + * can wrap around back to the beginning. */ > + if (frames < wanted) { > + frames = wanted - frames; > + snd_pcm_mmap_begin(wwo->pcm, &areas, &ofs, &frames); > + snd_pcm_mmap_commit(wwo->pcm, ofs, frames); > + } > + LeaveCriticalSection(&pdbi->mmap_crst); > + } > + TRACE("Destroyed MMAP thread\n"); > + return 0; > } > > -static void DSDB_PCMCallback(snd_async_handler_t *ahandler) > +/** > + * Cancel running MMAP thread */ > +static void DBSB_MMAPStop(IDsDriverBufferImpl* This) > { > - int periods; > - /* snd_pcm_t * handle = snd_async_handler_get_pcm(ahandler); */ > - IDsDriverBufferImpl* pdbi = snd_async_handler_get_callback_private(ahandler); > - TRACE("callback called\n"); > - > - /* Commit another block (the entire buffer if it's a direct hw buffer) */ > - periods = pdbi->mmap_mode == SND_PCM_TYPE_HW ? 0 : 1; > - DSDB_MMAPCopy(pdbi, periods); > + HANDLE Thread; > + TRACE("Destroying MMAP thread\n"); > + if (This->mmap_thread == NULL) return; > + Thread = This->mmap_thread; > + This->mmap_thread = NULL; > + WaitForSingleObject(Thread, INFINITE); > } > > /** > @@ -3158,21 +3145,47 @@ static int DSDB_CreateMMAP(IDsDriverBuff > unsigned int bits_per_sample; > unsigned int bits_per_frame; > int err; > + int mmap_mode; > + > + mmap_mode = snd_pcm_type(wwo->pcm); > + pdbi->mmap_thread = NULL; > + > + /* If we have a software buffer, change buffer size to only get 2 > + * > + * Why 2? We want a small number so that we don't get ahead of the > + * DirectSound mixer. But we don't want to ever let the buffer get > + * completely empty - having 2 periods gives us time to commit another > + * period when the first expires. > + * > + * The potential for buffer underrun is high, but that's the reality > + * of using a translated buffer (the whole point of DirectSound is > + * to provide direct access to the hardware). > + */ > > - err = snd_pcm_hw_params_get_format(wwo->hw_params, &format); > - err = snd_pcm_hw_params_get_buffer_size(wwo->hw_params, &frames); > - err = snd_pcm_hw_params_get_channels(wwo->hw_params, &channels); > - bits_per_sample = snd_pcm_format_physical_width(format); > - bits_per_frame = bits_per_sample * channels; > - pdbi->mmap_mode = snd_pcm_type(wwo->pcm); > - > - if (pdbi->mmap_mode == SND_PCM_TYPE_HW) { > + if (mmap_mode == SND_PCM_TYPE_HW) { > TRACE("mmap'd buffer is a hardware buffer.\n"); > } > else { > + snd_pcm_uframes_t psize; > + err = snd_pcm_hw_params_get_period_size(wwo->hw_params, &psize, NULL); > + /* Set only a buffer of 2 period sizes, to decrease latency */ > + if (err >= 0) > + err = snd_pcm_hw_params_set_buffer_size_near(wwo->pcm, wwo->hw_params, &psize); > + > + > + psize *= 2; > + if (err < 0) { > + ERR("Errno %d (%s) occured when setting buffer size\n", err, strerror(errno)); > + } > TRACE("mmap'd buffer is an ALSA emulation of hardware buffer.\n"); > } > > + err = snd_pcm_hw_params_get_format(wwo->hw_params, &format); > + err = snd_pcm_hw_params_get_buffer_size(wwo->hw_params, &frames); > + err = snd_pcm_hw_params_get_channels(wwo->hw_params, &channels); > + bits_per_sample = snd_pcm_format_physical_width(format); > + bits_per_frame = bits_per_sample * channels; > + > if (TRACE_ON(wave)) > ALSA_TraceParameters(wwo->hw_params, NULL, FALSE); > > @@ -3208,19 +3221,13 @@ static int DSDB_CreateMMAP(IDsDriverBuff > InitializeCriticalSection(&pdbi->mmap_crst); > pdbi->mmap_crst.DebugInfo->Spare[0] = (DWORD_PTR)"WINEALSA_mmap_crst"; > > - err = snd_async_add_pcm_handler(&pdbi->mmap_async_handler, wwo->pcm, DSDB_PCMCallback, pdbi); > - if ( err < 0 ) > - { > - ERR("add_pcm_handler failed. reason: %s\n", snd_strerror(err)); > - return DSERR_GENERIC; > - } > - > return DS_OK; > } > > static void DSDB_DestroyMMAP(IDsDriverBufferImpl* pdbi) > { > TRACE("mmap buffer %p destroyed\n", pdbi->mmap_buffer); > + DBSB_MMAPStop(pdbi); > pdbi->mmap_areas = NULL; > pdbi->mmap_buffer = NULL; > pdbi->mmap_crst.DebugInfo->Spare[0] = 0; > @@ -3254,6 +3261,7 @@ static ULONG WINAPI IDsDriverBufferImpl_ > > if (refCount) > return refCount; > + IDsDriverBuffer_Stop(iface); > if (This == This->drv->primary) > This->drv->primary = NULL; > DSDB_DestroyMMAP(This); > @@ -3372,29 +3380,13 @@ static HRESULT WINAPI IDsDriverBufferImp > err = snd_pcm_prepare(wwo->pcm); > state = snd_pcm_state(wwo->pcm); > } > - if ( state == SND_PCM_STATE_PREPARED ) > + if ( state == SND_PCM_STATE_PREPARED && This->mmap_thread == NULL) > { > - /* If we have a direct hardware buffer, we can commit the whole lot > - * immediately (periods = 0), otherwise we prime the queue with only > - * 2 periods. > - * > - * Why 2? We want a small number so that we don't get ahead of the > - * DirectSound mixer. But we don't want to ever let the buffer get > - * completely empty - having 2 periods gives us time to commit another > - * period when the first expires. > - * > - * The potential for buffer underrun is high, but that's the reality > - * of using a translated buffer (the whole point of DirectSound is > - * to provide direct access to the hardware). > - * > - * A better implementation would use the buffer Lock() and Unlock() > - * methods to determine how far ahead we can commit, and to rewind if > - * necessary. > - */ > - int periods = This->mmap_mode == SND_PCM_TYPE_HW ? 0 : 2; > - > - DSDB_MMAPCopy(This, periods); > - err = snd_pcm_start(wwo->pcm); > + err = snd_pcm_start(wwo->pcm); > + This->mmap_thread = CreateThread(NULL, 0, DBSB_MMAPStart, (LPVOID)(DWORD)This, 0, NULL); > + if (This->mmap_thread == NULL) > + return DSERR_GENERIC; > + SetThreadPriority(This->mmap_thread, THREAD_PRIORITY_TIME_CRITICAL); > } > return DS_OK; > } > @@ -3419,6 +3411,8 @@ static HRESULT WINAPI IDsDriverBufferImp > return DS_OK; > } > > + DBSB_MMAPStop(This); > + > if ( ( err = snd_pcm_drop(wwo->pcm)) < 0 ) > { > ERR("error while stopping pcm: %s\n", snd_strerror(err)); > > ------------------------------------------------------------------------ > > >

2 2

window managment bug
by Clinton Stimpson Dec. 26, 2006

Dec. 26, 2006

Does bug #6006 fall under window management? When buttons are turned invisible, the invalidated area in the parent window is not repainted, so it looks like the button is still there. Also, there's two invisible buttons that seem to repaint themselves when the parent window resizes, clicking on them does nothing. Any ideas where I can go to track this down? Thanks, Clinton

1 0

Re: oleaut32: preliminary support for VT_DISPATCH as parameters to VarAdd (and other 2-operand operations) (try 3)
by Alexandre Julliard Dec. 26, 2006

Dec. 26, 2006

a_villacis(a)palosanto.com writes: > This is another attempt to add IDispatch support to several 2-operand > functions. Now the macro was removed and the VT_DISPATCH handling was > copied to every affected function. Thanks, it's better that way. Still, it seems to me you don't always clean the variants properly, you should do some sort of "goto end" in all cases instead of simply returning. Also renaming the function parameters in most cases is not necessary. I'd also suggest to do a separate patch for each function, especially since in some functions a bit of restructuring will be needed to cleanup properly on error. -- Alexandre Julliard julliard(a)winehq.org

1 0

dlls/kernel32/tests: test failure in locale.c
by Claudio Fontana Dec. 26, 2006

Dec. 26, 2006

Hello, while tweaking other things in kernel32, I noticed that latest version fails a test in locale.c: $ cd dlls/kernel32/tests $ make test ../../../tools/runtest -q -P wine -M kernel32.dll -T ../../.. -p kernel32_test.exe.so locale.c && touch locale.ok locale.c:1990: Test failed: Enumerated grp locale 1084 not valid locale.c:1990: Test failed: Enumerated grp locale 1170 not valid locale.c:1990: Test failed: Enumerated grp locale 1171 not valid locale.c:1990: Test failed: Enumerated grp locale 2108 not valid locale.c:1990: Test failed: Enumerated grp locale 3132 not valid make: *** [locale.ok] Error 5 Hope this helps. Claudio

2 1

more meaningful traces?
by Lei Zhang Dec. 26, 2006

Dec. 26, 2006

Hi, I noticed several files in dlls/winmm have a getMessage() function to translate message numbers to message names. This is extremely helpful for debug traces, where one can immediately see the name of the messages sent, instead of wondering what 0x105 is. So my question is, why is winmm the only component translating its message? Is there some downside to having message names in the traces? Happy Holidays! - Lei

1 0

Re: [2/2] wined3d: Restore texture bindings in the FBO code
by Stefan Dösinger Dec. 25, 2006

Dec. 25, 2006

Am 25.12.2006 um 17:12 schrieb H. Verbeet: > Previously all textures were (re)bound each DrawPrimitive call. With > the new state management only dirty textures are bound, so we need to > restore the texture binding if we change it. > > Changelog: > - Restore texture bindings in the FBO code > <wined3d_fbo_texture_binding.diff.txt> Why are you not using glPushAttrib / glPopAttrib?

2 1

Re: [comctl32] MonthCalendar - Fix wrong date selected when clicknext/prev months day (#6967)
by Dmitry Timoshkov Dec. 25, 2006

Dec. 25, 2006

"Vijay Kiran Kamuju" <infyquest(a)gmail.com> wrote: > - retval = *daypos + (7 * *weekpos) - firstDay; > + retval = *daypos + ((7 * *weekpos) - firstDay); > return retval; > } This change doesn't make any difference, neither in functionality, nor in readability IMO. Please avoid this kind of changes, it only clutters the patch, making it hard to see what is the real change. > - lpht->st.wDay = MONTHCAL_MonthLength(lpht->st.wMonth,lpht->st.wYear) -day; > + lpht->st.wDay = day+MONTHCAL_MonthLength(lpht->st.wMonth,lpht->st.wYear) ; A couple of spaces around '+' won't hurt, and a space before ';' doesn't match an existing code style. > - return TRUE; > + /*return TRUE;*/ If you are not sure that the return should be removed, it's a sign that you need a test case, just commenting it out without any explanation is not a solution. Same comment to all other similar cases. > + MONTHCAL_CopyTime(&infoPtr->minSel,&nmsc.stSelStart); > + MONTHCAL_CopyTime(&infoPtr->maxSel,&nmsc.stSelEnd); > nmsc.nmhdr.hwndFrom = infoPtr->hwndSelf; > nmsc.nmhdr.idFrom = GetWindowLongPtrW(infoPtr->hwndSelf, GWLP_ID); > nmsc.nmhdr.code = MCN_SELCHANGE; > - MONTHCAL_CopyTime(&infoPtr->minSel,&nmsc.stSelStart); > - MONTHCAL_CopyTime(&infoPtr->maxSel,&nmsc.stSelEnd); This change is not needed. -- Dmitry.

2 2

kernel32 (profile.c): current section and key implementation
by Claudio Fontana Dec. 25, 2006

Dec. 25, 2006

Hello all, I posted previously a patch to wine-patches here: http://www.winehq.org/pipermail/wine-patches/2006-December/034022.html However, I now understand that although it solves the issue I had with the specific program, the change is incomplete, since in several other places, sections and keys are only considered meaningful when not equal to the empty string. In the Microsoft implementation, the unnamed section is valid, as are zero length keys. Thus, I think a broader scope change is required. Before I start working on a new patch, do you acknowledge that the current implementation is incorrect, and a patch fixing it by not requiring name[0] to be != '\0' is likely to be accepted? (btw, I've noticed you switched to git, so I will be using that instead now). Thanks, Claudio Fontana

2 2

Re: ntdll: Added stubs for the Rtl*GenericTable* functions
by Vitaliy Margolen Dec. 25, 2006

Dec. 25, 2006

Is there are any reason why do you need this functions at all? According to ddk they are part of "Installable File System Drivers". This is a kernel level thing only and no user app should use these routines. Especially stabbing them won't do you any good if something actually using them. But that something has no place to run in Wine yet. Vitaliy.

2 5

Re: winex11.drv: Only try to initialize OpenGL if the display has the GLX extension.
by Roderick Colenbrander Dec. 24, 2006

Dec. 24, 2006

Hi, I'm not sure if checking for the GLX extension is such a good idea as it is possible to use OpenGL without having the GLX extension. For instance when plain Mesa is used. Regards, Roderick -- Der GMX SmartSurfer hilft bis zu 70% Ihrer Onlinekosten zu sparen! Ideal für Modem und ISDN: http://www.gmx.net/de/go/smartsurfer

3 5

Re: [Winmm/winealsa] Don't use asynchronous callbacks in dsound any more
by Tomas Carnecky Dec. 24, 2006

Dec. 24, 2006

Maarten Lankhorst wrote: > Instead of using asynchronous callbacks that uses signals, use a > seperate thread that can be cancelled, this prevents deadlock issues. > > Basically we use snd_pcm_wait() that tells us when enough room is free > to commit another buffer, then we commit the previous buffer and make > the next buffer ready. > > Since snd_pcm_wait() uses poll(), we don't have signals in winealsa any > more. > Correct me if I'm wrong, but doesn't your code create and destroy the thread in ::Play() and ::Stop()? It would be better to destroy it together with the DirectSoundBuffer object (whether you create the thread when the object is created or the first time ::Play() is called is not so important, but I've been told the CreateFrame() overhead is big so that we should try to minimize its use). but I would suggect to create teh thread when the DSB object is created, as the CreateThread() in ::Play() may have side-effects (delay in the sound etc). tom

1 0

Re: [SPAM] [Winmm/winealsa] Don't use asynchronous callbacks in dsound any more
by a_villacis＠palosanto.com Dec. 24, 2006

Dec. 24, 2006

> Instead of using asynchronous callbacks that uses signals, use a > seperate thread that can be cancelled, this prevents deadlock issues. > > Basically we use snd_pcm_wait() that tells us when enough room is free > to commit another buffer, then we commit the previous buffer and make > the next buffer ready. > > Since snd_pcm_wait() uses poll(), we don't have signals in winealsa any > more. > > This patch causes a hang in the interactive dsound tests: bash$ cd wine-0.9.28-cvs-patch/dlls/dsound/tests bash$ WINETEST_INTERACTIVE=1 wine dsound_test.exe.so dsound ... fixme:wave:DBSB_MMAPStart Can't access sound driver's buffer directly. ds3d.c:655: Test failed: The sound played for 835 ms instead of 1000 ms dsound.c:602: Testing a primary buffer at 48000x8x1 with a secondary buffer at 11025x16x2 ds3d.c:447: Playing 1 second 440Hz tone at 11025x16x2 fixme:wave:DBSB_MMAPStart Can't access sound driver's buffer directly. fixme:wave:DBSB_MMAPStart Can't access sound driver's buffer directly. fixme:wave:DBSB_MMAPStart Can't access sound driver's buffer directly. ds3d.c:655: Test failed: The sound played for 371 ms instead of 1000 ms dsound.c:602: Testing a primary buffer at 48000x8x2 with a secondary buffer at 11025x16x2 ds3d.c:447: Playing 1 second 440Hz tone at 11025x16x2 fixme:wave:DBSB_MMAPStart Can't access sound driver's buffer directly. fixme:wave:DBSB_MMAPStart Can't access sound driver's buffer directly. ds3d.c:655: Test failed: The sound played for 834 ms instead of 1000 ms dsound.c:602: Testing a primary buffer at 48000x16x1 with a secondary buffer at 11025x16x2 ds3d.c:447: Playing 1 second 440Hz tone at 11025x16x2 fixme:wave:DBSB_MMAPStart Can't access sound driver's buffer directly. fixme:wave:DBSB_MMAPStart Can't access sound driver's buffer directly. ds3d.c:655: Test failed: The sound played for 767 ms instead of 1000 ms dsound.c:602: Testing a primary buffer at 48000x16x2 with a secondary buffer at 11025x16x2 ds3d.c:447: Playing 1 second 440Hz tone at 11025x16x2 fixme:wave:DBSB_MMAPStart Can't access sound driver's buffer directly. dsound.c:602: Testing a primary buffer at 96000x8x1 with a secondary buffer at 11025x16x2 ds3d.c:447: Playing 1 second 440Hz tone at 11025x16x2 fixme:wave:DBSB_MMAPStart Can't access sound driver's buffer directly. (then silent hang) kernel 2.6.19 wine 0.9.28 Fedora Core 5, snd_via82xx module

1 0

Re: user32: fix return value in DialogBoxParamA if no resource can befound + simple test
by Dmitry Timoshkov Dec. 24, 2006

Dec. 24, 2006

Louis. Lenders <xerox_xerox2000(a)yahoo.co.uk> wrote: > --- a/dlls/user32/dialog.c > +++ b/dlls/user32/dialog.c > @@ -800,7 +800,12 @@ INT_PTR WINAPI DialogBoxParamA( HINSTANC > HRSRC hrsrc; > LPCDLGTEMPLATEA ptr; > > - if (!(hrsrc = FindResourceA( hInst, name, (LPSTR)RT_DIALOG ))) return 0; > + if (!(hrsrc = FindResourceA( hInst, name, (LPSTR)RT_DIALOG ))) > + { > + SetLastError(ERROR_RESOURCE_NAME_NOT_FOUND); > + ERR("FindResource %s failed, returning -1\n",debugstr_a(name)); > + return -1; > + } Most likely the last error should be set by FindResource itself, not a more high level DialogBoxParam call. If that's not the case it should be fixed there. -- Dmitry.

1 0

re: fonts:add "framework" for tahoma font
by Dan Kegel Dec. 24, 2006

Dec. 24, 2006

Louis wrote: > [Here's a start at a tahoma-compatible font. Works with Steam...] Does it fix http://bugs.winehq.org/show_bug.cgi?id=4449 ? - Dan

3 3

comctl monthcalendar help
by Vijay Kiran Kamuju Dec. 24, 2006

Dec. 24, 2006

Hi, I am trying to fix visual bug in comctl monthcalendar control. When I am using the control spy v2.0 for comctl5.0, I can see that current date is highled with grey background on windows xp. (similar to when we click a day in the calendar) But on wine its not happening. I am trying to find out, how we draw that highlighting. And also how we draw the default calendar. Thanks, VJ

2 2

Re: d3d8: Don't report Shader Model 2.0 or higher
by H. Verbeet Dec. 23, 2006

Dec. 23, 2006

On 23/12/06, Chris Robinson <chris.kcat(a)gmail.com> wrote: > D3D8 never reports anything above SM 1.x. Does this fix something?

2 1

WineLib porting project for small Windows application
by Eric Rhodes Dec. 22, 2006

Dec. 22, 2006

Hi Wine developers, My name is Eric Rhodes and I work for Schmap, Inc. (www.schmap.com) I'm posting this message to find out if any of you would be interested in working with us on a Mac "porting" project for a small Windows application (using Wine). A full project description is set forth below. Please feel free to contact me directly at 919-246-0186 (or via email at <mailto:eric.rhodes(a)schmap.com>eric.rhodes(a)schmap.com) if you are interested in this project, or if you would like to refer someone else that might be. Thanks and happy holidays, Eric Rhodes ++++++++++++++++++++++++++++++++ Our Company Schmap is a media and technology company with a focus on developing client and server based consumer mapping solutions with a geospatial twist - see www.schmap.com. The Project We are seeking a senior engineer with strong C++ (preferably MS C++) to assist us with a self-contained WineLib porting project, working alongside two engineers who helped develop the Windows application to be ported. Mac programming experience is desired but not absolutely necessary. Specific Responsibilities Initial consultation and project setup; project planning (liasing with the CTO to develop a plan for the port); general code porting (outside of the consultancy role, you'll also be expected to get your hands dirty with some coding once the port gets underway); reporting on progress to the CTO. Duration Initial 4 to 8 week contract, with potential for extension depending on how the work progresses. Location One week onsite in Beijing, China required (beginning mid January 2007); remainder offsite. Must be eligible for Chinese visa. In addition to the fees for this project, we will fly you to Beijing and put you up in comfortable accommodations at our expense.

2 1

State of demoscene productions under wine.
by Wojciech 'arab' Arabczyk Dec. 22, 2006

Dec. 22, 2006

Hello So, i've been holding back with writing this mail for the last three releases of wine. Im pleased to infrom the whole wine team that almost every demoscene prod i keep a warm place for in my hearth is now (since 0.9.27) working with wine. The list with corresponding links below (all downloads are freeware or public domain): - Kewlers with "Variform" (working perfectly since i remember) http://www.pouet.net/prod.php?which=7138 - Kewlers and Mfx with "1995" (working since last [0.9.27] version, some strange slowdowns occur) http://www.pouet.net/prod.php?which=25783 - Kewlers with Protozoa (working since 0.9.23 i think, some strange slowdowns occur) - Farbrausch with fr-08 (working since 0.9.18 tho minor graphic glitches occur, between 0.9.21 and 0.9.22 occured a regression that does not allow the demo to finish. I'll do some regression test as soon as i'll have some time for it) http://www.pouet.net/prod.php?which=1221 - Farbrausch with fr-025 (working since last release [0.9.27] yet still some strange bugs [on-screen buffers that should not be displayed?] occur) http://www.pouet.net/prod.php?which=9450 - Kasparov from Elitegroup (some effects missing) http://www.pouet.net/prod.php?which=374 - Iconoclast by ASD (working since last [0.9.27] release, some strange slowdowns occur just as in 1995 by Kewlers and MFX and Protozoa by Kewlers [maybe a race condition?]) I hope the list is interesting? I could do some screenshot comparisions with effects/graphics problems or differences between the demos runing natively under windows and under wine. Also please note that the list is incomplete. I could add at least a donzen of demos still not working (i just thot i'll focus this time on those i like most). -- Best regards Wojciech Arabczyk

1 0

Re: dinput: AddRef and Release parent DirectInput class.
by Alexandre Julliard Dec. 22, 2006

Dec. 22, 2006

Vitaliy Margolen <wine-patches(a)kievinfo.com> writes: > This fixes bug 6918. > --- > dlls/dinput/joystick_linuxinput.c | 5 +++++ > dlls/dinput/keyboard.c | 8 ++++++++ > dlls/dinput/mouse.c | 5 +++++ > 3 files changed, 18 insertions(+), 0 deletions(-) It fails make test: ../../../tools/runtest -q -P wine -M dinput.dll -T ../../.. -p dinput_test.exe.so keyboard.c && touch keyboard.ok keyboard.c:159: Test failed: IDirectInput_Release() reference count = 1 -- Alexandre Julliard julliard(a)winehq.org

1 0

WineAlsa mmap patch
by Maarten Lankhorst Dec. 21, 2006

Dec. 21, 2006

This changes the winealsa dsound driver to stop using asynchronous callbacks and use snd_pcm_wait instead, I'm welcoming testing. For me this fixes the mmap_crst deadlock. I don't know what caused the deadlock in the first place, but this seems to have fixed it somehow. My test was basically running max payne 2 on my old pc, for some reason it's prone to this bug. With this patch I didn't have this problem. Detlef Riekenberg helped me testing first (buggy version) and now I hopefully fixed all issues that were found in it. It's not entirely ready yet I think, I'm not satisfied with latency for non-direct hw access. Basically it is the same loop as for direct hardware access at the moment. That means increased latency, which I think is noticable. Welcoming all comments on it and experiences with it. Maarten

1 0

Re: libwine: Fix some typos
by Alexandre Julliard Dec. 21, 2006

Dec. 21, 2006

Andrew Talbot <Andrew.Talbot(a)talbotville.com> writes: > I believe these identifiers are POSIX ones (from <sys/stat.h>), so they should > not possess one leading underscore (unlike the similar ones in "msvcrt"). These defines are meant for Windows, where they do have underscores. -- Alexandre Julliard julliard(a)winehq.org

2 3

STLPort 5.1 and winegcc / winelib
by gendo＠cox.net Dec. 21, 2006

Dec. 21, 2006

I was wondering if anyone had tried STLPort 5 with winelib. From searching the archives it seems you need to swap out gcc/c++ for winegcc/wineg++ and pass the -mno-cygwin option. Doing that gives me these errors. make -f gcc.mak all wineg++ -pthread -mno-cygwin -fexceptions -fident -fPIC -O2 -fuse-cxa-atexit -D_REENTRANT -D_GNU_SOURCE -I../../stlport -c -o obj/gcc/so/dll_main.o ../../src/dll_main.cpp /usr/include/../include/stdlib.h:469: error: expected ‘;’ before ‘*’ token /usr/include/../include/stdlib.h:470: error: expected ‘;’ before ‘*’ token /usr/include/../include/stdlib.h:471: error: expected ‘;’ before ‘*’ token /usr/include/../include/stdlib.h:475: error: expected ‘;’ before ‘*’ token /usr/include/../include/stdlib.h:479: error: ‘int32_t’ has not been declared winegcc: g++ failed. make: *** [obj/gcc/so/dll_main.o] Error 2 Anyway these are probably from grabbing the system headers which I have read you should not have happen, but if I try to use a -isystem/usr/include/wine/msvcrt or -Isystem/usr/include/wine/msvcrt and -nostdinc++ you then get tons of errors about the stl headers missing exception, cstring, etc. I was wondering if anyone had any thoughts or ideas on how to get this compiled so it is usable is porting applications. Thanks

3 3

Re: msxml.dll: nextNode and reset functions
by Colin Pitrat Dec. 21, 2006

Dec. 21, 2006

OK, I finally managed to cross-compile and run this test under windows and under wine. The patch (test and functions) works well. Colin Pitrat (Bull Services Telco) Bull, Architect of an Open World (TM) Tél : +33 (0) 1 30 80 72 93 www.bull.com Colin Pitrat wrote: > New version of this patch, including corrections suggested by Huw > Davies, a correction and the addition of a test case. > However, I didn't run the test case under Windows because I have a > problem when I run make crosstest : > Undefined reference to _IID_IXMLDOMDocument > and other similar messages with _IID_IXMLDOMNode, _IID_IXMLDOMElement... > > Maybe this error is related to a previous one when running make > crosstest in top directory, occuring for advpack : > Can't find -lcabinet > > Change Log: > Add an iterator to _xmlnodemap struct > Implement nextNode function that give nodes one by one > Implement resetNode that reset the iterator > Add a test case for these functions > > Colin Pitrat (Bull Services Telco) > Bull, Architect of an Open World (TM) > Tél : +33 (0) 1 30 80 72 93 > www.bull.com > > > Colin Pitrat wrote: >> Change Log: >> Add an iterator to _xmlnodemap struct >> Implement nextNode function that give nodes one by one >> Implement resetNode that reset the iterator >> >> It's my first patch, so please tell me if something is wrong. I saw on >> http://www.winehq.com/site/sending_patches that I should write a test >> case, but I don't know how to do this (do I need windows ? because I >> don't have it). >> >> >> ------------------------------------------------------------------------ >> >> diff --git a/dlls/msxml3/nodemap.c b/dlls/msxml3/nodemap.c >> index 0797436..ead0138 100644 >> --- a/dlls/msxml3/nodemap.c >> +++ b/dlls/msxml3/nodemap.c >> @@ -44,6 +44,7 @@ typedef struct _xmlnodemap >> const struct ISupportErrorInfoVtbl *lpSEIVtbl; >> LONG ref; >> IXMLDOMNode *node; >> + long iterator; >> } xmlnodemap; >> >> static inline xmlnodemap *impl_from_IXMLDOMNamedNodeMap( >> IXMLDOMNamedNodeMap *iface ) >> @@ -297,15 +298,47 @@ static HRESULT WINAPI xmlnodemap_nextNod >> IXMLDOMNamedNodeMap *iface, >> IXMLDOMNode** nextItem) >> { >> - FIXME("\n"); >> - return E_NOTIMPL; >> + xmlnodemap *This = impl_from_IXMLDOMNamedNodeMap( iface ); >> + xmlNodePtr node; >> + xmlAttrPtr curr; >> + long attrIndex; >> + >> + TRACE("%p %ld\n", This, This->iterator); >> + >> + *nextItem = NULL; >> + >> + node = xmlNodePtr_from_domnode( This->node, 0 ); >> + curr = node->properties; >> + >> + for (attrIndex = 0; attrIndex < This->iterator; attrIndex++) { >> + if (curr->next == NULL) >> + return S_FALSE; >> + else >> + curr = curr->next; >> + } >> + >> + if (curr->next == NULL) >> + return S_FALSE; >> + else >> + curr = curr->next; >> + >> + This->iterator++; >> + + *nextItem = create_node( (xmlNodePtr) curr ); >> + >> + return S_OK; >> } >> >> static HRESULT WINAPI xmlnodemap_reset( >> IXMLDOMNamedNodeMap *iface ) >> { >> - FIXME("\n"); >> - return E_NOTIMPL; >> + xmlnodemap *This = impl_from_IXMLDOMNamedNodeMap( iface ); >> + >> + TRACE("%p %ld\n", This, This->iterator); >> + >> + This->iterator = 0; >> + >> + return S_OK; >> } >> >> static HRESULT WINAPI xmlnodemap__newEnum( >> > > ------------------------------------------------------------------------ > > diff --git a/dlls/msxml3/nodemap.c b/dlls/msxml3/nodemap.c > index 0797436..5458542 100644 > --- a/dlls/msxml3/nodemap.c > +++ b/dlls/msxml3/nodemap.c > @@ -44,6 +44,7 @@ typedef struct _xmlnodemap > const struct ISupportErrorInfoVtbl *lpSEIVtbl; > LONG ref; > IXMLDOMNode *node; > + long iterator; > } xmlnodemap; > > static inline xmlnodemap *impl_from_IXMLDOMNamedNodeMap( IXMLDOMNamedNodeMap *iface ) > @@ -258,8 +259,8 @@ static HRESULT WINAPI xmlnodemap_get_len > > first = node->properties; > if (first == NULL) { > - *listLength = 0; > - return S_OK; > + *listLength = 0; > + return S_OK; > } > > curr = first; > @@ -297,15 +298,42 @@ static HRESULT WINAPI xmlnodemap_nextNod > IXMLDOMNamedNodeMap *iface, > IXMLDOMNode** nextItem) > { > - FIXME("\n"); > - return E_NOTIMPL; > + xmlnodemap *This = impl_from_IXMLDOMNamedNodeMap( iface ); > + xmlNodePtr node; > + xmlAttrPtr curr; > + long attrIndex; > + > + TRACE("%p %ld\n", This, This->iterator); > + > + *nextItem = NULL; > + > + node = xmlNodePtr_from_domnode( This->node, 0 ); > + curr = node->properties; > + > + for (attrIndex = 0; attrIndex < This->iterator; attrIndex++) { > + if (curr->next == NULL) > + return S_FALSE; > + else > + curr = curr->next; > + } > + > + This->iterator++; > + > + *nextItem = create_node( (xmlNodePtr) curr ); > + > + return S_OK; > } > > static HRESULT WINAPI xmlnodemap_reset( > IXMLDOMNamedNodeMap *iface ) > { > - FIXME("\n"); > - return E_NOTIMPL; > + xmlnodemap *This = impl_from_IXMLDOMNamedNodeMap( iface ); > + > + TRACE("%p %ld\n", This, This->iterator); > + > + This->iterator = 0; > + > + return S_OK; > } > > static HRESULT WINAPI xmlnodemap__newEnum( > @@ -389,6 +417,7 @@ IXMLDOMNamedNodeMap *create_nodemap( IXM > nodemap->lpSEIVtbl = &support_error_vtbl; > nodemap->node = node; > nodemap->ref = 1; > + nodemap->iterator = 0; > > IXMLDOMNode_AddRef( node ); > /* Since we AddRef a node here, we don't need to call xmldoc_add_ref() */ > diff --git a/dlls/msxml3/tests/domdoc.c b/dlls/msxml3/tests/domdoc.c > index a2824fd..d8f08ac 100644 > --- a/dlls/msxml3/tests/domdoc.c > +++ b/dlls/msxml3/tests/domdoc.c > @@ -114,7 +114,10 @@ static void test_domdoc( void ) > r = CoCreateInstance( &CLSID_DOMDocument, NULL, > CLSCTX_INPROC_SERVER, &IID_IXMLDOMDocument, (LPVOID*)&doc ); > if( r != S_OK ) > + { > + printf("CoCreateInstance failed : %d\n",r); > return; > + } > > /* try some stupid things */ > r = IXMLDOMDocument_loadXML( doc, NULL, NULL ); > @@ -411,7 +414,7 @@ todo_wine > ok( node == NULL, "getNamedItem should have returned NULL\n"); > SysFreeString( str ); > > - /* test indexed access of attributes */ > + /* test indexed access of attributes */ > r = IXMLDOMNamedNodeMap_get_length( map, &count ); > ok ( r == S_OK, "get_length wrong code\n"); > ok ( count == 1, "get_length != 1\n"); > @@ -434,6 +437,23 @@ todo_wine > r = IXMLDOMNode_get_nodeName( node, NULL ); > ok ( r == E_INVALIDARG, "get_nodeName (NULL) wrong code\n"); > > + /* test sequential access of attributes */ > + node = NULL; > + r = IXMLDOMNamedNodeMap_nextNode( map, &node ); > + ok ( r == S_OK, "nextNode (first time) wrong code\n"); > + ok ( node != NULL, "nextNode, should be attribute\n"); > + > + r = IXMLDOMNamedNodeMap_nextNode( map, &node ); > + ok ( r != S_OK, "nextNode (second time) wrong code\n"); > + ok ( node == NULL, "nextNode, there is no attribute\n"); > + > + r = IXMLDOMNamedNodeMap_reset( map ); > + ok ( r == S_OK, "reset should return S_OK\n"); > + > + r = IXMLDOMNamedNodeMap_nextNode( map, &node ); > + ok ( r == S_OK, "nextNode (third time) wrong code\n"); > + ok ( node != NULL, "nextNode, should be attribute\n"); > + > /* content doesn't matter here */ > str = SysAllocString( szNonExistentFile ); > r = IXMLDOMNode_get_nodeName( node, &str ); >

1 0

[dinput] regression c17e06d5c0e456d3052a1b6de688956f87c59d3e: No need to copy default constant format. Use already defined static instead
by Christoph Frick Dec. 21, 2006

Dec. 21, 2006

hiho with that patch: commit c17e06d5c0e456d3052a1b6de688956f87c59d3e Author: Vitaliy Margolen <wine-patches(a)kievinfo.com> Date: Sun Dec 10 14:15:08 2006 -0700 dinput: No need to copy default constant format. Use already defined static instead. :100644 100644 c5a35a1... 369c26d... M dlls/dinput/joystick_linuxinput.c The game Richard Burns Rally does no longer get any events. The game worked after some patches some month ago from me. As i had no intention on that pages to make things right but to make the game work i might have not picked the optimal solution. But this patches fixed all the apps, that worked with their own DataFormat. So the problem is like that: The old code copied the default constant format and applied caclulate_ids on it. So the dwTypes where set with proper MAKEINSTANCE(...) calls. So an app, that requested later its own data format with SetDataFormat using the data from EnumObjects. After the change now the default const has ANY_INSTANCE all over the place but the app request a certain instance - so on SetDataFormat there are no matches. So the big question is: is our SetDataFormat wrong or is this patch wrong? Please advice. -- cu

2 3

Re: gdi32: Extract the constituent TrueType files from Mac suitcase fonts into the user's caches directory and use these instead of the original suitcase.
by Pierre d'Herbemont Dec. 21, 2006

Dec. 21, 2006

> +#ifdef WORDS_BIGENDIAN > +#define GET_BE_WORD(x) (x) > +#else > +#define GET_BE_WORD(x) ((((x) & 0xff) << 8) | (((x) & 0xff00) >> 8)) > +#endif This could be replaced by CoreFoundation's CFSwapInt16BigToHost, which might be a bit more efficient... Pierre.

1 0

Re: gdi32: Extract the constituent TrueType files from Mac suitcase fonts into the user's caches directory and use these instead of the original suitcase.
by Alexandre Julliard Dec. 21, 2006

Dec. 21, 2006

Huw Davies <huw(a)codeweavers.com> writes: > + ret = malloc(sizeof(*ret)); > + if(!ret) > + { > + CloseResFile(res_ref); > + return NULL; > + } > + list_init(ret); It's better to avoid using malloc() in Wine. Also it seems to me it would be cleaner to return an array of char*, instead of a struct list that forces the caller to make assumptions about the type of the list elements. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: d3d9: Add a render target test
by Ivan Gyurdiev Dec. 21, 2006

Dec. 21, 2006

H. Verbeet wrote: > On 20/12/06, H. Verbeet <hverbeet(a)gmail.com> wrote: >> This time as part of the device.c test. >> >> Changelog: >> - Add a render target test >> > Use the attached patch instead, the previous version doesn't apply. This test really belongs in stateblock.c, along with every other set <--> get test if you ask me, that's what the framework there was written for. It might be kind of a pain to get this particular one to work though, since changing the rendertarget is currently considered an "event", across which all other set/get tests are ran, but I'm sure it's possible. The question of whether stateblocks track render targets should be answered eventually...

1 0

Re: setupapi: Use GetSystemDirectory() rather than hardcoding 'system32'.
by Alexandre Julliard Dec. 20, 2006

Dec. 20, 2006

Francois Gouget <fgouget(a)codeweavers.com> writes: > @@ -87,13 +87,13 @@ static HINF search_for_inf(LPCVOID InfSpec, DWORD SearchControl) > HINF hInf = INVALID_HANDLE_VALUE; > WCHAR inf_path[MAX_PATH]; > > + static const WCHAR bslashW[] = {'\\',0}; > static const WCHAR infW[] = {'\\','i','n','f','\\',0}; > - static const WCHAR system32W[] = {'\\','s','y','s','t','e','m','3','2','\\',0}; > > if (SearchControl == INFINFO_REVERSE_DEFAULT_SEARCH) > { > - GetWindowsDirectoryW(inf_path, MAX_PATH); > - lstrcatW(inf_path, system32W); > + GetSystemDirectoryW(inf_path, MAX_PATH); > + lstrcatW(inf_path, bslashW); It's really supposed to hardcode system32, at least according to MSDN. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: usp10: impelment ScriptStringGetLogicalWidths
by Alexandre Julliard Dec. 20, 2006

Dec. 20, 2006

Clinton Stimpson <cjstimpson(a)utwire.net> writes: > This patch implements ScriptStringGetLogicalWidths. > Update usp10.spec so it isn't a stub anymore. > Also added a test for this function. The test fails for me: ../../../tools/runtest -q -P wine -M usp10.dll -T ../../.. -p usp10_test.exe.so usp10.c && touch usp10.ok usp10.c:1297: Test failed: ScriptStringGetLogicalWidths should return index 4 = 10, not 5 -- Alexandre Julliard julliard(a)winehq.org

2 2

Re: gdi32: Improve the fontconfig filtering hack so that Mac suitcase fonts get through.
by Paul Vriens Dec. 20, 2006

Dec. 20, 2006

On Tue, 2006-12-19 at 21:21 +0000, Huw Davies wrote: > + if(strcasecmp(ext, "pfa") && strcasecmp(ext, "pfb")) > AddFontFileToList(file, NULL, ADDFONT_EXTERNAL_FONT); Hi, this patch was committed but I don't think this 'if' statement is correct as it will never be entered. No clue if it's just changing '&&' into '||'. Cheers, Paul.

2 2

Re: kernel32: suppress an unneeded warning (try 3)
by Alexandre Julliard Dec. 20, 2006

Dec. 20, 2006

Juan Lang <juan_lang(a)yahoo.com> writes: > This time I added a test. It wouldn't have failed before, it would merely > produce a useless fixme. This suppresses that fixme by removing comments > before checking whether the read string overflows the buffer. > > ChangeLog: remove comments from string read in GetPrivateProfileIntW There is nothing special about ';' in values, other characters will terminate the number conversion too. It's probably safe to simply remove the FIXME, or maybe make it a WARN. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: ws2_32: ws_sockaddr_u2ws should zero the WS_sockaddr before using it.
by Francois Gouget Dec. 20, 2006

Dec. 20, 2006

On Wed, 20 Dec 2006, Kai Blin wrote: @@ -946,6 +946,9 @@ static int ws_sockaddr_u2ws(const struct { int res; + /* Zero the ws_sockaddr structure before using */ + memset(wsaddr, 0, *wsaddrlen); Are you sure this is needed in ws_sockaddr_u2ws() or should it rather be done by getsockname() (as your test indicates). Also, do we know of a real-world application that needs this? Finally, this memset() makes the one in the AF_UNSPEC case redundant, and may itself be redundant with some of the memcpy()s... -- Francois Gouget <fgouget(a)free.fr> http://fgouget.free.fr/ Stolen from an Internet user: "f u cn rd ths, u cn gt a gd jb n cmptr prgrmmng !"

2 1

Re: [5/5] d3d9: Add a render target test
by Alexandre Julliard Dec. 20, 2006

Dec. 20, 2006

"H. Verbeet" <hverbeet(a)gmail.com> writes: > Test if the stuff I just sent works. > > Changelog: > - Add a render target test > --- > > dlls/d3d9/tests/Makefile.in | 1 > dlls/d3d9/tests/rendertarget.c | 156 ++++++++++++++++++++++++++++++++++++++++ Could you please merge the test into one of the existing files, like device.c? We don't want to create new files for every test. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: winemenubuilder: Write truecolor icons as PNGs
by Francois Gouget Dec. 20, 2006

Dec. 20, 2006

On Wed, 6 Dec 2006, Frank Richter wrote: > When libpng is available, support for writing out 24 and 32bpp icons is > added, which are written out as PNGs. Also changed wineshelllink to > support both xpm and png extensions for icons. > > Some parts based on an older patch by Vitaliy Margolen. The thing that blocked this patch from being applied last time is that, if I remember correctly, Alexandre would like the PNG functionality to be added to the IPicture implementation in dlls/oleaut32/olepicture.c. Then winemenubuilder would use this interface to save the icon into the proper format. It's easy to see where the code should go to extend this interface to support loading PNG files as it already supports loading Gif and Jpeg files. However I'm fuzzy on how one would use this interface to specify the format to use for saving. Maybe by defining Wine-specific PICTYPE_XXX values? But that would not work on Windows... The other semi-related part would be to use ExtractAssociatedIcon() instead of extract_icon32(), though I'm not sure that's compatible with trying to select the 'best' icon size/color depth. I'm pretty sure my memory mangled some of this. Hopefully Alexandre will clarify things. -- Francois Gouget <fgouget(a)free.fr> http://fgouget.free.fr/ Any sufficiently advanced Operating System is indistinguishable from Linux

2 2

Re: [18/19] WineD3D: Do not change the states if render target locking is disabled
by Stefan Dösinger Dec. 19, 2006

Dec. 19, 2006

Am Dienstag 19 Dezember 2006 23:27 schrieb Stefan Dösinger: > There is no point in messing with the states if we aren't going to draw > anything anyway. This patch will conflict slightly with Henri's multiple render target patch. (device->renderTarget vs device->renderTarget[0]). If it does not apply then its not a crucial problem for the other patches, they should still work fine.

1 0

Re: oleaut32: preliminary support for VT_DISPATCH as parameters to VarAdd (and other 2-operand operations) (try 2) (RESEND)
by Alexandre Julliard Dec. 19, 2006

Dec. 19, 2006

Alex Villacís Lasso <a_villacis(a)palosanto.com> writes: > +#define HANDLE_DISPATCH(left, right) \ > + if ((V_VT(param_left) & VT_TYPEMASK) == VT_DISPATCH && \ > + (V_VT(param_right) & VT_TYPEMASK) != VT_NULL) \ > + {\ > + HRESULT hres;\ > + left = &tempLeft;\ > + hres = VARIANT_FetchDispatchValue(param_left, left);\ > + if (!SUCCEEDED(hres)) return hres;\ > + }\ > + if ((V_VT(param_right) & VT_TYPEMASK) == VT_DISPATCH &&\ > + (V_VT(param_left) & VT_TYPEMASK) != VT_NULL)\ > + {\ > + HRESULT hres;\ > + right = &tempRight;\ > + hres = VARIANT_FetchDispatchValue(param_right, right);\ > + if (!SUCCEEDED(hres)) return hres;\ > + } This macro is quite ugly. Also I don't see why you check the other variant against VT_NULL first, aren't there cases where you still need the other value? -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: Add a test for IDirectDrawSurface_SetColorKey
by Stefan Dösinger Dec. 19, 2006

Dec. 19, 2006

Am Montag 18 Dezember 2006 22:10 schrieb Günther Brammer: > Only tested with wine, unfortunately. Tests for the issue causing Bug 5730. I will run your test on windows, but I'd expect it to work. Can you also test which color key GetSurfaceDesc returns? > > diff --git a/dlls/ddraw/tests/dsurface.c b/dlls/ddraw/tests/dsurface.c > index 3d70f82..74e4f28 100644 > --- a/dlls/ddraw/tests/dsurface.c > +++ b/dlls/ddraw/tests/dsurface.c > @@ -167,6 +167,7 @@ static void SrcColorKey32BlitTest(void) > LPDIRECTDRAWSURFACE lpDst; > DDSURFACEDESC ddsd; > DDSURFACEDESC ddsd2; > + DDCOLORKEY DDColorKey; > LPDWORD lpData; > HRESULT rc; > > @@ -222,6 +223,21 @@ static void SrcColorKey32BlitTest(void) > "Destination data after blitting is not correct\n"); > rc = IDirectDrawSurface_Unlock(lpDst, NULL); > ok(rc==DD_OK,"Unlock returned: %x\n",rc); > + > + /* Also test SetColorKey */ > + IDirectDrawSurface_GetColorKey(lpSrc, DDCKEY_SRCBLT, &DDColorKey); > + ok(DDColorKey.dwColorSpaceLowValue == 0xFF00FF && > DDColorKey.dwColorSpaceHighValue == 0xFF00FF, + "GetColorKey does not > return the colorkey used at surface creation\n"); + > + DDColorKey.dwColorSpaceLowValue = 0x00FF00; > + DDColorKey.dwColorSpaceHighValue = 0x00FF00; > + IDirectDrawSurface_SetColorKey(lpSrc, DDCKEY_SRCBLT, &DDColorKey); > + > + DDColorKey.dwColorSpaceLowValue = 0; > + DDColorKey.dwColorSpaceHighValue = 0; > + IDirectDrawSurface_GetColorKey(lpSrc, DDCKEY_SRCBLT, &DDColorKey); > + ok(DDColorKey.dwColorSpaceLowValue == 0x00FF00 && > DDColorKey.dwColorSpaceHighValue == 0x00FF00, + "GetColorKey does not > return the colorkey set with SetColorKey\n"); > > IDirectDrawSurface_Release(lpSrc); > IDirectDrawSurface_Release(lpDst);

1 0

Status of wine development
by Remco Dec. 19, 2006

Dec. 19, 2006

I ask it consisely, but maybe the answers could be posted as a news item on WineHQ. It would be great for Wine to get some exposure on tech sites from time to time. That requires some actual news (new minor releases aren't front-page news after all). So, what's the status/news of DX10? DX in general? the Mac-port? Wine 1.0? What's the status of anything else that has been a major thing last year? __________________________________________________ Do You Yahoo!? Tired of spam? Yahoo! Mail has the best spam protection around http://mail.yahoo.com

7 7

Conformance tests
by Michal Okresa Dec. 19, 2006

Dec. 19, 2006

Hello, wine developers guide says that conformance tests must pass successfully on all Windows platforms. I tried couple of tests in past weeks to catch wine's style of unit testing, but now I am confused. For example I run crosscompiled olefont test (oleaut32) on the windows machines (XP, 98). The test ended with 15 failures. The same test executed on my linux machine ended with 13 failures (reasons for failures are almost identical with previous runs on windows). Is this expected behaviour of conformance tests on windows platforms? thanks -- Michal Okresa

2 1

Re: Bug 50
by Pedro Araújo Chaves Jr. Dec. 19, 2006

Dec. 19, 2006

On 12/19/06, Alexandre Julliard <julliard(a)winehq.org> wrote: > Passing the DC is not an option, that's a private gdi32 structure that > the driver doesn't know about. You shouldn't need to change the > function signature in any case, it already takes an lpDx array, all > you have to do is put better spacing values in there. Well, I had dropped that idea altogether as soon as I started working early in the morning, before reading your e-mail... besides, my next move was to play around with the spacing array, of course. Anyway, for a number of reasons, the idea of changing the function signature didn't sound well from the start to me neither. Yet I wanted to give it a shot, since it once fixed the issue. I'll keep you informed. Regards, Pedro.

1 0

Bug 50
by Pedro Araújo Chaves Jr. Dec. 19, 2006

Dec. 19, 2006

First, sorry for the blank mail I accidentally sent earlier... my bad. Ok, so I have been assigned bug 50 [1] here at my job, and I'm trying to apply the patch [2] Keith Dunwoody proposed in June 2005, which fixed the issue but was rejected for technical reasons. Right now I can't think of another way around that bug except a patch that big, but I think we could break it into two smaller ones: the first one would change what must be changed for the second one ― the actual correction ― to work. Well, that first one involves changing the signatures of the ExtTextOut functions in winex11.drv, in a similar fashion to what Keith had proposed; but, instead of adding 'INT *breakTotalExtra, INT *breakCount' to their argument lists, I suggest that we pass them the whole DC instead of just its physDev ― that way we'd have access to the relevant attributes while still being able to get its physDev (assigning it to a variable right at the beginning of the functions, in order to keep their current functionality in this first step). I'm only afraid that may bring a little overhead, so I'd really like to see some opinions, or some better ideas... [1] http://bugs.winehq.org/show_bug.cgi?id=50 [2] http://bugs.winehq.org/attachment.cgi?id=912&action=view I'll be testing that here, anyway. Cheers, Pedro. -- "People think computers will keep them from making mistakes. They're wrong. With computers you make mistakes faster." (Adam Osborne)

2 1

Running winetest on Vista
by Paul Vriens Dec. 19, 2006

Dec. 19, 2006

Hi, I've started to run our winetest executable on Vista. The current output is 7.8 MB or so, so we will never be able to sent it out just yet (1.5MB limit). I will start reducing the number of test failures on Vista (if I can of course). One thing that struck me and most likely it's due to Vista security is that when I run winetest.exe it tells me that it couldn't retrieve the list for the setupapi tests. While investigating that (several hours up to now) I found (not 100% sure though) that if an executable (like our setupapi_test.exe) has the word setup in it and is not properly signed, Vista starts complaining !!! Just copying our setupapi_test.exe to paul.exe makes it possible to run 'paul.exe --list'. Any ideas about the latter? Cheers, Paul.

4 5

Re: [1] WineD3D: Add the state dirtification infrastructure
by Stefan Dösinger Dec. 18, 2006

Dec. 18, 2006

Am Montag 18 Dezember 2006 19:25 schrieb Stefan Dösinger: Another issue I stumbled uppon: This patch moves the + 1 for the array declaration to the right place. > Am Montag 18 Dezember 2006 14:06 schrieb Stefan Dösinger: > Another update, it fixes an spotted by Henri issue where the state was used > instead of the representative and uses shifting and bitmasks to calculate > the index and shift for the bitmap. > > > This is another updated version, it uses bitmaps for the isStateDirty > > member. > > > > Am Sonntag 17 Dezember 2006 14:45 schrieb Stefan Dösinger: > > > Am Freitag 15 Dezember 2006 18:31 schrieb Stefan Dösinger: > > > > This patch adds the dirty list infrastructure. A dirty marker > > > > consists of a DWORD containing the state it marks dirty, and the list > > > > structure to build the list. > > > > > > > > The wined3ddevice has 3 new members: > > > > dirtyStateList: The list of dirty states. This will be moved to a > > > > per-context stucture once we add multithreading support > > > > freeListElems: A list containing unused dirty markers to avoid > > > > unnecessary HeapAlloc/HeapFree calls. I have seen D3D games spending > > > > 15% cpu time in the Heap code, we don't want to increase that. > > > > dirtyStates: An array that tells if a state is dirty. It is used to > > > > avoid double dirtification. I do not use the > > > > stateblock->changed.XXXX[] array because this is used for different > > > > purposes in the stateblock code. Ivan, do we still need it? > > > > > > Attached is an updated patch that uses an array for dirtification > > > instead of a list. Actually there are 2 arrays, one to quickly find out > > > if a specific state is dirty, and one to quickly find all dirty states. > > > > > > Checking if a state is dirty is done very often, so we need the > > > isStateDirty. A linear search on the dirtyArray would take way too much > > > time. On the other hand, my experiance so far shows that usually not > > > more than 10-20 states are dirty at the beginning of a drawprim call, > > > so iterating over all states would be a waste of time too. That's why > > > the dirtyArray is there. > > > > > > In the end there will be around 1000 states, so the size of those > > > arrays will be 8-10 kilobytes. Even oldest games load textures in the > > > area of megabytes, with recent games it's about gigabytes, so I don't > > > think those 10 kb's are an issue :-) . I could easilly change the > > > DWORDs to shorts, but all d3d states are defined as DWORDs, so I > > > decided to use DWORDS. > > > > > > If I used a bitmap I could squeeze the same information in about 128 > > > bytes + extra size for the code dealing with the bitmap + extra runtime > > > needs for bit shifting and checking all DWORDs in the bitmap. D3D games > > > are usually cpu limited, so I prefer the more memory intensive version. > > > > > > The other patches should apply cleanly over this patch, so I do not > > > resend them.

1 0

re: mscoree: Add a stub implementation for CorExitProcess and GetRequestedRuntimeInfo.
by Dan Kegel Dec. 18, 2006

Dec. 18, 2006

Hans wrote: > These stubs make the installer happy that is common to many of the .NET apps I tried. Which apps? Now that your patches seem to have been committed, I've started trying to find and install apps that require .net, but for some reason I'm having trouble finding many. The first one I tried is Life Balance (http://appdb.winehq.org/appview.php?iAppId=2928) but even if Mono-1.2 is installed, Life Balance seems to still try to download and install the MS c# runtime. +relay shows this before the download starts: 0009:Call advapi32.RegOpenKeyExA(80000002,0042f148 "SOFTWARE\\Microsoft\\NET Framework Setup\\NDP",00000000,00020019,0033bb90) ret=0040fa26 0009:Ret advapi32.RegOpenKeyExA() retval=00000002 ret=0040fa26 And bingo, http://msdn2.microsoft.com/en-us/library/aa480243.aspx documents this as part of the check to see if .net is installed. Has somebody figured out a canned .reg file to use to fake out this install check? - Dan

3 4

Re: ws2_32: Implement the SIO_ADDRESS_LIST_QUERY ioctl.
by Dmitry Timoshkov Dec. 18, 2006

Dec. 18, 2006

"Hans Leidekker" <hans(a)it.vu.nl> wrote: > case WS_SIO_ADDRESS_LIST_QUERY: > - FIXME("-> SIO_ADDRESS_LIST_QUERY request: stub\n"); > - break; > + { > + int fd; > + DWORD size; > + > + TRACE("-> SIO_ADDRESS_LIST_QUERY request\n"); > + > + if (!lpcbBytesReturned || (fd = get_sock_fd(s, 0, NULL)) == -1) It appears that fd is not used later at all. -- Dmitry.

1 0

Bug 50
by Pedro Araújo Chaves Jr. Dec. 18, 2006

Dec. 18, 2006

-- "People think computers will keep them from making mistakes. They're wrong. With computers you make mistakes faster." (Adam Osborne)

1 0

Re: DDraw: Implement depth filling through blit. (re-resubmit) try3
by Stefan Dösinger Dec. 18, 2006

Dec. 18, 2006

Am Sonntag 17 Dezember 2006 23:24 schrieb greg87(a)online.de: > Original Author: > Elie Morisse > lachienne(a)wanadoo.fr > > Updated by: > Gregor Münch > greg87(a)online.de > > This is modified version of this patch: > http://www.winehq.org/pipermail/wine-patches/2006-September/030815.html > > With the help from Stefan Dösinger I altered 2 lines. Full credit > belongs to Elie Morisse > > fixes: http://bugs.winehq.org/show_bug.cgi?id=4306 A small issue: The indentions in IWineD3DSurfaceImpl_BltZ look a bit strange. --- a/dlls/ddraw/ddraw.c +++ b/dlls/ddraw/ddraw.c @@ -1787,7 +1787,7 @@ IDirectDrawImpl_CreateNewSurface(IDirect { Usage |= WINED3DUSAGE_OVERLAY; } - if(This->depthstencil) + if(This->depthstencil || (pDDSD->ddsCaps.dwCaps & DDSCAPS_ZBUFFER)) { Perhaps you can get rid of This->depthstencil by setting the DDSCAPS_ZBUFFER flag instead on the requested surface desc in the auto depth stencil creation callback. Mind you, this auto depth stencil creation isn't used nowadays any more, it was needed earlier to prevent wined3d from crashing. You can remove the function body and replace it by an ERR entirely I think, it shouldn't be called if everything works correctly in wined3d.

1 0

A wine success story
by Dan Kegel Dec. 18, 2006

Dec. 18, 2006

I admit, I cruise new blog posts just to see what people are writing about Wine. Today I saw one ( http://www.ubuntux.org/shockwave-player-ubuntu-linux ) that says "I have discovered a way to get Shockwave working under Linux. I am using Ubuntu 6.06 (Dapper Drake), Wine 0.99.22, Firefox 1.5.0.7, and Shockwave 10.1.4.020. 1. Install Wine. ... Wine will automatically associate .exe files with itself. More information on Wine is available at http://www.winehq.com . 2. Download the Windows version of Firefox and install. Wine will ensure the installer functions correctly. 3. Install the Windows version of Shockwave for Mozilla. I downloaded and installed it manually (slim install). After these steps, I had a working Shockwave player from within Firefox. " Now that's the kind of user story I like to see! - Dan

7 8

Re: WineD3D: paletted texture emulation using fragment shaders
by Frank Richter Dec. 17, 2006

Dec. 17, 2006

On 16.12.2006 22:24, Roderick Colenbrander wrote: > + "MUL index.x, index.x, constants.x;\n" /* Scale the index by 255/256 */ > + "ADD index.x, index.x, constants.y;\n" /* Add a bias of '0.5' in order to sample in the middle */ FWIW, this can be conflated to "MAD index.x, index.x, constants.x, constants.y;". -f.r.

1 0

usp10/tests: Testing of EnumLanguageGroupLocalesA
by Paul Vriens Dec. 17, 2006

Dec. 17, 2006

Hi, usp10/tests/usp10.c contains a test for EnumLanguageGroupLocalesA. The test that uses this function is not using any other functions. EnumLanguageGroupLocalesA is a kernel32 function and Win98 and NT4 don't have this function. The question now is should I change the usp10.c tests to check for the availability of EnumLanguageGroupLocalesA or just move the complete test to the kernel32 ones (where the availability is already tested). Cheers, Paul.

2 1

Re: WineD3D: paletted texture emulation using fragment shaders
by Chris Robinson Dec. 17, 2006

Dec. 17, 2006

On Saturday 16 December 2006 13:24, Roderick Colenbrander wrote: > Hi, > > This patch adds 8-bit paletted texture emulation support using a fragment > shader. Won't this break for programs that attempt to use paletted textures and fragment shaders itself? If you need to emulate paletted textures, IMHO, a better option would be to hold the palette and 8-bit image in RAM, and just convert it to RGB(A) when either's changed. Though hardware has dropped native support for paletted textures, so, also IMHO, that should be done on the ddraw side, instead of making all d3d apps think paletted textures are available.

3 3

Re: [1] WineD3D: Add the state dirtification infrastructure
by Stefan Dösinger Dec. 17, 2006

Dec. 17, 2006

Am Freitag 15 Dezember 2006 18:31 schrieb Stefan Dösinger: > This patch adds the dirty list infrastructure. A dirty marker consists of a > DWORD containing the state it marks dirty, and the list structure to build > the list. > > The wined3ddevice has 3 new members: > dirtyStateList: The list of dirty states. This will be moved to a > per-context stucture once we add multithreading support > freeListElems: A list containing unused dirty markers to avoid unnecessary > HeapAlloc/HeapFree calls. I have seen D3D games spending 15% cpu time in > the Heap code, we don't want to increase that. > dirtyStates: An array that tells if a state is dirty. It is used to avoid > double dirtification. I do not use the stateblock->changed.XXXX[] array > because this is used for different purposes in the stateblock code. Ivan, > do we still need it? Vitaly asked me on irc why I use a linked list instead of an array or bitmap. Well, my original plan was indeed an array, but after discussing it with Henri I decided to got for a list. See http://www.winehq.org/pipermail/wine-devel/2006-October/051868.html and following posts. The main arguments for a list were: * The shader constant code uses it too. It keeps the code somewhat simmilar * memory usage. An array has a fixed sizeof(DWORD) * STATE_HIGHEST mem usage, while a list has sizeof(DWORD) + 2*sizeof(void *) * number of dirty states memory usage. well, that was the idea In fact, the initial dirtification kills the memory usage argument. However, instead of dirtifying all the elements I could set the recording flag in the device on before initializing the initial stateblock and later apply the states instead of dirtifying them. When there are multiple contexts those can share the list of the free dirty markers. The way dirtification is done and the rest of the code are independent from each other, so it can be changed easilly when we decide to do so. I prefer to get the state management running first, and then move the initial opengl setup to Init3D().

2 2

Re: winsock.h undefined symbols ___wine_ulong_swap ___wine_ushort_swap
by Pierre d'Herbemont Dec. 16, 2006

Dec. 16, 2006

> Pierre's fix regarding use of hton and ntoh functions on Big Endian > when including from winsock2.h did not > completely resolve compilation errors. Sanjay, it did fix the compilation error, but it seems, looking at your diff, that you're wine sources are mangled : you have mixed the first patch and the second one that were sent to wine-patches. > diff -urN /Darwine/winesrc/wine-0.9.27/include/winsock.h /Darwine/ > winesrc/wine-0.9.27_compiled/include/winsock.h > --- /Darwine/winesrc/wine-0.9.27/include/winsock.h 2006-12-08 > 16:34:42.000000000 -0500 > +++ /Darwine/winesrc/wine-0.9.27_compiled/include/winsock.h > 2006-12-15 16:14:59.000000000 -0500 > @@ -516,12 +516,14 @@ Looking at the way you diff, I think you shouldn't send your patch against version release, git [1] would be more appropriate, and won't have let you send a patch that cannot be applied. [1] http://www.winehq.org/site/git http://www.winehq.org/site/sending_patches Pierre.

1 0

Re: setupapi: add SPFILENOTIFY_TARGETNEWER support to SetupDefaultQueueCallback
by Alexandre Julliard Dec. 16, 2006

Dec. 16, 2006

Robert Reif <reif(a)earthlink.net> writes: > Index: dlls/setupapi/queue.c > =================================================================== > RCS file: /home/wine/wine/dlls/setupapi/queue.c,v > retrieving revision 1.24 > diff -p -u -r1.24 queue.c > --- dlls/setupapi/queue.c 14 Nov 2006 11:15:45 -0000 1.24 > +++ dlls/setupapi/queue.c 15 Dec 2006 16:38:56 -0000 > @@ -1379,6 +1379,28 @@ UINT WINAPI SetupDefaultQueueCallbackA( > case SPFILENOTIFY_NEEDMEDIA: > TRACE( "need media\n" ); > return FILEOP_SKIP; > + case SPFILENOTIFY_TARGETNEWER: > + TRACE( "target newer? Source: %s Target: %s\n", > + debugstr_a(paths->Source), debugstr_a(paths->Target) ); > + { > + WIN32_FILE_ATTRIBUTE_DATA Source, Target; > + > + if (GetFileAttributesExA(paths->Source, GetFileExInfoStandard, &Source)) > + { > + if (GetFileAttributesExA(paths->Target, GetFileExInfoStandard, &Target)) > + { > + TRACE( "Target = 0x%08x%08x, Source = 0x%08x%08x\n", > + Target.ftCreationTime.dwHighDateTime, Target.ftCreationTime.dwLowDateTime, > + Source.ftCreationTime.dwHighDateTime, Source.ftCreationTime.dwLowDateTime ); > + > + if (CompareFileTime(&Target.ftCreationTime, &Source.ftCreationTime) == 1) > + return TRUE; > + > + return FALSE; > + } > + } > + } That doesn't make sense to me. At this point the file is known to be newer from the version info, why would you want to check creation time and overwrite a newer file? -- Alexandre Julliard julliard(a)winehq.org

1 0

Call to all packagers (especially Debian)
by Vitaliy Margolen Dec. 16, 2006

Dec. 16, 2006

It seems there is no end to how far will packagers go to brake Wine by trying to make it better I thought it's been fixed a long time ago, but it seems not. Wine packages for Debian split important Wine parts into separate packages: - wine - fonts, few not essential programs, symlinks and _premade_ content of ~/.wine directory (with ~/.wine/c as default c: drive)!!! - libwine - all the builtin dlls - wine-utils:(explorer, winecfg, winepath, cmd.exe, iexplore, winedbg) and many other packages. Also as default it's using "winelauncher" instead of standard "wine" binary. That makes it that much more complicated to troubleshoot any problems that users might have. So in case anyone having problems with Debian or Debian based distro, please check that all the "*wine*" packages installed. Also is there a way we can request packagers to follow some standard to how they package Wine? Can we ask packagers to package all parts of Wine into one single package? If they prefer, they can package optional sound drivers separately (arts, esd, jack, nas). Same for documentation, and development headers. However everything else is essential to Wine and most programs that ran under it. Also can packagers keep default method of starting Wine the same without using any additional scripts for starting Wine? And of course, if any alterations has been made, state so in the distro specific readme file. Of course I realize that this is open source and anyone can do whatever they pleased. But please, that's make it more supportable. It's such a huge PITA to waste several hours trying to find the reason why something doesn't work to realize that person doesn't have explorer. Needless to say that lots of things won't work right. Vitaliy.

6 6

Cast-qual warnings: current status
by Andrew Talbot Dec. 16, 2006

Dec. 16, 2006

Hi, Apart from any oversights, I have completed my run of cast-qual fixes for the "dlls" and "dlls/tests" directories. The one file that most distorts the picture when -Wcast-qual is applied is "unicode.h", since its wide-string functions are inlined and it is included quite widely. I have posted a patch (dated 16 Dec 2006, entitled "include: Cast-qual warnings fix using const_cast macro") that, if not accepted, you can apply locally, if you want to build with "-Wcast-qual" to see the extent of the remaining problem. For the places where we still must use casts, I propose the introduction of a macro that takes on the role of the "const_cast" in C++. My attempt at that is as follows. #ifdef __cplusplus #define const_cast(t, e) const_cast<t>(e) #else #define const_cast(t, e) (t)(ULONG_PTR)(e) #endif This will make its purpose stand out from those of other casts. So, for example, the definition of strchrW() would become: extern inline WCHAR *strchrW( const WCHAR *str, WCHAR ch ) { do { if (*str == ch) return const_cast(WCHAR *, str); } while (*str++); return NULL; } The positive point of aiming to add -Wcast-qual universally - I would argue - is that it lets the compiler watch out for inadvertent introductions of fresh violations, so I become redundant, and you proper programmers can concentrate on more interesting things. :) I invite you, please, to debate the above issues, so we can determine the way forward concerning this issue. Meanwhile, I shall move on to the few "programs" and "server" files that generate cast-qual warnings. Thanks, -- Andy.

1 0

Re: [5/5] usp10: remote todo_wine
by James Hawkins Dec. 16, 2006

Dec. 16, 2006

On 12/14/06, Clinton Stimpson <cjstimpson(a)utwire.net> wrote: > Hi, > > Part 5 of 5. > Remove many todo_wine's from the tests, now that the functions are > implemented. > You have to remove the todo_wine's in the same patch that fixes the tests, or the tests will fail for at least one commit. Patches have to be atomic and error free, and a failing test is an error. -- James Hawkins

3 4

Re: [AppDB] Add generic sound help (resend)
by Chris Morgan Dec. 16, 2006

Dec. 16, 2006

Is it ok if we link to this entry in the Wine documentation instead: http://winehq.org/site/docs/wineusr-guide/config-wine-main#AEN282 Chris On Friday 15 December 2006 12:27 pm, Alexander Nicolaysen Sørnes wrote: > (I assume this was overlooked.) > > Add a paragraph about sound problems on the bottom of the generic help page > (as most sound problem reports are due to incorrect configurations). > > Also update the outdated usage of --debugmsg. > > > Regards, > > Alexander N. Sørnes

1 0

Re: msi: Enable cab compression and delete any resulting cabs using SHFileOperation
by Alexandre Julliard Dec. 15, 2006

Dec. 15, 2006

"James Hawkins" <truiken(a)gmail.com> writes: > +static void delete_cab_files(void) > +{ > + SHFILEOPSTRUCT shfl; > + CHAR path[MAX_PATH]; > + > + lstrcpyA(path, CURR_DIR); > + lstrcatA(path, "\\*.cab\0"); > + > + shfl.hwnd = NULL; > + shfl.wFunc = FO_DELETE; > + shfl.pFrom = (LPCSTR)path; > + shfl.pTo = NULL; > + shfl.fFlags = FOF_FILESONLY | FOF_NOCONFIRMATION | FOF_NORECURSION | FOF_SILENT; > + > + SHFileOperation(&shfl); > +} If you are going to delete files with wildcards, you need to create a temp dir and do everything inside it, otherwise you risk deleting files that are not part of the test. -- Alexandre Julliard julliard(a)winehq.org

1 0