August 2010 - Wine-Devel - list.winehq.org

Re: wiki slightly broken still?

by Dimi Paun

On Tue, 2010-08-03 at 14:30 +0200, Alexandru Băluț wrote: > How difficult would it be to use ReCaptcha? > > http://www.google.com/recaptcha Hm, don't know. We could hack our version to support recaptcha, but I'm not familiar with the code base, and I don't have the time right now. But I can take patches if someone is willing to do it. -- Dimi Paun <dimi(a)lattica.com> Lattica, Inc.

Aug. 3, 2010

Re: [PATCH 1/6] winealsa.drv: Init mixer on cards with a single Mic control, like snd_usb_audio mics (try 3)

by Vitaliy Margolen

On 08/02/2010 05:48 PM, Jeff Cook wrote: > Moved check to conform with C90. > > + if (mastelem) { > + /* Master control */ ... > + } else { Please follow file's format. Vitaliy

Aug. 3, 2010

Re: [PATCH 6/6] winealsa.drv: Open default device only once per card and don't discard non-defaults. (resend)

by Vitaliy Margolen

On 08/02/2010 05:50 PM, Jeff Cook wrote: > - > + You adding trailing spaces here. > + } else { > + default_used = 1; > } > Please follow files' format - curly braces go on separate line. Also you don't need braces for one line "else". Vitaliy.

Aug. 3, 2010

Re: [PATCH 3/6] winealsa.drv: Always scan devices, even if a default device exists. (resend)

by Vitaliy Margolen

On 08/02/2010 05:49 PM, Jeff Cook wrote: > --- > dlls/winealsa.drv/waveinit.c | 6 +++++- > 1 files changed, 5 insertions(+), 1 deletions(-) > > - int scan_devices = (fixedpcmdev == -1); > + /* we always want to scan_devices; otherwise, if a default is set and the > + appropriate registry keys are not set, only the default device will be seen > + by wine. This is undesirable, for instance, if you have separate cards for > + capture and playback, like a USB microphone for capture */ > + int scan_devices = 1; Please make it possible to force old behavior via registry. This was done for a reason - some devices were crashing during initialization, some were taking way too long to initialize. Vitaliy

Aug. 3, 2010

Re: [PATCH 4/6] winealsa.drv: Append hw address to handle to prevent name collision. (resend)

by Vitaliy Margolen

On 08/02/2010 05:49 PM, Jeff Cook wrote: > --- > dlls/winealsa.drv/waveinit.c | 9 +++++++-- > 1 files changed, 7 insertions(+), 2 deletions(-) > + sprintf(description, "%s %d:%d", snd_pcm_info_get_name(infop), snd_pcm_info_get_card(infop), Please use snprintf to prevent possible buffer overflow. > + memcpy(ww->ds_desc.szDesc, description, > + min( (sizeof(ww->ds_desc.szDesc) - 1), strlen(description)) ); This does not guarantee that ww->ds_desc.szDesc will be \0 terminated. Vitaliy.

Aug. 3, 2010

Re: [PATCH 2/5] gameux: Add stub of IGameStatisticsMgr implementation (try 2)

by Mariusz Pluciński

W dniu 03.08.2010 13:28, Nikolay Sivov pisze: > On 8/3/2010 13:58, Mariusz Pluciński wrote: >> --- >> dlls/gameux/Makefile.in | 1 + >> dlls/gameux/factory.c | 5 ++ >> dlls/gameux/gamestatistics.c | 153 >> ++++++++++++++++++++++++++++++++++++++++++ >> dlls/gameux/gameux_private.h | 1 + >> dlls/gameux/regsvr.c | 9 +++ >> 5 files changed, 169 insertions(+), 0 deletions(-) >> create mode 100644 dlls/gameux/gamestatistics.c >> + TRACE("(%p, %s, 0x%x, %p, %p)\n", iface, debugstr_w(GDFBinaryPath), >> openType, pOpenResult, ppiStats); >> + FIXME("stub\n"); >> + return E_NOTIMPL; > No need for duplicated traces, add one FIXME with full parameter output > in it. > I hope my patches are acceptable despite of this. But in the future I'll avoid it, thanks.

Aug. 3, 2010

Re: wiki slightly broken still?

by Alexandru Băluț

On Thu, Jul 29, 2010 at 18:01, Dimi Paun <dimi(a)lattica.com> wrote: > Now that this issue is fixed, we can look again at the spam > problem. It was suggested that we use a 'TextChas' for non > logged in users: > http://moinmo.in/HelpOnSpam > > But it seems it's not too easy to come up with decent questions. > Should we try it? How difficult would it be to use ReCaptcha? http://www.google.com/recaptcha Thanks, Alex

Aug. 3, 2010

Re: [PATCH 2/5] gameux: Add stub of IGameStatisticsMgr implementation (try 2)

by Nikolay Sivov

On 8/3/2010 13:58, Mariusz Pluciński wrote: > --- > dlls/gameux/Makefile.in | 1 + > dlls/gameux/factory.c | 5 ++ > dlls/gameux/gamestatistics.c | 153 > ++++++++++++++++++++++++++++++++++++++++++ > dlls/gameux/gameux_private.h | 1 + > dlls/gameux/regsvr.c | 9 +++ > 5 files changed, 169 insertions(+), 0 deletions(-) > create mode 100644 dlls/gameux/gamestatistics.c > + TRACE("(%p, %s, 0x%x, %p, %p)\n", iface, debugstr_w(GDFBinaryPath), openType, pOpenResult, ppiStats); > + FIXME("stub\n"); > + return E_NOTIMPL; No need for duplicated traces, add one FIXME with full parameter output in it.

Aug. 3, 2010

Re: regedit: Added Hebrew translation

by Yaron Shahrabani

How can I compile a single RC file? Yaron Shahrabani <Hebrew translator> On Tue, Aug 3, 2010 at 12:39 PM, Alexandre Julliard <julliard(a)winehq.org>wrote: > Yaron Shahrabani <sh.yaron(a)gmail.com> writes: > > > Hope everything is cool this time :) > > +IDD_EXPORT_TEMPLATE DIALOG 50, 50, 278, 54 > > +STYLE DS_3DLOOK | DS_CONTROL | WS_CHILD | WS_VISIBLE | WS_CLIPSIBLINGS | > > + WS_BORDER > > +WS_EX_LAYOUTRTL > > +FONT 8, "MS Shell Dlg" > > That's not valid syntax. Please try to compile at least the files you > are changing, if you can't do a full build. > > -- > Alexandre Julliard > julliard(a)winehq.org >

Aug. 3, 2010

Re: [5/5] wined3d: Fix NaN and Inf generation in msvc

by Stefan Dösinger

Am 03.08.2010 um 02:47 schrieb Vitaly Budovski: > The INFINITY macro is part of C99, but for NaN, why not use something > like sqrt(-1) ? We don't use C99 features in Wine, some compilers don't support them. sqrt(-1) may or may not work, but it will be essentially the same as 0.0 / 0.0. Compilers may complain about sqrt(-1) just as they do with the 0.0 / 0.0 division.

Aug. 3, 2010

Re: [PATCH 5/6] winealsa.drv: Don't open the same card on every loop. (resend)

by Alexandre Julliard

Jeff Cook <jeff(a)deserettechnology.com> writes: > @@ -753,7 +753,7 @@ static int ALSA_ScanDevices(int directhw, > char *pcmname = NULL; > snd_pcm_t *pcm; > > - sprintf(defaultpcmname, "default"); > + sprintf(defaultpcmname, "default:%d", card); We used to do that, but it was changed because it's not correct, cf. bug 10942. -- Alexandre Julliard julliard(a)winehq.org

Aug. 3, 2010

Re: regedit: Added Hebrew translation

by Alexandre Julliard

Yaron Shahrabani <sh.yaron(a)gmail.com> writes: > Hope everything is cool this time :) > +IDD_EXPORT_TEMPLATE DIALOG 50, 50, 278, 54 > +STYLE DS_3DLOOK | DS_CONTROL | WS_CHILD | WS_VISIBLE | WS_CLIPSIBLINGS | > + WS_BORDER > +WS_EX_LAYOUTRTL > +FONT 8, "MS Shell Dlg" That's not valid syntax. Please try to compile at least the files you are changing, if you can't do a full build. -- Alexandre Julliard julliard(a)winehq.org

Aug. 3, 2010

Re: [PATCH 2/5] explorerframe/nstc: Create the treeview on initialization. (try 2)

by Alexandre Julliard

