March 2007 - Wine-Devel - list.winehq.org

Re: gdi32: Add a GetTextMetrics test, make it pass under Wine
by Lei Zhang March 9, 2007

March 9, 2007

I tested it out on a couple machines and found many failure cases. (output attached) As a sanity check, I copied the deja vu family of fonts to a windows box and they passed the tests there. On 3/9/07, Dmitry Timoshkov <dmitry(a)codeweavers.com> wrote: > Hello, > > this patch should partially resolve a problem reported in the bug 5783. > > In Wine GetTextMetricsW returns wrong tmFirstChar/tmLastChar, and > GetFontUnicodeRanges returns wrong ranges for fonts with SYMBOL_CHARSET. > > Changelog: > gdi32: Add a GetTextMetrics test, make it pass under Wine. > > --- > dlls/gdi32/font.c | 9 ++- > dlls/gdi32/freetype.c | 4 +- > dlls/gdi32/tests/font.c | 141 ++++++++++++++++++++++++++++++++++++++++++++++- > 3 files changed, 149 insertions(+), 5 deletions(-) > > diff --git a/dlls/gdi32/font.c b/dlls/gdi32/font.c > index 1b1d8b4..785a990 100644 > --- a/dlls/gdi32/font.c > +++ b/dlls/gdi32/font.c > @@ -287,10 +287,12 @@ static void FONT_TextMetricWToA(const TEXTMETRICW *ptmW, LPTEXTMETRICA ptmA ) > ptmA->tmOverhang = ptmW->tmOverhang; > ptmA->tmDigitizedAspectX = ptmW->tmDigitizedAspectX; > ptmA->tmDigitizedAspectY = ptmW->tmDigitizedAspectY; > - ptmA->tmFirstChar = ptmW->tmFirstChar > 255 ? 255 : ptmW->tmFirstChar; > + ptmA->tmBreakChar = ptmW->tmBreakChar > 255 ? 255 : ptmW->tmBreakChar; > + /* This appears to be what Windows does */ > + ptmA->tmFirstChar = ptmW->tmBreakChar; > + if (ptmA->tmFirstChar >= 2) ptmA->tmFirstChar -= 2; > ptmA->tmLastChar = ptmW->tmLastChar > 255 ? 255 : ptmW->tmLastChar; > ptmA->tmDefaultChar = ptmW->tmDefaultChar > 255 ? 255 : ptmW->tmDefaultChar; > - ptmA->tmBreakChar = ptmW->tmBreakChar > 255 ? 255 : ptmW->tmBreakChar; > ptmA->tmItalic = ptmW->tmItalic; > ptmA->tmUnderlined = ptmW->tmUnderlined; > ptmA->tmStruckOut = ptmW->tmStruckOut; > @@ -1366,6 +1368,9 @@ BOOL WINAPI GetTextMetricsW( HDC hdc, TEXTMETRICW *metrics ) > /* device layer returns values in device units > * therefore we have to convert them to logical */ > > + metrics->tmDigitizedAspectX = GetDeviceCaps(hdc, LOGPIXELSX); > + metrics->tmDigitizedAspectY = GetDeviceCaps(hdc, LOGPIXELSY); > + > #define WDPTOLP(x) ((x<0)? \ > (-abs(INTERNAL_XDSTOWS(dc, (x)))): \ > (abs(INTERNAL_XDSTOWS(dc, (x))))) > diff --git a/dlls/gdi32/freetype.c b/dlls/gdi32/freetype.c > index db990fa..03adfe5 100644 > --- a/dlls/gdi32/freetype.c > +++ b/dlls/gdi32/freetype.c > @@ -4480,7 +4480,7 @@ static DWORD get_font_unicode_ranges(FT_Face face, GLYPHSET *gs) > { > DWORD num_ranges = 0; > > - if (face->charmap->encoding == FT_ENCODING_UNICODE && pFT_Get_First_Char) > + if (pFT_Get_First_Char) > { > FT_UInt glyph_code; > FT_ULong char_code, char_code_prev; > @@ -4488,7 +4488,7 @@ static DWORD get_font_unicode_ranges(FT_Face face, GLYPHSET *gs) > glyph_code = 0; > char_code_prev = char_code = pFT_Get_First_Char(face, &glyph_code); > > - TRACE("face encoding FT_ENCODING_UNICODE, number of glyphs %ld, first glyph %u, first char %04lx\n", > + TRACE("number of glyphs %ld, first glyph %u, first char %04lx\n", > face->num_glyphs, glyph_code, char_code); > > if (!glyph_code) return 0; > diff --git a/dlls/gdi32/tests/font.c b/dlls/gdi32/tests/font.c > index 43e9c3a..a996ff0 100644 > --- a/dlls/gdi32/tests/font.c > +++ b/dlls/gdi32/tests/font.c > @@ -408,7 +408,7 @@ static void test_GdiGetCharDimensions(void) > > if (!pGdiGetCharDimensions) > { > - skip("GetFontUnicodeRanges not available on this platform\n"); > + skip("GdiGetCharDimensions not available on this platform\n"); > return; > } > > @@ -1149,6 +1149,7 @@ static void test_GetFontUnicodeRanges(void) > > size = pGetFontUnicodeRanges(hdc, gs); > ok(size, "GetFontUnicodeRanges failed\n"); > + ok(gs->cRanges != 0, "GetFontUnicodeRanges returned empty ranges\n"); > #if 0 > for (i = 0; i < gs->cRanges; i++) > trace("%03d wcLow %04x cGlyphs %u\n", i, gs->ranges[i].wcLow, gs->ranges[i].cGlyphs); > @@ -1397,6 +1398,143 @@ else > ReleaseDC(0, hdc); > } > > +static void test_text_metrics(const LOGFONTA *lf) > +{ > + HDC hdc; > + HFONT hfont, hfont_old; > + TEXTMETRICA tmA; > + TEXTMETRICW tmW; > + UINT first_unicode_char = 0, last_unicode_char = 0; > + INT ret, test_char; > + const char *font_name = lf->lfFaceName; > + > + trace("Testing font metrics for %s, charset %d\n", font_name, lf->lfCharSet); > + > + hdc = GetDC(0); > + > + SetLastError(0xdeadbeef); > + hfont = CreateFontIndirectA(lf); > + ok(hfont != 0, "CreateFontIndirect error %u\n", GetLastError()); > + > + hfont_old = SelectObject(hdc, hfont); > + > + if (pGetFontUnicodeRanges) > + { > + DWORD size; > + GLYPHSET *gs; > + > + size = pGetFontUnicodeRanges(hdc, NULL); > + ok(size, "GetFontUnicodeRanges failed unexpectedly\n"); > + > + gs = HeapAlloc(GetProcessHeap(), 0, size); > + > + size = pGetFontUnicodeRanges(hdc, gs); > + ok(size, "GetFontUnicodeRanges failed\n"); > + ok(gs->cRanges != 0, "GetFontUnicodeRanges returned empty ranges\n"); > + > + first_unicode_char = gs->ranges[0].wcLow; > + last_unicode_char = gs->ranges[gs->cRanges - 1].wcLow + gs->ranges[gs->cRanges - 1].cGlyphs - 1; > + trace("for %s first char %x, last char %x\n", font_name, > + first_unicode_char, last_unicode_char); > + > + HeapFree(GetProcessHeap(), 0, gs); > + } > + > + SetLastError(0xdeadbeef); > + ret = GetTextMetricsA(hdc, &tmA); > + ok(ret, "GetTextMetricsA error %u\n", GetLastError()); > + > + trace("A: first %x, last %x, default %x, break %x\n", > + tmA.tmFirstChar, tmA.tmLastChar, tmA.tmDefaultChar, tmA.tmBreakChar); > + > + SetLastError(0xdeadbeef); > + ret = GetTextMetricsW(hdc, &tmW); > + ok(ret, "GetTextMetricsA error %u\n", GetLastError()); > + > + trace("W: first %x, last %x, default %x, break %x\n", > + tmW.tmFirstChar, tmW.tmLastChar, tmW.tmDefaultChar, tmW.tmBreakChar); > + > + /* This appears to be what Windows does */ > + test_char = tmW.tmBreakChar; > + if (test_char >= 2) test_char -= 2; > + if (test_char > 255) test_char = 255; > + ok(tmA.tmFirstChar == test_char, "tmFirstChar for %s %02x != %02x\n", > + font_name, tmA.tmFirstChar, test_char); > + > + /* This test failes for Marlett, Wingdings 2 and Wingdings 3, but succeeds > + * for all other 327 truetype fonts on my system. > + */ > +#if 0 > + test_char = tmW.tmLastChar > 255 ? 255 : tmW.tmLastChar; > + ok(tmA.tmLastChar == test_char, "tmLastChar for %s %02x != %02x\n", > + font_name, tmA.tmLastChar, test_char); > +#endif > + if (pGetFontUnicodeRanges) > + { > + ok(tmW.tmFirstChar == first_unicode_char, "tmFirstChar for %s %02x != %02x\n", > + font_name, tmW.tmFirstChar, first_unicode_char); > +/* FIXME: remove completely once Wine is fixed > + * GetFontUnicodeRanges in Wine returns wrong data for fonts with > + * SYMBOL_CHARSET. > + */ > +if (tmW.tmLastChar != last_unicode_char) > +todo_wine ok(tmW.tmLastChar == last_unicode_char, "tmLastChar for %s %02x != %02x\n", > + font_name, tmW.tmLastChar, last_unicode_char); > +else > + ok(tmW.tmLastChar == last_unicode_char, "tmLastChar for %s %02x != %02x\n", > + font_name, tmW.tmLastChar, last_unicode_char); > + } > + > + ret = GetDeviceCaps(hdc, LOGPIXELSX); > + ok(tmW.tmDigitizedAspectX == ret, "tmDigitizedAspectX %u != %u\n", > + tmW.tmDigitizedAspectX, ret); > + ret = GetDeviceCaps(hdc, LOGPIXELSY); > + ok(tmW.tmDigitizedAspectX == ret, "tmDigitizedAspectY %u != %u\n", > + tmW.tmDigitizedAspectX, ret); > + > + SelectObject(hdc, hfont_old); > + DeleteObject(hfont); > + > + ReleaseDC(0, hdc); > +} > + > +static INT CALLBACK enum_truetype_font_proc(const LOGFONT *lf, const TEXTMETRIC *ntm, DWORD type, LPARAM lParam) > +{ > + INT *enumed = (INT *)lParam; > + > + if (type == TRUETYPE_FONTTYPE) > + { > + (*enumed)++; > + test_text_metrics(lf); > + } > + return 1; > +} > + > +static void test_GetTextMetrics(void) > +{ > + LOGFONTA lf; > + HDC hdc; > + INT enumed; > + > + SetLastError(0xdeadbeef); > + GetTextMetricsW(0, NULL); > + if (GetLastError() == ERROR_CALL_NOT_IMPLEMENTED) > + { > + skip("Skipping GetTextMetrics test on a Win9x platform\n"); > + return; > + } > + > + hdc = GetDC(0); > + > + memset(&lf, 0, sizeof(lf)); > + lf.lfCharSet = DEFAULT_CHARSET; > + enumed = 0; > + EnumFontFamiliesExA(hdc, &lf, enum_truetype_font_proc, (LPARAM)&enumed, 0); > + trace("Tested metrics of %d truetype fonts\n", enumed); > + > + ReleaseDC(0, hdc); > +} > + > START_TEST(font) > { > init(); > @@ -1427,4 +1565,5 @@ START_TEST(font) > } > else > skip("Arial Black or Symbol/Wingdings is not installed\n"); > + test_GetTextMetrics(); > } > -- > 1.5.0.2 > > > > > >

