wine-devel November 2008

110 participants
239 discussions

Re: [2/5] WineD3D: Use a separate enum for pixel format fixups

by Henri Verbeet

2008/11/27 Stefan Dösinger <stefan(a)codeweavers.com>: > +enum shader_color_conv > +{ > + fmtconv_none = 1, > + set_blue_1 = 2, > + signed_2_no_ba = 3, > + signed_2 = 4, > + signed_4 = 5, > + signed_l6v5u5 = 6, > + swap_rg_blue_1 = 7, > + a_to_g_ba_1 = 8, > + yuv_yuy2 = 9, > + yuv_uyvy = 10, > + yuv_yv12 = 11, > + force_dword = 0x7ffffff > +}; > Could you please use uppercase enum elements like the rest of the world? :-/

15 years, 5 months

Re: ntdll: DPRINTF -> TRACE. (heap.c)

by Dmitry Timoshkov

"Andrew Riedi" <andrewriedi(a)gmail.com> wrote: > Changelog: > ntdll: DPRINTF -> TRACE. (heap.c) DPRINTF is used for a reason here. -- Dmitry.

15 years, 5 months

Re: imm32: Add some really basic tests for ImmSetCompositionString().

by Dmitry Timoshkov

"Andrew Riedi" <andrewriedi(a)gmail.com> wrote: > +static int test_ImmSetCompositionString(void) It's a usual practice to have the test cases return 'void', especially if you don't use the return value. > +{ > + HIMC imc; > + BOOL ret; > + DWORD error; > + > + imc = ImmGetContext(hwnd); First thing to test is the return value of ImmGetContext() and report last error in the case of a failure. -- Dmitry.

15 years, 5 months

Re: [1/5] D3D9: Document a Microsoft header bug

by Adam Petaccia

On Thu, 2008-11-27 at 01:30 +0100, Stefan Dösinger wrote: > > + > + /* Note: Microsoft's d3d9.h does not declare > IDirect3D9Ex::RegisterSoftwareDevice . This would mean that > + * the offsets of the other methods in the Vtable change too. > This, however, seems wrong. In Microsoft's > + * d3d9.dll, the offsets for the other functions are still > compatible with IDirect3D9. > + * (This is probably because even in MS's header IDirect3D9Ex > inherits from IDirect3D9, which makes the > + * C++ inferface compatible, and nobody uses the C interface in > Windows world > + */ Maybe this should be documented via Wine's API doc "BUGS: " declaration as well: http://www.winehq.org/site/docs/winedev-guide/api-docs

15 years, 5 months

Re: mshtml: Do a case-insensitive compare of type (resend 1)

by Alistair Leslie-Hughes

Any thing wrong with this patch? Best Regards Alistair Leslie-Hughes "Alistair Leslie-Hughes" <leslie_alistair(a)hotmail.com> wrote in message news:[email protected]... > Hi, > > Changelog: > mshtml: Do a case-insensitive compare of type > -------------------------------------------------------------------------------- > >From ab3346563682349645f07147c5adbcfc3ce90ebf Mon Sep 17 00:00:00 2001 > From: Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> > Date: Thu, 13 Nov 2008 21:38:13 +1100 > Subject: [PATCH] Do a case-insensitive compare of type > To: wine-patches <wine-patches(a)winehq.org> > > --- > dlls/mshtml/script.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/dlls/mshtml/script.c b/dlls/mshtml/script.c > index 4d10853..90bfef9 100644 > --- a/dlls/mshtml/script.c > +++ b/dlls/mshtml/script.c > @@ -640,7 +640,7 @@ static BOOL get_guid_from_type(LPCWSTR type, GUID > *guid) > {'t','e','x','t','/','j','a','v','a','s','c','r','i','p','t',0}; > > /* FIXME: Handle more types */ > - if(!strcmpW(type, text_javascriptW)) { > + if(!strcmpiW(type, text_javascriptW)) { > *guid = CLSID_JScript; > }else { > FIXME("Unknown type %s\n", debugstr_w(type)); > -- > 1.5.4.3 > > -------------------------------------------------------------------------------- > >

15 years, 5 months

re: RFC: Proposed new web site design

by Dan Kegel

Looks great so far. Comments: 1) nested scrollbars are evil. Better to put the news on its own tab, lose the little sidebar the news was in, and have a very small area in the main box for just the one most important headline. 2) The main choice text needs to be closer to user tasks. So instead of Information Overview of Wine features. we should have About What is Wine, and why should I use it? and we're missing the user task Will my app work with Wine? which should link to the appdb.

15 years, 5 months

Re: atl: Remove WINAPI on static functions where not needed.

by Ken Thomases

Thanks for taking care of this, Francois. On Nov 23, 2008, at 5:15 PM, Francois Gouget wrote: > Astute readers will notice that this leaves one loophole: nasty macro > use like in dlls/winmm/message16.c: > > #define A(_x,_y) MMDRV_InstallMap(_x, \ > MMDRV_##_y##_Map16To32W, MMDRV_##_y##_UnMap16To32W, \ > MMDRV_##_y##_Map32WTo16, MMDRV_##_y##_UnMap32WTo16, \ > MMDRV_##_y##_Callback) > A(MMDRV_AUX, Aux); > > A grep would not find that MMDRV_Aux_Callback is referenced by the > above > code. Could you perhaps grep the output from the preprocessor, instead? -Ken

15 years, 5 months

Re: mshtml: suppress module unloading while gecko is loaded (take 2)

by Paul Vriens

Dan Kegel wrote: > Thanks to Mike Hearn for pointing out that mshtml reference > counting was wrong, and Jacek for pointing out how to fix > it for now. > > Once gecko is loaded, don't let mshtml unload. > This fixes a crash in starting Sketchup (bug 16164), > and probably fixes a number of other apps that > went pear-shaped after unloading mshtml... > > > ------------------------------------------------------------------------ > > C++ style comment? Was this a Patchwatcher test ;) -- Cheers, Paul.

15 years, 5 months

Icons as cursors patch

by Krzysztof Drewniak

This is my first patch. It allows icons to be loaded as cursors because it defaults hotspots to x = 0 and y = 0 if the hotspot isn't found. Pasting patch into body of message. This is against the latest git . -------------------------------------------------------------------------------------------------------------------------- --- cursoricon.c.orig 2008-11-23 15:07:32.000000000 -0600 +++ cursoricon.c 2008-11-23 15:13:22.000000000 -0600 @@ -932,7 +932,10 @@ static HICON CURSORICON_LoadFromFile( LP entry = CURSORICON_FindBestIconFile( dir, width, height, colors ); if ( !entry ) - goto end; + /* this is my change. If there is no hotspot, default to 0, 0. */ + hotspot.x = 0; + hotspot.y = 0; + goto iconskip; /* check that we don't run off the end of the file */ if ( entry->dwDIBOffset > filesize ) @@ -942,6 +945,7 @@ static HICON CURSORICON_LoadFromFile( LP hotspot.x = entry->xHotspot; hotspot.y = entry->yHotspot; +iconskip: hIcon = CURSORICON_CreateIconFromBMI( (BITMAPINFO *)&bits[entry->dwDIBOffset], hotspot, !fCursor, 0x00030000, width, height, loadflags ); -------------------------------------------------------------------------------------------------------------------------- Please look over the code. Krzys D.

15 years, 5 months

Re: mshtml: suppress module unloading until we figure out why some apps crash with it enabled

by Jacek Caban

Dan Kegel wrote: > This prevents http://bugs.winehq.org/show_bug.cgi?id=16164 > > ( http://www.microsoft.com/msj/0197/activex/activex0197.aspx looks > like a nice, scary article on the subject. ) > - return module_ref ? S_FALSE : S_OK; + /* Some apps crash if we let mshtml be unloaded. + * Until we figure out why, don't let mshtml be unloaded. + */ + return S_FALSE; /* module_ref ? S_FALSE : S_OK; */ I think we should prevent unloading mshtml.dll if Gecko was loaded. To do it I'd suggest to increase module count in load_gecko function (it will never be decreased so DLL won't be unloaded). Jacek

15 years, 5 months

← Newer
1
2
3
4
5
6
7
8
...
24
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-devel November 2008