David Hedberg <david.hedberg(a)gmail.com> writes: > +static LRESULT resize_namespacetree(NSTC2Impl *This) > +{ > + RECT rc; > + TRACE("%p\n", This); > + > + GetClientRect(This->hwnd_main, &rc); > + SetWindowPos(This->hwnd_tv, HWND_TOP, rc.left, rc.top, > + rc.right, rc.bottom, SWP_SHOWWINDOW); Again, that's not the correct usage of SetWindowPos. You don't want to show or activate it. What you probably want is MoveWindow. -- Alexandre Julliard julliard(a)winehq.org

Aug. 3, 2010

Re: [PATCH 1/6] winealsa.drv: Init mixer on cards with a single Mic control, like snd_usb_audio mics (try 2)

by Michael Stefaniuc

Jeff Cook wrote: > What compiler are you running? It works fine for me and I don't see > any errors or warnings. You are passing the compiler the wrong flags: -Wdeclaration-after-statement is set by default in Wine. > On Mon, Aug 2, 2010 at 6:05 AM, Alexandre Julliard <julliard(a)winehq.org> wrote: >> Jeff Cook <jeff(a)deserettechnology.com> writes: >> >>> @@ -245,6 +245,10 @@ static void fillcontrols(mixer *mmixer) >>> for (id = 0; id < mmixer->chans; ++id) >>> { >>> line *mline = &mmixer->lines[id]; >>> + if (!mline->elem) >>> + { >>> + break; >>> + } >>> int ofs = CONTROLSPERLINE * id; >> You can't do that: >> >> mixer.c: In function ‘fillcontrols’: >> mixer.c:252: error: ISO C90 forbids mixed declarations and code bye michael

Aug. 3, 2010

Re: GetVolumeInformationA: please test this on MS-Windows

by Jeff Zaroyko

On Mon, Aug 2, 2010 at 9:43 PM, <Joerg-Cyril.Hoehle(a)t-systems.com> wrote: > Jeff Zaroyko wrote: > >>Would you mind supplying a new binary? > Testbot did it for us: job #4154 > https://testbot.winehq.org/JobDetails.pl?Key=4154 > > Regards, > Jörg Höhle This new #4154 binary doesn't have any failures when run from the MinGW shell: volume.c:345: Tests skipped: Please re-run from another device than C: volume: 392 tests executed (0 marked as todo, 0 failures), 1 skipped. Only the earlier binary from #4014 does as per the earlier email. Jeff

Aug. 3, 2010

Re: [PATCH 04/10] ddraw/tests: New visual back buffer flipping tests. (try 2)

by Oldřich Jedlička

Hi Stefan, On Tuesday 03 August 2010 04:45:57 testbot(a)testbot.winehq.org wrote: > Hi, > > While running your changed tests on Windows, I think I found new failures. > Being a bot and all I'm not very good at pattern recognition, so I might be > wrong, but could you please double-check? > Full results can be found at > http://testbot.winehq.org/JobDetails.pl?Key=4179 > > Your paranoid android. > > > === W2KPROSP4 (32 bit visual) === > visual.c:3027: Test failed: got R FF G 00 B 00, expected R FF G FF B FF this looks like a broken driver and I should fix this by adding a broken() check, right? Oldřich.

Aug. 3, 2010

Re: [PATCH 09/10] ddraw: Add checking of SURFACEDESC type into CreateSurface methods. (try 2)

by testbot＠testbot.winehq.org

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=4188 Your paranoid android. === W98SE (32 bit dsurface) === No test summary line found

Aug. 3, 2010

Re: [PATCH 08/10] ddraw: Allow attaching backbuffers and primaries. (try 2)

by testbot＠testbot.winehq.org

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=4187 Your paranoid android. === W2KPROSP4 (32 bit visual) === visual.c:3027: Test failed: got R FF G 00 B 00, expected R FF G FF B FF === W98SE (32 bit dsurface) === No test summary line found

Aug. 3, 2010

Re: [PATCH 07/10] ddraw: Allow creating back buffer for DirectX 1 interfaces. (try 2)

by testbot＠testbot.winehq.org

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=4186 Your paranoid android. === W2KPROSP4 (32 bit visual) === visual.c:3027: Test failed: got R FF G 00 B 00, expected R FF G FF B FF === W98SE (32 bit dsurface) === No test summary line found

Aug. 3, 2010

Re: [PATCH 2/5] ws2_32/tests: test ConnectEx

by Mike Kaplinskiy

On Mon, Aug 2, 2010 at 11:50 PM, Mike Kaplinskiy <mike.kaplinskiy(a)gmail.com> wrote: > --- > dlls/ws2_32/tests/sock.c | 227 ++++++++++++++++++++++++++++++++++++++++++++++ > 1 files changed, 227 insertions(+), 0 deletions(-) > Shoot, any chance you can just take #1 and take 2-5 from the first batch? That's an outdated not working version of the patches. Otherwise I'll resend tomorrow. Sorry, Mike.

Aug. 3, 2010

Re: [PATCH 04/10] ddraw/tests: New visual back buffer flipping tests. (try 2)

by testbot＠testbot.winehq.org

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=4179 Your paranoid android. === W2KPROSP4 (32 bit visual) === visual.c:3027: Test failed: got R FF G 00 B 00, expected R FF G FF B FF

Aug. 3, 2010

Re: [PATCH 03/10] ddraw/tests: Add test for NULL/bad size of surface caps in CreateSurface. (try 2)

by testbot＠testbot.winehq.org

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=4178 Your paranoid android. === W98SE (32 bit dsurface) === No test summary line found

Aug. 3, 2010

Re: [PATCH 02/10] ddraw/tests: New attachment/flipping tests for back buffers. (try 2)

by testbot＠testbot.winehq.org

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=4175 Your paranoid android. === W98SE (32 bit dsurface) === No test summary line found

Aug. 3, 2010

Re: [PATCH 01/10] ddraw/tests: New CreateSurface for DDSCAPS_BACKBUFFER tests. (try 2)

by testbot＠testbot.winehq.org

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=4174 Your paranoid android. === W98SE (32 bit dsurface) === No test summary line found

Aug. 3, 2010

Re: [5/5] wined3d: Fix NaN and Inf generation in msvc

by Vitaly Budovski

On 02/08/2010, Henri Verbeet <hverbeet(a)gmail.com> wrote: > On 2 August 2010 13:20, GOUJON Alexandre <ale.goujon(a)gmail.com> wrote: > > Why don't you use math.h NAN macro ? > > A quick search on google gave me > > http://www.gnu.org/software/libc/manual//html_node/Infinity-and-NaN.html > > > > That's not portable enough. > The INFINITY macro is part of C99, but for NaN, why not use something like sqrt(-1) ?

Aug. 3, 2010

Re: shdocvw: Update French translation

by Frédéric Delanoy

Please discard; has been mangled by misconfigured git 2010/8/2 Frédéric Delanoy <frederic.delanoy(a)gmail.com>: > --- > dlls/shdocvw/Fr.rc | 5 +++++ > 1 files changed, 5 insertions(+), 0 deletions(-) > > diff --git a/dlls/shdocvw/Fr.rc b/dlls/shdocvw/Fr.rc > index 94b4a7e..b7326bb 100644 > --- a/dlls/shdocvw/Fr.rc > +++ b/dlls/shdocvw/Fr.rc > @@ -55,6 +55,11 @@ IDR_BROWSE_MAIN_MENU MENU > MENUITEM SEPARATOR > MENUITEM "&Propriétés...", ID_BROWSE_PROPERTIES > } > + POPUP "&Favoris" > + { > + MENUITEM "&Ajouter aux favoris..." ID_BROWSE_ADDFAV > + MENUITEM SEPARATOR > + } > POPUP "&Aide" > { > MENUITEM "À &propos d'Internet Explorer...", ID_BROWSE_ABOUT > -- > 1.7.2 > >

Aug. 3, 2010

Re: [PATCH 1/6] winealsa.drv: Init mixer on cards with a single Mic control, like snd_usb_audio mics (try 2)

by Ricardo Filipe

2010/8/2 Jeff Cook <jeff(a)deserettechnology.com>: > What compiler are you running? It works fine for me and I don't see > any errors or warnings. > > On Mon, Aug 2, 2010 at 6:05 AM, Alexandre Julliard <julliard(a)winehq.org> wrote: >> Jeff Cook <jeff(a)deserettechnology.com> writes: >> >>> @@ -245,6 +245,10 @@ static void fillcontrols(mixer *mmixer) >>> for (id = 0; id < mmixer->chans; ++id) >>> { >>> line *mline = &mmixer->lines[id]; >>> + if (!mline->elem) >>> + { >>> + break; >>> + } >>> int ofs = CONTROLSPERLINE * id; >> >> You can't do that: >> >> mixer.c: In function ‘fillcontrols’: >> mixer.c:252: error: ISO C90 forbids mixed declarations and code >> >> -- >> Alexandre Julliard >> julliard(a)winehq.org >> > > > it's not compiler related. you can't mix variable declarations with code in C90 standard. just move that check to the end of all variable declarations and it'll be fine.

Aug. 2, 2010

Re: [PATCH 1/6] winealsa.drv: Init mixer on cards with a single Mic control, like snd_usb_audio mics (try 2)

by Jeff Cook

What compiler are you running? It works fine for me and I don't see any errors or warnings. On Mon, Aug 2, 2010 at 6:05 AM, Alexandre Julliard <julliard(a)winehq.org> wrote: > Jeff Cook <jeff(a)deserettechnology.com> writes: > >> @@ -245,6 +245,10 @@ static void fillcontrols(mixer *mmixer) >> for (id = 0; id < mmixer->chans; ++id) >> { >> line *mline = &mmixer->lines[id]; >> + if (!mline->elem) >> + { >> + break; >> + } >> int ofs = CONTROLSPERLINE * id; > > You can't do that: > > mixer.c: In function ‘fillcontrols’: > mixer.c:252: error: ISO C90 forbids mixed declarations and code > > -- > Alexandre Julliard > julliard(a)winehq.org >

Aug. 2, 2010

Re: The WineAPI wiki.

by Gert van den Berg

On Mon, Aug 2, 2010 at 19:26, Max TenEyck Woodbury <max(a)mtew.isa-geek.net> wrote: > On 08/02/2010 01:04 PM, Gert van den Berg wrote: > >> There seem to be several interfaces to retreive MSDN articles... Some >> of those interface might be more stable / provide a way to retrieve a >> current link? (Framing the content would probably not be allowed, but >> retreiving links should be...) >> > If it *looks* like copying, it should be avoided. What I meant is, that it would allow for things like some really nice side-to-side postings of Wine's documentation and MSDN... It is unlikely to be legal though... (And should therefore be avoided) > I think it will be necessary to regenerate articles as the Wine project > evolves. I regenerated the 'dlls' page after Alexandre's CVS run today. > There was some new content so I saved the result. I've regenerated the > individual DLL pages several times as I improved the generating > scripts. Both activities put a load on me that I am trying to automate. > In fact, if you look at the original post before it got hijacked into a > discussion of the project name, it asks about a way to improve that > automation. > > Since there will be fairly frequent semi-intelligent reviews of pages, > such queries can probably be incorporated into that process. > > Can you help me with this, please? I can give an overview of what I think... I might do a bit of implementation, but finishing something bigger than a short script when I have other things to do isn't always easy... Sourceforge provides quite decent hosting in the project hosting space as well... Automating things from there might be a good idea. To handle MSDN, I can see the following (very) high level process: Posted MSDN links gets converted to point to redirect system in the project space. (The conversion retrieves and save some additional data). The redirect system will redirect the user to relevant content on current MSDN (allowing user to configure some parameters, such as view type / language) What I managed to figure out about MSDN content for the Web services documentation this far: 1. Content is identified by a Content identifier, which can be any of the following: a. GUID b. Short ID (Short (~8 character) unique identifier such as ms224917) c. Alias ("Friendly string") d. asset ID (Documented here: http://msdn.microsoft.com/en-us/magazine/cc163541.aspx Can be used to retrieve other (non-URL) identifiers) e Content URL (The URL to an MSDN page) 2. A Content key uniquely identifies an article and consists of a content identifier, locale and version 3 The version.identifies the various versions of the documented item. e.g. the .NET version / VC++ version 4. Locale identifies preferred language More detailed version: 1. Find MSDN URL and retrieve other identifiers (GetContent without a locale seem to retreive almost all the metadata ("partial match" in the docmentaiton)) 2. Store data in database, with GUID / short ID / assetID as primary key (they are always-present unique identifiers) other fields include alias, current URL. (content table) 3. Other tables: Locales and versions. Mapping tables between content and possible locales and versions hould also exist. The content table should save when the data was last updated and it should be automatically updated once it reaches a certain age and an user wants to retreive the page / select a non-default locale / version. 4. Generate new link pointing to the redirector system. The redirector system needs the unique content identifier (GUID / shortID / assetID) and preferably a version. It should be able to take a parameter to preview the URL (probably a smaller "options link") and allow the user to choose other locales / versions and generate links to those. This "content options" page should also allow problems with the redirector's link to be reported. The reports should be used to update the information for that link by querying it again from MSDN. It might require several trips to MSDN to initially add an item / to update versions and locales. This should possibly scheduled once the data get really old (somewhere between 180 days and 1 year) and be run on demand when someone requests the content from somewhere where they might want to see the other versions / locales about half the scheduled update age. This should keep the request volumes to MSDN low... MSDN changes and updates required: 1. Changes to web services interface: Update relevant parts (This shouldn't be too serious as long as the entire interface is not redesigned) 2. Link changes: Change how the URLs are generated from the identifiers in the database (I don't see an easy way to request an URL from the web services interface) Other possible extensions: The "redirector" can build a more complete view of what is available to allow documentation to be found easier (the information should only be retrieved on demand and saved) and to prevent duplicate trips to MSDN. (This should eventually provide a nice "tree" of MSDN with reliable links to the content) Gert

Aug. 2, 2010

Latest version of ddraw/tests plus fixes

by Oldřich Jedlička

Hi Stefan, Roderick, I've sent my latest version of ddraw/tests plus some fixes right now (it takes some time to get to wine-patches ML), one of them fixes bug #9008. Unfortunately I don't have much spare time (and actually I didn't have it the whole time working on patches), my spare time is almost over now, so I would appreciate some last comments - what to remove (if there is something unacceptable), what to change. But I don't have more time to enhance the backbuffers+3D interaction investigations/tests, sorry. It would be great if you let the fix for bug #9008 go into the wine tree. Thanks! Regards, Oldřich.

Aug. 2, 2010

Re: wordpad: Added Hebrew translation.

by Yaron Shahrabani

On Mon, Aug 2, 2010 at 6:33 PM, Alexandre Julliard <julliard(a)winehq.org>wrote: > Paul Vriens <paul.vriens.wine(a)gmail.com> writes: > > >> Eventually we probably have to but I'm not sure what Alexandre > >> thinks about introducing it (now). We probably also have to teach > >> our tools (like the translation site) that detects translation > >> differences/issues to cope with this. > >> > >> So I shouldn't add that for now? > > > > I don't know what Alexandre's opinion is here. > > Adding WS_EX_LAYOUTRTL is fine. It won't have any effect today, but it > will be useful as a test case once we start implementing it. And yes, > the translation site will need to cope with this (and maybe signal an > error when the RTL flag doesn't match the dialog language). > Great! Thanks a lot! And Paul, my computer can barely run the basic games that ship with Ubuntu, so compiling wine is out of question :) > > -- > Alexandre Julliard > julliard(a)winehq.org >

Aug. 2, 2010

Re: The WineAPI wiki.

by Max TenEyck Woodbury

On 08/02/2010 01:04 PM, Gert van den Berg wrote: > There seem to be several interfaces to retreive MSDN articles... Some > of those interface might be more stable / provide a way to retrieve a > current link? (Framing the content would probably not be allowed, but > retreiving links should be...) > If it *looks* like copying, it should be avoided. > .... (It might also work to try and retreive as much information > about MSDN links when they are posted, in order to allow the > links to be updated when the site stucture change?) (Doing a web > service query everytime to retreive a link would probably be slow >and inefficient...) > I think it will be necessary to regenerate articles as the Wine project evolves. I regenerated the 'dlls' page after Alexandre's CVS run today. There was some new content so I saved the result. I've regenerated the individual DLL pages several times as I improved the generating scripts. Both activities put a load on me that I am trying to automate. In fact, if you look at the original post before it got hijacked into a discussion of the project name, it asks about a way to improve that automation. Since there will be fairly frequent semi-intelligent reviews of pages, such queries can probably be incorporated into that process. Can you help me with this, please? - Max

Aug. 2, 2010

Re: The WineAPI wiki.

by Gert van den Berg

On Mon, Aug 2, 2010 at 18:16, Max TenEyck Woodbury <max(a)mtew.isa-geek.net> wrote: > There should be links to the Microsoft documentation in each article, > but Microsoft has been known to move their documentation around without > maintaining redirects. To avoid being at Microsoft's mercy in this, a > summary (*not* a duplicate) of the Microsoft articles, which would be > fair use, should be maintained. There seem to be several interfaces to retreive MSDN articles... Some of those interface might be more stable / provide a way to retrieve a current link? (Framing the content would probably not be allowed, but retreiving links should be...) http://blogs.msdn.com/b/innovation/archive/2009/04/22/launching-low-bandwid… Some more: http://blogs.msdn.com/b/innovation/archive/2009/10/26/launching-lightweight… Web service interface: http://services.msdn.microsoft.com/ContentServices/ContentService.asmx (possibly the most useful for building current links?) (It might also work to try and retreive as much information about MSDN links when they are posted, in order to allow the links to be updated when the site stucture change?) (Doing a web service query everytime to retreive a link would probably be slow and inefficient...) Gert

Aug. 2, 2010

Re: write: Added Hebrew translation

by Yaron Shahrabani

LOL, I knew I forgot something... Yaron Shahrabani <Hebrew translator> On Mon, Aug 2, 2010 at 6:13 PM, Paul Vriens <paul.vriens.wine(a)gmail.com>wrote: > On 08/02/2010 04:57 PM, Yaron Shahrabani wrote: > >> +++ b/programs/write/He.rc >> @@ -0,0 +1,30 @@ >> +/* >> + * English language support >> > > Should be 'Hebrew' of course ;) > > -- > Cheers, > > Paul. >

Aug. 2, 2010

Re: The WineAPI wiki.

by Max TenEyck Woodbury

On 08/02/2010 11:18 AM, James Mckenzie wrote: > Max TenEyck Woodbury<max(a)mtew.isa-geek.net> wrote: > >> On 08/02/2010 07:00 AM, Dmitry Timoshkov wrote: >>> Max TenEyck Woodbury<max(a)mtew.isa-geek.net> wrote: >>> >>>> I have started the WineAPI wiki at >>>> >>>> https://sourceforge.net/apps/mediawiki/wineapi/index.php?title=Main_Page >>> >>> You have chosen not very good name. There is no such a thing as Wine API, >>> Wine implements Win32 API, and doesn't specify/add anything custom to it. >>> The name "WineAPI" implies that Wine defines its own API which is not true, >>> and is confusing. >>> > This should be named, the Windows32/Window64 API as implemented by Wine > Documentation site. That is what it is. We have found shortcomings in > the MSDN documentation (I'm dealing with this right now in one function > call for Richedit.) This should be a collaborative site where Wine > developers, Windows on Wine developers and others can place information > on what Wine has implemented today, what needs to be implemented (with > case code and discoveries) and what Wine cannot implement. > Hmm, I may change the brief project description to match that. I will think about it... >> There is a lot of information specific to Wine, particularly its >> internal structure, that is not shared with Microsoft's product. >> Further, there is a little confusing and incorrect information in the >> Microsoft documentation. Bluntly, the Microsoft documentation is what >> they want it to be. We need to document what it really is. >> >> We have tried to embed API documentation in the source code. That has >> not worked as well as it could. Alexandre Julliard has said as much. I >> think that having good documentation will help the project. Not having >> good documentation can and does hurt Wine in my opinion. I also think >> that trying to prevent the creation of that documentation might be >> harmful. > > AJ has stated that documenting the Windows API and internal API calls >should NOT be documented just in the source. It bloats the source code >terribly and can make code unreadable. > > I agree that this Wiki should only be used to document the external > interfaces to Wine as we implement Windows32/Windows64 API/ABI calls > and what we have found in our testing that is different than MSDN. > We should NOT duplicate MSDN where is is proper, but rather have a > note that states MSDN and other sources have been found to be proper. > I disagree. Really good documentation includes 'Theory of Operation'. Of course we do not have that for the Microsoft code, but it *may* be helpful for our own code. I think documenting our test code would also be useful. There should be links to the Microsoft documentation in each article, but Microsoft has been known to move their documentation around without maintaining redirects. To avoid being at Microsoft's mercy in this, a summary (*not* a duplicate) of the Microsoft articles, which would be fair use, should be maintained. - Max

Aug. 2, 2010

Re: The WineAPI wiki.

by Max TenEyck Woodbury

On 08/02/2010 11:25 AM, Dmitry Timoshkov wrote: > Max TenEyck Woodbury<max(a)mtew.isa-geek.net> wrote: > >>>> Further, there is a little confusing and incorrect information in the >>>> Microsoft documentation. Bluntly, the Microsoft documentation is what >>>> they want it to be. We need to document what it really is. >>> >>> Regardless of the quality of Microsoft documentation it's still Win32 API, >>> not a Wine API. >>> >> And it is Microsoft's documentation and nominally documents their code. >> We have to document *our* code. There will be differences. Differences >> that I expect will be removed when found unless we can show that the >> Microsoft documentation is, in fact, incorrect. > > An API is Application Programming Interface, not the code or implementation > details. The API is for application programmers, not for system's analists. > If the Wine implementaion of win32 differs from Microsoft one - that's a bug, > it's not worth documenting, it's worth a bug report and fixing. > Your implication that I do not know wat 'API' stands for comes very close to being an insult. Please stop. The readable title is 'Wine API *and related documentation*'. Documentation is used for many purposes. Application programmers use it, but system's analysts also use it. Telling me that it can not be used any way but your way is arrogant. Implementation and documentation are two different things. Bugs can, and in some cases should, be documented. I am not asking you to do anything in particular. That would be arrogance on my part. - Max

Aug. 2, 2010

Re: [PATCH 1/4] wined3d: separate driver info from GPU info

by Roderick Colenbrander

On Mon, Aug 2, 2010 at 2:58 AM, Henri Verbeet <hverbeet(a)gmail.com> wrote: > On 1 August 2010 05:37, Roderick Colenbrander <thunderbird2k(a)gmail.com> wrote: >> struct driver_version_information >> { >> + WORD vendor; /* reported PCI card vendor ID */ > ... >> + enum wined3d_display_driver driver; >> +}; > That doesn't make all that much sense. > I agree that the vendor is redundant or did you mean the driver field didn't make sense? It is needed for selecting a different driver + version (especially on Intel the driver name depends on the gpu). Roderick