1 0

re: Writing a winelib plugin
by Dan Kegel March 9, 2007

March 9, 2007

Shachar wrote: >It used to not be possible to write a plugin, to be loaded from a >standard Linux ELF program, that will be itself a winelib shared object. >All sorts of issues regarding running wineserver and memory layout >initializations were problematic. > >I'm wondering whether there is any news on that front. Is it still not >practically possible to do this? Last time I asked about this, I got the impression that there was some hope but nobody was working on it. And I think it would require linking something statically into the program, too, so if you're trying to foist winelib shared objects onto an unsuspecting Big App, you're probably going to be out of luck even if this gets worked on. What application did you have in mind?

3 2

Re: [Bug 7503] Eve Online pasword box on login screen doesn't work
by Lei Zhang March 9, 2007

March 9, 2007

Can we change wineprefixcreate to copy the truetype fonts (if any) from /usr/X11R6/lib/X11/fonts/TTF or /usr/share/X11/fonts/TTF or [insert distro specific X11/fonts/TTF directory] to $WINEPREFIX/drive_c/windows/fonts/ ? On 2/20/07, Wine Bugs <wine-bugs(a)winehq.org> wrote: > http://bugs.winehq.org/show_bug.cgi?id=7503 > > > > > > ------- Additional Comments From kelfe(a)gmx.de 2007-20-02 16:57 ------- > to fix the issue you need to copy the font file arial.ttf into > .wine/drive_c/windows/fonts folder (.wine is a hidden folder) > > the arial.ttf can be found in the mscorefonts packages or search engine/windows box > > I don't know if font files should be placed in the fonts folder by wine or by > the user. So close this bug or keep it open until basic fonts are installed by > wine when creating the .wine folder. > > -- > Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email > ------- You are receiving this mail because: ------- > You are the assignee for the bug, or are watching the assignee. > > >