Aug. 2, 2010

Re: wordpad: Added Hebrew translation.

by Alexandre Julliard

Paul Vriens <paul.vriens.wine(a)gmail.com> writes: >> Eventually we probably have to but I'm not sure what Alexandre >> thinks about introducing it (now). We probably also have to teach >> our tools (like the translation site) that detects translation >> differences/issues to cope with this. >> >> So I shouldn't add that for now? > > I don't know what Alexandre's opinion is here. Adding WS_EX_LAYOUTRTL is fine. It won't have any effect today, but it will be useful as a test case once we start implementing it. And yes, the translation site will need to cope with this (and maybe signal an error when the RTL flag doesn't match the dialog language). -- Alexandre Julliard julliard(a)winehq.org

Aug. 2, 2010

Re: The WineAPI wiki.

by Dmitry Timoshkov

Max TenEyck Woodbury <max(a)mtew.isa-geek.net> wrote: > >> Further, there is a little confusing and incorrect information in the > >> Microsoft documentation. Bluntly, the Microsoft documentation is what > >> they want it to be. We need to document what it really is. > > > > Regardless of the quality of Microsoft documentation it's still Win32 API, > > not a Wine API. > > > And it is Microsoft's documentation and nominally documents their code. > We have to document *our* code. There will be differences. Differences > that I expect will be removed when found unless we can show that the > Microsoft documentation is, in fact, incorrect. An API is Application Programming Interface, not the code or implementation details. The API is for application programmers, not for system's analists. If the Wine implementaion of win32 differs from Microsoft one - that's a bug, it's not worth documenting, it's worth a bug report and fixing. -- Dmitry.

Aug. 2, 2010

Re: The WineAPI wiki.

by James Mckenzie

Max TenEyck Woodbury <max(a)mtew.isa-geek.net> wrote: >Sent: Aug 2, 2010 4:25 AM >To: wine-devel(a)winehq.org >Subject: Re: The WineAPI wiki. > >On 08/02/2010 07:00 AM, Dmitry Timoshkov wrote: >> Max TenEyck Woodbury<max(a)mtew.isa-geek.net> wrote: >> >>> I have started the WineAPI wiki at >>> >>> https://sourceforge.net/apps/mediawiki/wineapi/index.php?title=Main_Page >> >> You have chosen not very good name. There is no such a thing as Wine API, >> Wine implements Win32 API, and doesn't specify/add anything custom to it. >> The name "WineAPI" implies that Wine defines its own API which is not true, >> and is confusing. >> This should be named, the Windows32/Window64 API as implemented by Wine Documentation site. That is what it is. We have found shortcomings in the MSDN documentation (I'm dealing with this right now in one function call for Richedit.) This should be a collaborative site where Wine developers, Windows on Wine developers and others can place information on what Wine has implemented today, what needs to be implemented (with case code and discoveries) and what Wine cannot implement. >This has been discussed elsewhere on this mailing list. > Agreed. >There is a lot of information specific to Wine, particularly its >internal structure, that is not shared with Microsoft's product. >Further, there is a little confusing and incorrect information in the >Microsoft documentation. Bluntly, the Microsoft documentation is what >they want it to be. We need to document what it really is. > >We have tried to embed API documentation in the source code. That has >not worked as well as it could. Alexandre Julliard has said as much. I >think that having good documentation will help the project. Not having >good documentation can and does hurt Wine in my opinion. I also think >that trying to prevent the creation of that documentation might be >harmful. AJ has stated that documenting the Windows API and internal API calls should NOT be documented just in the source. It bloats the source code terribly and can make code unreadable. I agree that this Wiki should only be used to document the external interfaces to Wine as we implement Windows32/Windows64 API/ABI calls and what we have found in our testing that is different than MSDN. We should NOT duplicate MSDN where is is proper, but rather have a note that states MSDN and other sources have been found to be proper. As to the griping, we need to be public with our progress. This eliminates confusion and also gives new developers a place to start. James McKenzie

Aug. 2, 2010

Re: wordpad: Added Hebrew translation.

by Paul Vriens

On 08/02/2010 05:04 PM, Yaron Shahrabani wrote: > On Sun, Aug 1, 2010 at 4:59 PM, Paul Vriens <paul.vriens.wine(a)gmail.com > <mailto:paul.vriens.wine(a)gmail.com>> wrote: > > Hi Yaron, > > > On 08/01/2010 10:52 AM, Yaron Shahrabani wrote: > > Alexander and Paul, please notice I added the EXSTYLE > WS_EX_LAYOUTRTL > for the dialogs, hope that helps. > > Also found out that menus does not need any special RTL > property, they > are self aligned according to the app. > > > What do you mean by that? Running Wordpad in Hebrew still gives me > LTR menu styles/justification. The global (per process) RTL settings > still need to be implemented for our internal apps. > > Of course but menus don't have a special propery meaning that there is > no special addition to the POPUP object... (like WS_EX_LAYOUTRTL for > windows) Ok, I thought you meant that it magically worked for you. > Eventually we probably have to but I'm not sure what Alexandre > thinks about introducing it (now). We probably also have to teach > our tools (like the translation site) that detects translation > differences/issues to cope with this. > > So I shouldn't add that for now? I don't know what Alexandre's opinion is here. > ../../../wine-git/programs/wordpad/He.rc:279:32: Error: Stringtable > ID 1416 already in use > > Is there any way to deal with such errors (I can't compile wordpad since > there's no Makefile)? I usually run a full 'make' from the root of the tree (I'm using an out-of-tree build directory btw). -- Cheers, Paul.

Aug. 2, 2010

Re: write: Added Hebrew translation

by Paul Vriens