3 3

New opengl thread context selection patches for testing
by Stefan Dösinger March 9, 2007

March 9, 2007

Hi, Here is a new patchset for testing which implements duplicating gl contexts for new threads in wined3d. Again, it does not implement any synchronisation measures(except ENTER_GL and glFinish), so running multithreaded games is still kinda a lottery. This patchset also contains a patch for offscreen rendering, which may fix the problems introduced with TES: Oblivion before. This is just a format-patch of my git tree, the first 3 patches are unrelated. One of them is a clone of Henri's already sent patch, and the other 2 patches were sent by me already. Happy testing, Stefan

3 7

Re: [2/2] wined3d: IWineD3DDeviceImpl_UpdateSurface: return D3DERR_INVALIDCALL if source and dest format do not match
by Fabian Bieler March 9, 2007

March 9, 2007

Please ignore this patch.

1 0

Is there someone working on dxdiag?
by Mirek March 9, 2007

March 9, 2007

Hi, I would like to start some applications like Carmageddon TDR 2000 in HW mode, Neverwinter Nights 2 or just GetGPUAndSystemInfo.exe from Nvidia SDK demos and maybe 3DMark with HW info, but there are problems with detecting video card, I assume the main problem is in dxdiag library. So I would like to ask if there is someone working on this library, or if was there some code which was not accepted in wine tree to fix that? I am trying to hack this library, but with only small success. Mirek Slugen

3 3

Re: quartz: Check allocation failure and clear memory in DSound Renderer
by Robert Shearman March 9, 2007

March 9, 2007