On 08/02/2010 04:57 PM, Yaron Shahrabani wrote: > +++ b/programs/write/He.rc > @@ -0,0 +1,30 @@ > +/* > + * English language support Should be 'Hebrew' of course ;) -- Cheers, Paul.

Aug. 2, 2010

Re: wordpad: Added Hebrew translation.

by Yaron Shahrabani

On Sun, Aug 1, 2010 at 4:59 PM, Paul Vriens <paul.vriens.wine(a)gmail.com>wrote: > Hi Yaron, > > > On 08/01/2010 10:52 AM, Yaron Shahrabani wrote: > >> Alexander and Paul, please notice I added the EXSTYLE WS_EX_LAYOUTRTL >> for the dialogs, hope that helps. >> >> Also found out that menus does not need any special RTL property, they >> are self aligned according to the app. >> > > What do you mean by that? Running Wordpad in Hebrew still gives me LTR menu > styles/justification. The global (per process) RTL settings still need to be > implemented for our internal apps. > Of course but menus don't have a special propery meaning that there is no special addition to the POPUP object... (like WS_EX_LAYOUTRTL for windows) > > >> Kind regards, >> Yaron Shahrabani >> >> <Hebrew translator> >> >> >> >> 0002-wordpad-Added-Hebrew-translation.patch >> >> >> From 8f5dfcb55656e7e4d7943d622234fb37753210f7 Mon Sep 17 00:00:00 2001 >> From: Yaron Shahrabani<sh.yaron(a)gmail.com> >> Date: Sun, 1 Aug 2010 11:48:23 +0300 >> Subject: wordpad: Added Hebrew translation. >> >> --- >> programs/wordpad/He.rc | 282 >> ++++++++++++++++++++++++++++++++++++++++++ >> programs/wordpad/Makefile.in | 1 + >> 2 files changed, 283 insertions(+), 0 deletions(-) >> create mode 100644 programs/wordpad/He.rc >> >> diff --git a/programs/wordpad/He.rc b/programs/wordpad/He.rc >> new file mode 100644 >> index 0000000..091ec98 >> --- /dev/null >> +++ b/programs/wordpad/He.rc >> @@ -0,0 +1,282 @@ >> +/* >> + * Copyright 2004 by Krzysztof Foltman >> + * Hebrew Translation by Yaron Shahrabani<sh.yaron(a)gmail.com>, 2010 >> > > Minor, but why did you not use the same format (as we see in most files)? > So something like: > > /* > * Hebrew Translation of wordpad > * > * Copyright 2004 by Krzysztof Foltman > * Copyright 2010 Yaron Shahrabani<sh.yaron(a)gmail.com> > I'll keep that in mind from now on, thanks! > > +IDD_DATETIME DIALOG 30, 20, 130, 80 >> +STYLE DS_MODALFRAME | WS_CAPTION | WS_SYSMENU >> +EXSTYLE WS_EX_LAYOUTRTL >> > > Eventually we probably have to but I'm not sure what Alexandre thinks about > introducing it (now). We probably also have to teach our tools (like the > translation site) that detects translation differences/issues to cope with > this. > So I shouldn't add that for now? > > +STRINGTABLE >> +BEGIN >> + STRING_ALIGN_LEFT, "שמאל" >> + STRING_ALIGN_RIGHT, "ימין" >> + STRING_ALIGN_CENTER, "מרכז" >> +END >> > > This piece is already defined starting on line 226 and thus leads to: > > ../../../wine-git/programs/wordpad/He.rc:279:32: Error: Stringtable ID 1416 > already in use > Is there any way to deal with such errors (I can't compile wordpad since there's no Makefile)? How come the original file compiles while the translated one doesn't? What's the difference? they have the exact same resources. Kind regards, Yaron Shahrabani > > -- > Cheers, > > Paul. >

Aug. 2, 2010

Re: The WineAPI wiki.

by Max TenEyck Woodbury

On 08/02/2010 07:53 AM, Dmitry Timoshkov wrote: > Max TenEyck Woodbury<max(a)mtew.isa-geek.net> wrote: > >>> You have chosen not very good name. There is no such a thing as Wine API, >>> Wine implements Win32 API, and doesn't specify/add anything custom to it. >>> The name "WineAPI" implies that Wine defines its own API which is not true, >>> and is confusing. >>> >> This has been discussed elsewhere on this mailing list. >> >> There is a lot of information specific to Wine, particularly its >> internal structure, that is not shared with Microsoft's product. > > Then "Wine internals" or "Wine architecture" would be more appropriate. > But it is NOT just the internals. I believe that good API documentations should include a 'Theory of Operation' section which describes how it is actually implemented. There should also be a section that documents regression and other tests. In other words, I hope this will become a *really* good set of API documentation. So 'Wine internals' would have been a bad name for the project. Similarly 'Wine architecture' is inappropriate. This is not particularly a name I chose. The other discussions of this have also used 'WineAPI'. I did consider your objections that you have voiced previously. I understand that the target API is in fact the Win32API, but the Win32API is not something we control. We do control the actual API Wine uses. There will be differences between the WineAPI and the Win32API. When those differences are pointed out, I expect the WineAPI will be changed to match the Win32API. We are free to do that. We are *not* free to change the Win32API. I think 'WineAPI' is an appropriate name and better than either of the names you suggested. If anyone comes up with a better name, I will try to get the Sourceforge project name changed, but the project is currently called 'wineapi'. So, for the moment, it is what it is. >> Further, there is a little confusing and incorrect information in the >> Microsoft documentation. Bluntly, the Microsoft documentation is what >> they want it to be. We need to document what it really is. > > Regardless of the quality of Microsoft documentation it's still Win32 API, > not a Wine API. > And it is Microsoft's documentation and nominally documents their code. We have to document *our* code. There will be differences. Differences that I expect will be removed when found unless we can show that the Microsoft documentation is, in fact, incorrect. You are setting up a 'straw man' argument. You are assuming that we can and *have* implemented the Win32 API correctly and that Microsoft's published documentation is completely correct. We have not and Microsoft has not. Our documentation will help us fix our code. We can not and should not even try to get try to get Microsoft fix their problems. From that your argument fails. - Max

Aug. 2, 2010

Re: winex11.drv: implement drag and drop from X to OLE (try 3)

by Alexandre Julliard

Damjan Jovanovic <damjan.jov(a)gmail.com> writes: > @@ -300,6 +499,66 @@ static void X11DRV_XDND_ResolveProperty(Display *display, Window xwin, Time tm, > wine_tsx11_unlock(); > } > > + /* We get the list of formats in descending order of preference, yet the linked > + * list is a stack so the order is inverted. Reverse the linked list here so > + * that we get the right order. > + */ > + current = XDNDData; You should first convert it to a standard Wine list, then you can use it in either order. > + /* For some strange reason, load-time linking SHELL32.DLL to WINEX11.DRV > + * causes wine to segfault on startup. So rather use run-time linking. > + */ > + shell32 = GetModuleHandleA("SHELL32.DLL"); > + if (shell32 == NULL) > + shell32 = LoadLibraryA("SHELL32.DLL"); A delayed import should work fine. -- Alexandre Julliard julliard(a)winehq.org

Aug. 2, 2010

Re: winex11.drv: implement X session management (try 2)

by Alexandre Julliard

Damjan Jovanovic <damjan.jov(a)gmail.com> writes: > +static void SM_Init(void) > +{ > + CreateThread(NULL, 0, manage_session, NULL, 0, NULL); > +} > +#endif /* defined(SONAME_LIBSM) */ You don't want to create a new thread in every process for this. It will cause lots of trouble. -- Alexandre Julliard julliard(a)winehq.org

Aug. 2, 2010

Re: [PATCH 07/10] explorerframe/nstc: Implement Initialize.

by Alexandre Julliard

David Hedberg <david.hedberg(a)gmail.com> writes: > + /* Native accepts a NULL prc, so should we. */ > + if(prc) > + SetWindowPos(This->hwnd_main, HWND_TOP, prc->left, prc->top, > + prc->right - prc->left, prc->bottom - prc->top, SWP_SHOWWINDOW); It would be better to create it directly with the correct rectangle. Also SWP_SHOWWINDOW is not useful since the window is created visible. And please don't add comments like "native does this", that should be true of pretty much anything we do. Native behavior should be documented by test cases anyway, not comments. -- Alexandre Julliard julliard(a)winehq.org

Aug. 2, 2010

Re: shell32: Partial implementation of IShellItem::BindToHandler.

by Nikolay Sivov