Chris Robinson wrote: > return CLASS_E_NOAGGREGATION; > > pDSoundRender = CoTaskMemAlloc(sizeof(DSoundRenderImpl)); > + if (!pDSoundRender) > + return E_OUTOFMEMORY; > + ZeroMemory(pDSoundRender, sizeof(DSoundRenderImpl)); > > pDSoundRender->lpVtbl = &DSoundRender_Vtbl; > pDSoundRender->IBasicAudio_vtbl = &IBasicAudio_Vtbl; > @@ -325,9 +328,15 @@ HRESULT DSoundRender_create(IUnknown * pUnkOuter, LPVOID * ppv) > pDSoundRender->pClock = NULL; > pDSoundRender->init = FALSE; > pDSoundRender->started = FALSE; > - ZeroMemory(&pDSoundRender->filterInfo, sizeof(FILTER_INFO)); > The check for allocation failure is good, but clearing the memory that is going to be initialised anyway is inefficient and unnecessary. -- Rob Shearman

2 1

Re: [2/2] wined3d: IWineD3DDeviceImpl_UpdateSurface: return D3DERR_INVALIDCALL if source and dest format do not match
by H. Verbeet March 9, 2007

March 9, 2007

On 09/03/07, Fabian Bieler <der.fabe(a)gmx.net> wrote: > + if (destFormat != srcFormat) { > + WARN("source %p and dest %p must match, returning WINED3DERR_INVALIDCALL\n", pSourceSurface, pDestinationSurface); > + return WINED3DERR_INVALIDCALL; > + } You should probably print the actual formats there, using debug_d3dformat(). Also, a test wouldn't hurt.

1 0

Fwd: wined3d: Update cursor position on ShowCursor
by Erich Hoover March 9, 2007

March 9, 2007

Request for comments, as per http://www.winehq.com/site/sending_patches. For details on the need for this patch please see Bug #7542<http://bugs.winehq.org/show_bug.cgi?id=7542>. Thanks! Erich Hoover ehoover(a)mines.edu ---------- Forwarded message ---------- From: Erich Hoover <ehoover(a)mines.edu> Date: Mar 3, 2007 6:46 PM Subject: wined3d: Update cursor position on ShowCursor To: wine-patches(a)winehq.org Real Name: Erich Hoover Description: When ShowCursor is first called it should make the cursor appear at the OS's last known cursor position. Because of this, some applications just repetitively call ShowCursor in order to update the cursor's position. This behavior is undocumented. Changelog: wined3d: Update cursor position on ShowCursor

3 2

Re: Alexandre Julliard : advapi32: Fixed registry test that failed because of a buffer overflow.
by James Hawkins March 9, 2007

March 9, 2007

On 3/8/07, Alexandre Julliard <julliard(a)wine.codeweavers.com> wrote: > Module: wine > Branch: master > Commit: 7e2228f15aa7a6fc0d63fe36c34906035422e95a > URL: http://source.winehq.org/git/wine.git/?a=commit;h=7e2228f15aa7a6fc0d63fe36c… > > Author: Alexandre Julliard <julliard(a)winehq.org> > Date: Thu Mar 8 21:06:31 2007 +0100 > > advapi32: Fixed registry test that failed because of a buffer overflow. > The test passes in Windows, so isn't this hiding a bug in our implementation of RegSetValueW? -- James Hawkins

2 1

How does wine determine to draw non-client area according to managed mode setting?
by Sunil Baek March 9, 2007

March 9, 2007

Hello, When wine runs on managed mode, it delegate drawing non-client area(caption, frame-border) to window manager. I found NC_DrawCaption function is not invoked on managed mode. On non-managed mode, that function is invoked. I tried to find routine which determine to draw non-client area according to managed mode setting, but I can't. please give me some hints about that. Thank you.

1 0

Re: comctl32: listview tests
by Lei Zhang March 8, 2007

March 8, 2007

On 3/8/07, George Gov <ggov(a)ucla.edu> wrote: > @@ -573,10 +699,242 @@ static void test_redraw(void) > trace("invalidate & update\n"); > InvalidateRect(hwnd, NULL, TRUE); > UpdateWindow(hwnd); > - ok_sequence(sequences, LISTVIEW_SEQ_INDEX, redraw_listview_seq, "redraw listview", FALSE); > + ok_sequence(sequences, LISTVIEW_SEQ_INDEX, redraw_listview_seq, "redraw listview", TRUE); This test succeeds for me, so I'm not sure why you marked it as todo. - Lei

1 0

Re: quartz: Use proper alloc/free functions for COM objects
by Detlef Riekenberg March 8, 2007

March 8, 2007

On Do, 2007-03-08 at 03:06 -0800, Chris Robinson wrote: > Subject: [PATCH] quartz: Use proper alloc/free functions for COM > objects > + omr->messages = CoTaskMemAlloc(omr->ring_buffer_size * > sizeof(Event)); > + ZeroMemory(omr->messages, omr->ring_buffer_size * sizeof(Event)); > > + fimpl = CoTaskMemAlloc(sizeof(*fimpl)); > fimpl->IGraphBuilder_vtbl = &IGraphBuilder_VTable; > + This->ppPins = CoTaskMemAlloc((This->cStreams + 2) * sizeof(IPin > *)); > memcpy(This->ppPins, ppOldPins, (This->cStreams + 1) * > sizeof(IPin *)); > + This->ppPins = CoTaskMemAlloc(sizeof(IPin *) * 1); > memcpy(This->ppPins, ppOldPins, sizeof(IPin *) * 1); While reading your Patch, I see the above Places, where the result from the allocation is used without a NULL-check. The other allocations are checked for NULL before used. Is this correct? -- By by ... Detlef

2 1

Command and Conquer 3 Demo
by Matthew Clark March 8, 2007

March 8, 2007

I just wanted to write to let all of the devs know how proud of the wine project I am. This Demo seems to work for the most part except for the movies in it don't seem to want to resize and when you are under attack the game crashes. Here is the console dump: fixme:dsound:DSOUND_MixOne problem with underrun detection (mixlen=49332 < primary_done=49356) fixme:dsound:DSOUND_MixOne problem with underrun detection (mixlen=39800 < primary_done=39824) fixme:dsound:DSOUND_MixOne problem with underrun detection (mixlen=35960 < primary_done=35984) fixme:dsound:DSOUND_MixOne problem with underrun detection (mixlen=36008 < primary_done=36032) fixme:dsound:DSOUND_MixOne problem with underrun detection (mixlen=39968 < primary_done=39992) fixme:dsound:DSOUND_MixOne problem with underrun detection (mixlen=36128 < primary_done=36152) fixme:dsound:DSOUND_MixOne problem with underrun detection (mixlen=36128 < primary_done=36152) fixme:dsound:DSOUND_MixOne problem with underrun detection (mixlen=38072 < primary_done=38076) fixme:dsound:DSOUND_MixOne problem with underrun detection (mixlen=38072 < primary_done=38076) fixme:dsound:DSOUND_MixOne problem with underrun detection (mixlen=38072 < primary_done=38076) fixme:dsound:DSOUND_MixOne problem with underrun detection (mixlen=34232 < primary_done=34236) fixme:dsound:DSOUND_MixOne problem with underrun detection (mixlen=38072 < primary_done=38096) fixme:dsound:DSOUND_MixOne problem with underrun detection (mixlen=34232 < primary_done=34256) fixme:dsound:DSOUND_MixOne problem with underrun detection (mixlen=36152 < primary_done=36176) fixme:dsound:DSOUND_MixOne problem with underrun detection (mixlen=36152 < primary_done=36176) fixme:dsound:DSOUND_MixOne problem with underrun detection (mixlen=36152 < primary_done=36176) err:dsound:DSOUND_MixOne underrun on sound buffer 0x210b10 fixme:dsound:DSOUND_MixOne problem with underrun detection (mixlen=34568 < primary_done=34592) wine: Unhandled page fault on read access to 0x0000fff7 at address 0x7efa2553 (thread 000e), starting debugger... fixme:dsound:DSOUND_MixOne problem with underrun detection (mixlen=50292 < primary_done=50316) fixme:dsound:DSOUND_MixOne problem with underrun detection (mixlen=40784 < primary_done=40808) fixme:dsound:DSOUND_MixOne problem with underrun detection (mixlen=40784 < primary_done=40808) fixme:dsound:DSOUND_MixOne problem with underrun detection (mixlen=40784 < primary_done=40808) fixme:dsound:DSOUND_MixOne problem with underrun detection (mixlen=40784 < primary_done=40808) fixme:dsound:DSOUND_MixOne problem with underrun detection (mixlen=40784 < primary_done=40808) fixme:alsa:ALSA_AddRingMessage two fast messages in the queue!!!! toget = 52(WINE_WM_RESTARTING), tosave=53(UNKNOWN(0x00000000)) fixme:alsa:ALSA_AddRingMessage two fast messages in the queue!!!! toget = 36(WINE_WM_RESTARTING), tosave=37(UNKNOWN(0x00000000)) hope that helps. Again, greats that that is all that's wrong^_^ The graphics look great, it sounds great, and the videos are crystal clear(though small). -Matthew Clark

3 4

modification to allow -Wcast-qual to find potential warnings without unicode.h causing them
by Andrew Talbot March 8, 2007

March 8, 2007

Assuming it is valid and does what is required, maybe the following pragma would be more precise. #pragma GCC diagnostic warning "-Wwrite-strings" -- Andy.

1 2

Interesting Winelib report
by Jeremy White March 8, 2007

March 8, 2007

Hi Folks, John Gilmore alerted me to an attempted use of Winelib by the Cairo folks: http://lists.freedesktop.org/archives/cairo/2007-February/009707.html I've chatted privately with Behdad, who doesn't have a ton of time, but would be willing to poke a bit further if he had some help. I know that from time to time, people like to pick up this sort of project as a fun one, and I thought I'd see if I could spark someone into doing so with Cairo. Cheers, Jeremy

1 0

-Wcast-qual result
by Mike Schaadt March 8, 2007

March 8, 2007