On 8/2/2010 16:07, David Hedberg wrote: > On Mon, Aug 2, 2010 at 10:50 AM, Nikolay Sivov<nsivov(a)codeweavers.com> wrote: > >> Ok, this test is present already cause Desktop list is an empty list. So >> please change this test for >> mkid.cb magic value to _ILIsEmpty() in BindToObject() and all other >> occurrences too (InitializeTreeView() for example). >> > Is something like the attached patch acceptable? I have only changed > the checks where it would not introduce an extra check for NULL, and I > have also not touched debughlp.c. Should I be more radical and also > change places that only checks pidl->mkid.cb? It's fine I think, for only pidl->mkid.cb I'd say no for now, cause I'm no sure about it. All this check thing needs to be revised and simplified probably. > Also, the check in _ILIsDesktop() seems a bit counter-intuitive to me > (interpreting it as "pidl&& (pidl->mkid.cb ? FALSE : TRUE)" is closer > to what you would expect as the current version also considers NULL > pidl's to be the desktop (or empty)), but I assume that the current > meaning is the intended one. I'm not aware about assumptions in code that NULL pidl means desktop. For now your change doesn't change functionality, but only clarifies things. Again, occurrences of "NULL means desktop" should be found and tested if possible, but later (if you'll feel brave enough after your gsoc project end, for example).

Aug. 2, 2010

Re: [PATCH 1/5] include/mswsock: add WS_ prefix if necessary

by Alexandre Julliard

Mike Kaplinskiy <mike.kaplinskiy(a)gmail.com> writes: > @@ -173,11 +174,28 @@ typedef BOOL (WINAPI * LPFN_TRANSMITFILE)(SOCKET, HANDLE, DWORD, DWORD, LPOVERLA > typedef BOOL (WINAPI * LPFN_TRANSMITPACKETS)(SOCKET, LPTRANSMIT_PACKETS_ELEMENT, DWORD, DWORD, LPOVERLAPPED, DWORD); > typedef INT (WINAPI * LPFN_WSARECVMSG)(SOCKET, LPWSAMSG, LPDWORD, LPWSAOVERLAPPED, LPWSAOVERLAPPED_COMPLETION_ROUTINE); > typedef INT (WINAPI * LPFN_WSASENDMSG)(SOCKET, LPWSAMSG, DWORD, LPDWORD, LPWSAOVERLAPPED, LPWSAOVERLAPPED_COMPLETION_ROUTINE); > +#else > +typedef BOOL (WINAPI * LPFN_ACCEPTEX)(SOCKET, SOCKET, PVOID, DWORD, DWORD, DWORD, LPDWORD, LPOVERLAPPED); > +typedef BOOL (WINAPI * LPFN_CONNECTEX)(SOCKET, const struct WS_sockaddr *, int, PVOID, DWORD, LPDWORD, LPOVERLAPPED); > +typedef BOOL (WINAPI * LPFN_DISCONNECTEX)(SOCKET, LPOVERLAPPED, DWORD, DWORD); > +typedef VOID (WINAPI * LPFN_GETACCEPTEXSOCKADDRS)(PVOID, DWORD, DWORD, DWORD, struct WS_sockaddr **, LPINT, struct WS_sockaddr **, LPINT); > +typedef BOOL (WINAPI * LPFN_TRANSMITFILE)(SOCKET, HANDLE, DWORD, DWORD, LPOVERLAPPED, LPTRANSMIT_FILE_BUFFERS, DWORD); > +typedef BOOL (WINAPI * LPFN_TRANSMITPACKETS)(SOCKET, LPTRANSMIT_PACKETS_ELEMENT, DWORD, DWORD, LPOVERLAPPED, DWORD); > +typedef INT (WINAPI * LPFN_WSARECVMSG)(SOCKET, LPWSAMSG, LPDWORD, LPWSAOVERLAPPED, LPWSAOVERLAPPED_COMPLETION_ROUTINE); > +typedef INT (WINAPI * LPFN_WSASENDMSG)(SOCKET, LPWSAMSG, DWORD, LPDWORD, LPWSAOVERLAPPED, LPWSAOVERLAPPED_COMPLETION_ROUTINE); > +#endif What you want is the WS() macro. -- Alexandre Julliard julliard(a)winehq.org

Aug. 2, 2010

Re: shell32: Partial implementation of IShellItem::BindToHandler.

by David Hedberg

On Mon, Aug 2, 2010 at 10:50 AM, Nikolay Sivov <nsivov(a)codeweavers.com> wrote: > > Ok, this test is present already cause Desktop list is an empty list. So > please change this test for > mkid.cb magic value to _ILIsEmpty() in BindToObject() and all other > occurrences too (InitializeTreeView() for example). > Is something like the attached patch acceptable? I have only changed the checks where it would not introduce an extra check for NULL, and I have also not touched debughlp.c. Should I be more radical and also change places that only checks pidl->mkid.cb? Also, the check in _ILIsDesktop() seems a bit counter-intuitive to me (interpreting it as "pidl && (pidl->mkid.cb ? FALSE : TRUE)" is closer to what you would expect as the current version also considers NULL pidl's to be the desktop (or empty)), but I assume that the current meaning is the intended one.

Aug. 2, 2010

Re: [PATCH 1/6] winealsa.drv: Init mixer on cards with a single Mic control, like snd_usb_audio mics (try 2)

by Alexandre Julliard

Jeff Cook <jeff(a)deserettechnology.com> writes: > @@ -245,6 +245,10 @@ static void fillcontrols(mixer *mmixer) > for (id = 0; id < mmixer->chans; ++id) > { > line *mline = &mmixer->lines[id]; > + if (!mline->elem) > + { > + break; > + } > int ofs = CONTROLSPERLINE * id; You can't do that: mixer.c: In function ‘fillcontrols’: mixer.c:252: error: ISO C90 forbids mixed declarations and code -- Alexandre Julliard julliard(a)winehq.org

Aug. 2, 2010

Re: [5/5] shdocvw: Recurse into directories when adding Favorites

by Alexandre Julliard

Alexander Nicolaysen Sørnes <alex(a)thehandofagony.com> writes: > @@ -109,10 +109,10 @@ static void add_fav_to_menu(HMENU menu, LPWSTR title, LPCWSTR url) > { > MENUITEMINFOW item; > /* Subtract the number of standard elements in the Favorites menu */ > - INT favcount = GetMenuItemCount(menu) - 2; > + static INT favcount; > LPWSTR urlbuf; The previous version was better, you don't want a static variable for this, you may want to do it multiple times, or the menu may get changed somewhere else. -- Alexandre Julliard julliard(a)winehq.org

Aug. 2, 2010

Re: The WineAPI wiki.

by Dmitry Timoshkov

Max TenEyck Woodbury <max(a)mtew.isa-geek.net> wrote: > > You have chosen not very good name. There is no such a thing as Wine API, > > Wine implements Win32 API, and doesn't specify/add anything custom to it. > > The name "WineAPI" implies that Wine defines its own API which is not true, > > and is confusing. > > > This has been discussed elsewhere on this mailing list. > > There is a lot of information specific to Wine, particularly its > internal structure, that is not shared with Microsoft's product. Then "Wine internals" or "Wine architecture" would be more appropriate. > Further, there is a little confusing and incorrect information in the > Microsoft documentation. Bluntly, the Microsoft documentation is what > they want it to be. We need to document what it really is. Regardless of the quality of Microsoft documentation it's still Win32 API, not a Wine API. -- Dmitry.

Aug. 2, 2010

Re: mshtml: Send more informative messages to OleInPlaceFrame

by Jacek Caban

On 08/02/10 13:30, Alexander Nicolaysen Sørnes wrote: > These are displayed in Internet Explorer. > > > Alexander N. Sørnes > +LPCWSTR get_statustext(INT id, BOOL has_arg, LPCWSTR arg_in) WHy do you need has_arg argument? Isn't NULL arg_in enough to recognize no argument? +{ + static LPWSTR strings[NUM_STATUS_STRINGS]; + int index = id - IDS_STATUS_DONE; + DWORD maxlen = 255; + DWORD len = maxlen; + LPWSTR* p = &strings[index]; + + if(!*p) + { + if(has_arg) + len *= 3; + + *p = heap_alloc(len * sizeof(WCHAR)); + len = LoadStringW(hInst, id, *p, len); + len++; + + if(!has_arg) + *p = heap_realloc(*p, len * sizeof(WCHAR)); + } + + if(has_arg) + { + LPWSTR p2 = strings[index] + maxlen; + LPCWSTR arg = arg_in; + + if(!arg) + { + const WCHAR empty = 0; + arg = ∅ + ERR("Missing arg for message (id=%d) '%s'\n", id, debugstr_w(strings[index])); + } + snprintfW(p2, (maxlen * 2) - 1, strings[index], arg); + + return p2; + } + + return *p; +} This caching mechanism is broken and not thread safe. + LPCWSTR text = get_statustext(IDS_STATUS_DONE, FALSE, NULL); + IOleInPlaceFrame_SetStatusText(doc_obj->frame, text); The helper function should take care of calling SetStatusText. if(doc->frame) - IOleInPlaceFrame_SetStatusText(doc->frame, NULL /* FIXME */); + { + LPCWSTR text = get_statustext(IDS_STATUS_DOWNLOADINGFROM, TRUE, task->url); + CoTaskMemFree(task->url); + IOleInPlaceFrame_SetStatusText(doc->frame, text); + } You leak task->url here if doc->frame is NULL. Jacek

Aug. 2, 2010

GetVolumeInformationA: please test this on MS-Windows

by Joerg-Cyril.Hoehle＠t-systems.com

Jeff Zaroyko wrote: >Would you mind supplying a new binary? Testbot did it for us: job #4154 https://testbot.winehq.org/JobDetails.pl?Key=4154 Regards, Jörg Höhle

Aug. 2, 2010

Re: [5/5] wined3d: Fix NaN and Inf generation in msvc

by Henri Verbeet

On 2 August 2010 13:20, GOUJON Alexandre <ale.goujon(a)gmail.com> wrote: > Why don't you use math.h NAN macro ? > A quick search on google gave me > http://www.gnu.org/software/libc/manual//html_node/Infinity-and-NaN.html > That's not portable enough.

Aug. 2, 2010

Re: The WineAPI wiki.

by Max TenEyck Woodbury

On 08/02/2010 07:00 AM, Dmitry Timoshkov wrote: > Max TenEyck Woodbury<max(a)mtew.isa-geek.net> wrote: > >> I have started the WineAPI wiki at >> >> https://sourceforge.net/apps/mediawiki/wineapi/index.php?title=Main_Page > > You have chosen not very good name. There is no such a thing as Wine API, > Wine implements Win32 API, and doesn't specify/add anything custom to it. > The name "WineAPI" implies that Wine defines its own API which is not true, > and is confusing. > This has been discussed elsewhere on this mailing list. There is a lot of information specific to Wine, particularly its internal structure, that is not shared with Microsoft's product. Further, there is a little confusing and incorrect information in the Microsoft documentation. Bluntly, the Microsoft documentation is what they want it to be. We need to document what it really is. We have tried to embed API documentation in the source code. That has not worked as well as it could. Alexandre Julliard has said as much. I think that having good documentation will help the project. Not having good documentation can and does hurt Wine in my opinion. I also think that trying to prevent the creation of that documentation might be harmful. - Max

Aug. 2, 2010

Re: [5/5] wined3d: Fix NaN and Inf generation in msvc

by GOUJON Alexandre

On 08/02/2010 12:00 PM, Henri Verbeet wrote: > On 31 July 2010 08:48, Stefan Dösinger<stefan(a)codeweavers.com> wrote: > >> This workaround isn't ideal, >> > Yeah, I don't think this is much of an improvement Why don't you use math.h NAN macro ? A quick search on google gave me http://www.gnu.org/software/libc/manual//html_node/Infinity-and-NaN.html

Aug. 2, 2010

Re: The WineAPI wiki.

by Dmitry Timoshkov

Max TenEyck Woodbury <max(a)mtew.isa-geek.net> wrote: > I have started the WineAPI wiki at > > https://sourceforge.net/apps/mediawiki/wineapi/index.php?title=Main_Page You have chosen not very good name. There is no such a thing as Wine API, Wine implements Win32 API, and doesn't specify/add anything custom to it. The name "WineAPI" implies that Wine defines its own API which is not true, and is confusing. -- Dmitry.

Aug. 2, 2010

The WineAPI wiki.

by Max TenEyck Woodbury

I have started the WineAPI wiki at https://sourceforge.net/apps/mediawiki/wineapi/index.php?title=Main_Page Some issues have come up that may need to be discussed here. In particular, there is going to be an 'Overview' section for many articles and I think the information to fill it in should come from the wine source tree. Specifically, I'd like to create 'Overview.wiki' files to contain this information. - Max

Aug. 2, 2010

Re: [5/5] wined3d: Fix NaN and Inf generation in msvc

by Henri Verbeet

On 31 July 2010 08:48, Stefan Dösinger <stefan(a)codeweavers.com> wrote: > This workaround isn't ideal, Yeah, I don't think this is much of an improvement.

Aug. 2, 2010

Re: [PATCH 1/4] wined3d: separate driver info from GPU info

by Henri Verbeet

On 1 August 2010 05:37, Roderick Colenbrander <thunderbird2k(a)gmail.com> wrote: > struct driver_version_information > { > + WORD vendor; /* reported PCI card vendor ID */ ... > + enum wined3d_display_driver driver; > +}; That doesn't make all that much sense.

Aug. 2, 2010

Re: [PATCH 4/4] jscript: Use generic property for Error.number.

by testbot＠testbot.winehq.org

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=4153 Your paranoid android. === W98SE (32 bit) === No test summary line found === WNT4WSSP6 (32 bit) === No test summary line found === W2KPROSP4 (32 bit) === No test summary line found === WXPPROSP3 (32 bit) === No test summary line found === W2K3R2SESP2 (32 bit) === No test summary line found === WVISTAADM (32 bit) === No test summary line found === W2K8SE (32 bit) === No test summary line found === W7PRO (32 bit) === No test summary line found === W7PROX64 (32 bit) === No test summary line found === W7PROX64 (64 bit) === No test summary line found

Aug. 2, 2010

Re: GetVolumeInformationA: please test this on MS-Windows

by Jeff Zaroyko

On Mon, Aug 2, 2010 at 7:03 PM, <Joerg-Cyril.Hoehle(a)t-systems.com> wrote: > Hi, > > Jeff Zaroyko wrote: >>No failures outside of MinGW. > Thanks. This validates the relationship between GetVolumeInformation > and the per-drive current directory on win95/98/2k/xp/7. > >>Inside MinGW, I invoke cmd to run the batch file: >>volume: 406 tests executed (0 marked as todo, 2 failures), 0 skipped. > Wait! When starting from C: as in cases a) and b) there should have been a skip. > if (currentdir[0] == windowsdir[0]) { > skip("Please re-run from another device than %c:\n", windowsdir[0]); > If properly skipped, there would have been 0 failures. > > Could you please add a > TRACE("%s vs. %s\n", currentdir, windowsdir); Would you mind supplying a new binary? My regular GNU/Linux system is dead due to hardware issues and I don't have a new enough version of flex to try building with MinGW on Windows.

Aug. 2, 2010

Re: [PATCH 3/4] jscript: Fixed test crash on some win9x installations.

by testbot＠testbot.winehq.org

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=4152 Your paranoid android. === WNT4WSSP6 (32 bit jscript) === jscript.c:388: Test failed: CoCreateInstance failed: 80040154 jscript.c:468: Test failed: CoCreateInstance failed: 80040154

Aug. 2, 2010

Re: [PATCH 2/4] jscript: Use generic properties for NaN and Infinity implementation.

by testbot＠testbot.winehq.org

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=4151 Your paranoid android. === W98SE (32 bit) === No test summary line found === WNT4WSSP6 (32 bit) === No test summary line found === W2KPROSP4 (32 bit) === No test summary line found === WXPPROSP3 (32 bit) === No test summary line found === W2K3R2SESP2 (32 bit) === No test summary line found === WVISTAADM (32 bit) === No test summary line found === W2K8SE (32 bit) === No test summary line found === W7PRO (32 bit) === No test summary line found === W7PROX64 (32 bit) === No test summary line found === W7PROX64 (64 bit) === No test summary line found

Aug. 2, 2010

GetVolumeInformationA: please test this on MS-Windows

by Joerg-Cyril.Hoehle＠t-systems.com

Hi, Jeff Zaroyko wrote: >No failures outside of MinGW. Thanks. This validates the relationship between GetVolumeInformation and the per-drive current directory on win95/98/2k/xp/7. >Inside MinGW, I invoke cmd to run the batch file: >volume: 406 tests executed (0 marked as todo, 2 failures), 0 skipped. Wait! When starting from C: as in cases a) and b) there should have been a skip. if (currentdir[0] == windowsdir[0]) { skip("Please re-run from another device than %c:\n", windowsdir[0]); If properly skipped, there would have been 0 failures. Could you please add a TRACE("%s vs. %s\n", currentdir, windowsdir); Either the path is in a strange format, or there's a case issue. Perhaps GetCurrentDir reports "C:\..." while GetWindowsDir uses "c:\..."? Then I should use toupper(currentdir[0]) == toupper(windowsdir[0]) Could you please retry the test with if (toupper(currentdir[0]) == toupper(windowsdir[0])) { skip(...) > a) C:\ # many GetVolumeInfo tests are skipped > b) C:\windows # dito > c) X:\ with \windows as CD on C: > d) X:\anysubdir with \windows as CD on C: > e) X:\anysubdir with \ as CD on C: > f) X:\ with \ as CD on C: Regards, Jörg Höhle

Aug. 2, 2010

Re: shell32: Partial implementation of IShellItem::BindToHandler.

by Nikolay Sivov

On 8/2/2010 12:07, David Hedberg wrote: > Hi, > > On Mon, Aug 2, 2010 at 7:04 AM, Nikolay Sivov<nsivov(a)codeweavers.com> wrote: >> On 8/2/2010 02:36, David Hedberg wrote: >>> +static HRESULT ShellItem_get_shellfolder(ShellItem *This, IBindCtx *pbc, >>> IShellFolder **ppsf) >>> +{ >>> + IShellFolder *desktop; >>> + HRESULT ret; >>> + >>> + ret = SHGetDesktopFolder(&desktop); >>> + if (SUCCEEDED(ret)) >>> + { >>> + if (_ILIsDesktop(This->pidl)) >>> + { >>> + *ppsf = desktop; >>> + IShellFolder_AddRef(*ppsf); >>> + } >>> + else >>> + { >>> + ret = IShellFolder_BindToObject(desktop, This->pidl, >>> pbc,&IID_IShellFolder, (void**)ppsf); >>> + } >>> + >>> + IShellFolder_Release(desktop); >>> + } >>> + >>> + return ret; >>> +} >>> + >>> >> I don't think this explicit check for desktop folder is needed here, as I >> understand ::BindToObject() it should handle this case internally. >> > I wrote a quick testcase and tested it with wine, Windows XP and > Windows 7. BindToObject() does not seem to handle this case in any of > these, instead they all return E_INVALIDARG. Wine appears to > accomplish this with the check for "!pidlComplete->mkid.cb" in > shlfolder.c:Shell32_BindToChild. So as far as I can tell, the explicit > check here seems to be needed.. Ok, this test is present already cause Desktop list is an empty list. So please change this test for mkid.cb magic value to _ILIsEmpty() in BindToObject() and all other occurrences too (InitializeTreeView() for example).

Aug. 2, 2010

Re: shell32: Partial implementation of IShellItem::BindToHandler.

by David Hedberg

Hi, On Mon, Aug 2, 2010 at 7:04 AM, Nikolay Sivov <nsivov(a)codeweavers.com> wrote: > On 8/2/2010 02:36, David Hedberg wrote: >> >> +static HRESULT ShellItem_get_shellfolder(ShellItem *This, IBindCtx *pbc, >> IShellFolder **ppsf) >> +{ >> + IShellFolder *desktop; >> + HRESULT ret; >> + >> + ret = SHGetDesktopFolder(&desktop); >> + if (SUCCEEDED(ret)) >> + { >> + if (_ILIsDesktop(This->pidl)) >> + { >> + *ppsf = desktop; >> + IShellFolder_AddRef(*ppsf); >> + } >> + else >> + { >> + ret = IShellFolder_BindToObject(desktop, This->pidl, >> pbc,&IID_IShellFolder, (void**)ppsf); >> + } >> + >> + IShellFolder_Release(desktop); >> + } >> + >> + return ret; >> +} >> + >> > I don't think this explicit check for desktop folder is needed here, as I > understand ::BindToObject() it should handle this case internally. > I wrote a quick testcase and tested it with wine, Windows XP and Windows 7. BindToObject() does not seem to handle this case in any of these, instead they all return E_INVALIDARG. Wine appears to accomplish this with the check for "!pidlComplete->mkid.cb" in shlfolder.c:Shell32_BindToChild. So as far as I can tell, the explicit check here seems to be needed..

Aug. 2, 2010

Re: gdi32: Do not crash on an invalid pointer passed to AddFontMemResourceEx.

by testbot＠testbot.winehq.org

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=4149 Your paranoid android. === W98SE (32 bit font) === Failure running script in VM: Exceeded timeout limit of 315 sec === WNT4WSSP6 (32 bit font) === Timeout

Aug. 2, 2010

Re: shell32: Partial implementation of IShellItem::BindToHandler.

by Nikolay Sivov

On 8/2/2010 02:36, David Hedberg wrote: > +static HRESULT ShellItem_get_shellfolder(ShellItem *This, IBindCtx *pbc, IShellFolder **ppsf) > +{ > + IShellFolder *desktop; > + HRESULT ret; > + > + ret = SHGetDesktopFolder(&desktop); > + if (SUCCEEDED(ret)) > + { > + if (_ILIsDesktop(This->pidl)) > + { > + *ppsf = desktop; > + IShellFolder_AddRef(*ppsf); > + } > + else > + { > + ret = IShellFolder_BindToObject(desktop, This->pidl, pbc,&IID_IShellFolder, (void**)ppsf); > + } > + > + IShellFolder_Release(desktop); > + } > + > + return ret; > +} > + > I don't think this explicit check for desktop folder is needed here, as I understand ::BindToObject() it should handle this case internally.

Aug. 2, 2010

Re: wordpad: Added Hebrew translation.

by Paul Vriens

Hi Yaron, On 08/01/2010 10:52 AM, Yaron Shahrabani wrote: > Alexander and Paul, please notice I added the EXSTYLE WS_EX_LAYOUTRTL > for the dialogs, hope that helps. > > Also found out that menus does not need any special RTL property, they > are self aligned according to the app. What do you mean by that? Running Wordpad in Hebrew still gives me LTR menu styles/justification. The global (per process) RTL settings still need to be implemented for our internal apps. > > Kind regards, > Yaron Shahrabani > > <Hebrew translator> > > > > 0002-wordpad-Added-Hebrew-translation.patch > > > From 8f5dfcb55656e7e4d7943d622234fb37753210f7 Mon Sep 17 00:00:00 2001 > From: Yaron Shahrabani<sh.yaron(a)gmail.com> > Date: Sun, 1 Aug 2010 11:48:23 +0300 > Subject: wordpad: Added Hebrew translation. > > --- > programs/wordpad/He.rc | 282 ++++++++++++++++++++++++++++++++++++++++++ > programs/wordpad/Makefile.in | 1 + > 2 files changed, 283 insertions(+), 0 deletions(-) > create mode 100644 programs/wordpad/He.rc > > diff --git a/programs/wordpad/He.rc b/programs/wordpad/He.rc > new file mode 100644 > index 0000000..091ec98 > --- /dev/null > +++ b/programs/wordpad/He.rc > @@ -0,0 +1,282 @@ > +/* > + * Copyright 2004 by Krzysztof Foltman > + * Hebrew Translation by Yaron Shahrabani<sh.yaron(a)gmail.com>, 2010 Minor, but why did you not use the same format (as we see in most files)? So something like: /* * Hebrew Translation of wordpad * * Copyright 2004 by Krzysztof Foltman * Copyright 2010 Yaron Shahrabani<sh.yaron(a)gmail.com> > +IDD_DATETIME DIALOG 30, 20, 130, 80 > +STYLE DS_MODALFRAME | WS_CAPTION | WS_SYSMENU > +EXSTYLE WS_EX_LAYOUTRTL Eventually we probably have to but I'm not sure what Alexandre thinks about introducing it (now). We probably also have to teach our tools (like the translation site) that detects translation differences/issues to cope with this. > +STRINGTABLE > +BEGIN > + STRING_ALIGN_LEFT, "שמאל" > + STRING_ALIGN_RIGHT, "ימין" > + STRING_ALIGN_CENTER, "מרכז" > +END This piece is already defined starting on line 226 and thus leads to: ../../../wine-git/programs/wordpad/He.rc:279:32: Error: Stringtable ID 1416 already in use -- Cheers, Paul.

Aug. 1, 2010

Re: user32: Add a test for GetClassInfo, make it pass under Wine. Take 2.

by testbot＠testbot.winehq.org

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=4119 Your paranoid android. === WNT4WSSP6 (32 bit class) === class.c:939: Test failed: expected ERROR_NOACCESS, got -559038737

Aug. 1, 2010

Re: New winetricks 20100731: new verbs eadm eufonts gfw glut ie8 safari secur32 xlive

by Octavian Voicu

On Sun, Aug 1, 2010 at 2:38 AM, Dan Kegel <dank(a)kegel.com> wrote: > Changes: > > Austin English: > New verbs eufonts [...] Nice, thanks! Octavian

Aug. 1, 2010

Re: New winetricks 20100731: new verbs eadm eufonts gfw glut ie8 safari secur32 xlive

by Dan Kegel

On Sat, Jul 31, 2010 at 4:38 PM, Dan Kegel <dank(a)kegel.com> wrote: > Changes: > > Austin English: > New verbs eadm eufonts gfw (aka xlive) ie8 safari secur32 I should have mentioned: ie8 doesn't quite work yet, but safari works well enough to do light browsing. I asked Austin to add the ie8 verb because some software requires it as a prerequisite, see http://bugs.winehq.org/show_bug.cgi?id=22732

Aug. 1, 2010

New wisotool 20100731: 44 new verbs (aa3, apb, ... masseffect2_demo, ... zen_of_sudoku_demo_steam)

by Dan Kegel

wisotool is a handy winetricks-like script for automatically installing games from .iso files copied from your own dvds (or, if the game is freely downloadable, it will download it). Just like winetricks, it is intended to make testing Wine easier, but might be useful for people who simply want to install games. Unlike winetricks, wisotool is quiet by default; it doesn't ask any questions (except it prompts for your password so it can mount the .iso file, and it prompts you to insert the disk if the .iso file is not found). Wisotool currently supports about 145 games and/or benchmarks (see below). Please consider contributing support for your favorite game, it's not (too) hard. Thanks to everyone who contributed changes for this release. Not all of these games currently work under wine. Workarounds for specific Wine bugs are applied if available. Wisotool is available at http://kegel.com/wine/wisotool You can read more about it at http://wiki.winehq.org/wisotool Please report bugs via http://code.google.com/p/winezeug/issues/ Changes since version 20100618: Austin English If iso isn't present, offer to make one for the user Generic Steam installer New verbs: aa3 (America's Army 3) apb (All Points Bulletin) arx_fatalis_demo_steam bookworm_adventures_deluxe_demo_steam daytona_usa_evolutions dc2ba (Dora's Carnival 2: Boardwalk Adventure) eets_demo_steam fallout3_goty (Fallout 3: Game of the Year edition) ffxivbench (Final Fantasy XIV benchmark) fifa10 fifa10_demo joint_task_force_demo_steam longest_journey_demo_steam lost_planet_dx10_demo_steam lost_planet_dx9_demo_steam metro2033 multiwinia_demo_steam napoleon_total_war_demo_steam painkiller_demo_steam sega_rally_2 singularity ssdd2 (SpongeBob SquarePants Diner Dash 2 Demo) starcraft2 total_war_and_empire_demo_steam twfc (Transformers: War for Cybertron) wowtrial (World of Warcraft trial) zen_of_sudoku_demo_steam Dan Kegel: Allow specifying a .verb file on commandline New verbs: algodoo_demo deadspace lotsn (Monkey Island: Launch of the Screaming Narwhal) masseffect2_demo secondlife_beta Eleazar Galano: New verbs: cnc3_demo (Command and Conquer 3 Demo) cnc4 fifaonline3_beta hphbp, hphbp_demo (Harry Potter Half Blood Prince) lhp (Lego Harry Potter) moh_beta (Medal of Honor) nfsshift nfsshift_demo oblivion sf4bench Jon Angelo: New verb hod3_jp_demo Thai Nguyen: New verb smbx

Aug. 1, 2010