I submited a patch today that modified unicode.h that disables warnings(which allows for -Wcast-qual to actually give relevant results) I don't know if it will be accepted or not(it has the potential of being problematic if modifying unicode.h that would normally result in a warning, but this patch would cause it to be suppresed), but I figured I would at least share the results I found. after configuring with cast-qual, I stored the warnings that resulted(which I've included as an attachment for those that are interested.). There were no warnings in the make depend, and make generated 545 warnings(all from .c files). The included file does not contain the sub directory for what file generates the warning. If anyone knows how to pull that data, let me know and I'll redo it and extract that information as well. I plan on start looking through these and working on fixing some here soon, but I thought I would share what I found with this group in general.

1 0

Re: wine-devel Digest, Vol 20, Issue 30
by Alessandro Pignotti March 8, 2007

March 8, 2007

> Date: Thu, 8 Mar 2007 08:22:59 +0100 > From: Kai Blin <kai.blin(a)gmail.com> > > > /* FIXME: Should we be storing these dwFlags or the creation ones? */ > > - lpPData = DP_CreatePlayer( This, lpidPlayer, lpPlayerName, dwFlags, > > + /* I think we should pass creation flags, so we can distinguish > > sysplayers and not count them in the current + player total */ > > + lpPData = DP_CreatePlayer( This, lpidPlayer, lpPlayerName, > > dwCreateFlags, hEvent, bAnsi ); > > Every time someone uses "I think" in the code, usually this calls for a > test case. This one should be rather straightforward to test. It's not possible to write a conformance test because i'm using a flag internal to wine's implementation of directplay, It's possible to safely change "I think we should" with "we have to". I've used "i think" more as an answer to the previous FIXME comment we have inherited from the author of the code. Regards Alessandro Pignotti > > 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. -- Vi Veri Veniversum Vivus Vici -Dr. Faustus - Marlowe Public GPG Key ID 0x650B3ED9 on subkeys.gpg.net Key Fingerprint 6243 AAD3 E3EC 52D8 DFAA 2A2F 9FCD 0457 650B 3ED9 Encrypted mails are welcome

2 1

Serial port problem - again
by Aleš Rom March 8, 2007

March 8, 2007

Hello. I'm new to this list. First I must say that I apologise for posting this mail again here. I hope you wine gurus can see what is going wrong here. If I run this app <http://appdb.winehq.org/appview.php?iAppId=4165> with WINEDEBUG=+relay , I can transfer data from my device to PC, but not all of it. It stops somewhere in the middle of the transfer. Without +relay option, device is not even accesible. It just say: [romcek(a)core2 ~]$ WINEDEBUG=+trace,+comm wine /home/romcek/.wine/drive_c/Program\ Files/SMA/Sunny\ Data\ Control/SDC.exe err:systray:delete_icon invalid tray icon ID specified: 0d fixme:tapi:lineInitialize (0x832f88, 0x11c0000, 0x11c2dfc, "TAPICOMM", 0x832f8c): stub. fixme:tapi:lineGetDevCapsA ((nil), 00000000, 00010003, 00000000, 0xf7eaf8): stub. fixme:tapi:lineGetDevCapsA ((nil), 00000000, 00010003, 00000000, 0xf7eb2c): stub. fixme:ras:RasEnumEntriesA ((nil),(null),0xf7fc80,0x34f5d8,0xf7ec38),stub! fixme:ras:RasEnumDevicesA ((nil),0x34f60c,0x34f608),stub! fixme:ras:RasEnumDevicesA (0x777fe0,0x34f60c,0x34f608),stub! trace:comm:GetCommState handle 0x144, ptr 0x34f9b8 trace:comm:io_control 0x144 IOCTL_SERIAL_GET_BAUD_RATE (nil) 0 0x34f988 4 0x34f914 trace:comm:io_control 0x144 IOCTL_SERIAL_GET_LINE_CONTROL (nil) 0 0x34f98d 3 0x34f914 trace:comm:io_control 0x144 IOCTL_SERIAL_GET_HANDFLOW (nil) 0 0x34f970 16 0x34f914 trace:comm:io_control 0x144 IOCTL_SERIAL_GET_CHARS (nil) 0 0x34f982 6 0x34f914 trace:comm:GetCommState OK trace:comm:dump_dcb bytesize=8 baudrate=19200 fParity=0 Parity=0 stopbits=1 trace:comm:dump_dcb ~IXON ~IXOFF trace:comm:dump_dcb fOutxCtsFlow=0 fRtsControl=0 trace:comm:dump_dcb fOutxDsrFlow=0 fDtrControl=1 trace:comm:dump_dcb ~CRTSCTS trace:comm:dump_dcb bytesize=8 baudrate=19200 fParity=0 Parity=0 stopbits=1 trace:comm:dump_dcb ~IXON ~IXOFF trace:comm:dump_dcb fOutxCtsFlow=0 fRtsControl=1 trace:comm:dump_dcb fOutxDsrFlow=0 fDtrControl=1 trace:comm:dump_dcb ~CRTSCTS trace:comm:io_control 0x144 IOCTL_SERIAL_SET_BAUD_RATE 0x34f988 4 (nil) 0 0x34f914 trace:comm:io_control 0x144 IOCTL_SERIAL_SET_LINE_CONTROL 0x34f98d 3 (nil) 0 0x34f914 trace:comm:io_control 0x144 IOCTL_SERIAL_SET_HANDFLOW 0x34f970 16 (nil) 0 0x34f914 trace:comm:io_control 0x144 IOCTL_SERIAL_SET_CHARS 0x34f982 6 (nil) 0 0x34f914 trace:comm:io_control 0x144 IOCTL_SERIAL_SET_QUEUE_SIZE 0x34f98c 8 (nil) 0 0x34f930 fixme:comm:set_queue_size insize 1024 outsize 1024 unimplemented stub trace:comm:io_control 0x144 IOCTL_SERIAL_GET_COMMSTATUS (nil) 0 0x61b196c8 20 0x61b19660 trace:comm:io_control 0x144 IOCTL_SERIAL_GET_COMMSTATUS (nil) 0 0x61b196c8 20 0x61b19660 trace:comm:io_control 0x144 IOCTL_SERIAL_GET_COMMSTATUS (nil) 0 0x61b196c8 20 0x61b19660 trace:comm:io_control 0x144 IOCTL_SERIAL_GET_COMMSTATUS (nil) 0 0x61b196c8 20 0x61b19660 trace:comm:io_control 0x144 IOCTL_SERIAL_GET_COMMSTATUS (nil) 0 0x61b196c8 20 0x61b19660 last line is repeating and then nothing hapens. It looks like delay issue to me, but I'm not a programmer. Can someone help, plese. Thanks, Ales

1 0

[winealsa] Mind buffer loops when pausing
by Chris Bandy March 8, 2007

March 8, 2007

This patch should fix the incorrect handling of WaveHdr loops when pausing. Like the non-loop case written before, this sums the buffer lengths between dwPlayedTotal and dwWrittenTotal to calculate the new dwPartialOffset. Instead of blindly moving from one buffer to the next, however, this traverses the list using BeginWaveHdr and PlayPtrNext. This has added benefit of correctly initializing all pointers and counters for playback. It looks correct on paper, but I don't seem to have any applications that even attempt to pause. Can anyone suggest an application to test this with? All feedback welcome. Note: this branch in the function is a 'software' pause. To test you may need to override the check for ALSA's hardware pause at line 418 (snd_pcm_hw_params_can_pause) -- Chris diff --git a/dlls/winealsa.drv/waveout.c b/dlls/winealsa.drv/waveout.c index 6a8a435..4ac50c6 100644 --- a/dlls/winealsa.drv/waveout.c +++ b/dlls/winealsa.drv/waveout.c @@ -319,9 +319,9 @@ static DWORD wodPlayer_NotifyCompletions(WINE_WAVEDEV* wwo, BOOL force) * * wodPlayer helper. Resets current output stream. */ -static void wodPlayer_Reset(WINE_WAVEDEV* wwo, BOOL reset) +static void wodPlayer_Reset(WINE_WAVEDEV* wwo, BOOL reset) { - int err; + int err; TRACE("(%p)\n", wwo); /* flush all possible output */ @@ -329,7 +329,7 @@ static void wodPlayer_Reset(WINE_WAVEDEV* wwo, BOOL reset) wodUpdatePlayedTotal(wwo, NULL); /* updates current notify list */ - wodPlayer_NotifyCompletions(wwo, FALSE); + wodPlayer_NotifyCompletions(wwo, reset); if ( (err = snd_pcm_drop(wwo->pcm)) < 0) { FIXME("flush: %s\n", snd_strerror(err)); @@ -345,9 +345,6 @@ static void wodPlayer_Reset(WINE_WAVEDEV* wwo, BOOL reset) DWORD param; HANDLE ev; - /* remove any buffer */ - wodPlayer_NotifyCompletions(wwo, TRUE); - wwo->lpPlayPtr = wwo->lpQueuePtr = wwo->lpLoopPtr = NULL; wwo->state = WINE_WS_STOPPED; wwo->dwPlayedTotal = wwo->dwWrittenTotal = 0; @@ -368,32 +365,34 @@ static void wodPlayer_Reset(WINE_WAVEDEV* wwo, BOOL reset) ((LPWAVEHDR)param)->dwFlags &= ~WHDR_INQUEUE; ((LPWAVEHDR)param)->dwFlags |= WHDR_DONE; - wodNotifyClient(wwo, WOM_DONE, param, 0); + wodNotifyClient(wwo, WOM_DONE, param, 0); } ALSA_ResetRingMessage(&wwo->msgRing); LeaveCriticalSection(&wwo->msgRing.msg_crst); } else { - if (wwo->lpLoopPtr) { - /* complicated case, not handled yet (could imply modifying the loop counter */ - FIXME("Pausing while in loop isn't correctly handled yet, except strange results\n"); - wwo->lpPlayPtr = wwo->lpLoopPtr; - wwo->dwPartialOffset = 0; - wwo->dwWrittenTotal = wwo->dwPlayedTotal; /* this is wrong !!! */ - } else { - LPWAVEHDR ptr; - DWORD sz = wwo->dwPartialOffset; + DWORD diff = wwo->dwWrittenTotal - wwo->dwPlayedTotal; + DWORD sz; + + /* no pointers or counters need to change */ + if (diff <= wwo->dwPartialOffset) + wwo->dwPartialOffset -= diff; + else { + wwo->lpLoopPtr = NULL; + wodPlayer_BeginWaveHdr(wwo, wwo->lpQueuePtr); - /* reset all the data as if we had written only up to lpPlayedTotal bytes */ - /* compute the max size playable from lpQueuePtr */ - for (ptr = wwo->lpQueuePtr; ptr != wwo->lpPlayPtr; ptr = ptr->lpNext) { - sz += ptr->dwBufferLength; + diff -= wwo->dwPartialOffset; + sz = wwo->lpPlayPtr->dwBufferLength; + + while (sz < diff) { + wodPlayer_PlayPtrNext(wwo); + sz += wwo->lpPlayPtr->dwBufferLength; } - /* because the reset lpPlayPtr will be lpQueuePtr */ - if (wwo->dwWrittenTotal > wwo->dwPlayedTotal + sz) ERR("grin\n"); - wwo->dwPartialOffset = sz - (wwo->dwWrittenTotal - wwo->dwPlayedTotal); - wwo->dwWrittenTotal = wwo->dwPlayedTotal; - wwo->lpPlayPtr = wwo->lpQueuePtr; + + wwo->dwPartialOffset = sz - diff; + wodPlayer_BeginWaveHdr(wwo, wwo->lpQueuePtr); } + + wwo->dwWrittenTotal = wwo->dwPlayedTotal; wwo->state = WINE_WS_PAUSED; } }

2 1

Writing a winelib plugin
by Shachar Shemesh March 8, 2007

March 8, 2007

Hi all, It used to not be possible to write a plugin, to be loaded from a standard Linux ELF program, that will be itself a winelib shared object. All sorts of issues regarding running wineserver and memory layout initializations were problematic. I'm wondering whether there is any news on that front. Is it still not practically possible to do this? Many thanks, Shachar

1 0

Re: Alexandre Julliard : wine_common_ver: Set company name to Microsoft, some apps check for that.
by Paul Chitescu March 8, 2007

March 8, 2007

Hi! Ummm, isn't this legally dangerous? Stamping a piece of code with Microsoft's name can be interpreted as counterfeiting. Maybe a better choice would be to add a configure option --with-company-name and let the users set it to... whatever... Paul Chitescu

4 3

Re: [RESEND] [PATCH] DirectPlay should keep track of the current number of players
by Kai Blin March 8, 2007

March 8, 2007

On Wednesday 07 March 2007 10:52, Alessandro Pignotti wrote: > + /* I think we have to add a player to the total number of players only if it's not a player created for system > + purpose like the name server. This way it seems we get a correct player number */ > + if( ~dwFlags & DPLAYI_PLAYER_SYSPLAYER ) > + This->dp2->lpSessionDesc->dwCurrentPlayers++; > + [...] > /* FIXME: Should we be storing these dwFlags or the creation ones? */ > - lpPData = DP_CreatePlayer( This, lpidPlayer, lpPlayerName, dwFlags, > + /* I think we should pass creation flags, so we can distinguish sysplayers and not count them in the current > + player total */ > + lpPData = DP_CreatePlayer( This, lpidPlayer, lpPlayerName, > dwCreateFlags, hEvent, bAnsi ); Every time someone uses "I think" in the code, usually this calls for a test case. This one should be rather straightforward to test. 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

RS232 latency, redirection possible?
by [IDC]Dragon March 7, 2007

March 7, 2007

Hi, I have 2 questions about the serial port handling under Wine: 1) I see a very annoying latency in an application which talks to a controller over RS232. Under Windows, the traffic is "fluent", under Wine each command has a turnaround time of almost a second. Since there are many commands, this gets useless. I sthere anything known, some settings I can try? 2) Is it possible to map the e.g. COM1 port traffic to another device, or is it firmly assigned to matching RS232 interfaces of the Linux box? Ideally, I'd like to have the Windows app talk to an XPort device (remote RS232) connected over network. A program called "socat" (http://www.dest-unreach.org/socat) can be used to create a device and tunnel the traffic over a TCP port. I'd like to map that device to a windows-emulated COM port. Maybe this even eliminates problem 1). I hope I was able to explain especially 2) and kindly wait for answers, thanks! Jörg -- Ist Ihr Browser Vista-kompatibel? Jetzt die neuesten Browser-Versionen downloaden: http://www.gmx.net/de/go/browser

3 4

wininet: Don't discard the extra info part of a URL in InternetOpenUrl.
by Ron Yorston March 7, 2007

March 7, 2007

A fix to winnet/internet.c turned up in 0.9.32 that's supposed to ensure that the "extra info" part of a URL is correctly included. I think there's a problem with the patch. On line 2865 space is allocated for the extra info: if (!(path_extra = HeapAlloc(GetProcessHeap(), 0, size))) Shouldn't this be in units of sizeof(WCHAR)? That is, if (!(path_extra = HeapAlloc(GetProcessHeap(), 0, size*sizeof(WCHAR)))) Ron

2 1