June 2013 - Wine-Devel - list.winehq.org

Re: [PATCH 3/4] ddraw: Device2 and Device3 do not have a lighting render state
by Henri Verbeet June 20, 2013

June 20, 2013

On 20 June 2013 12:17, Stefan Dösinger <stefan(a)codeweavers.com> wrote: > /* Light state */ > DWORD material; > + DWORD dummy_rs_lighting; > Considering the render state doesn't exist in early ddraw versions, do we really care that its value is preserved? It seems unlikely that anything except our tests would depend on it.

2 3

Re: [website] Added Portuguese translation for call for translators
by Frédéric Delanoy June 20, 2013

June 20, 2013

2013/6/19 Lucas Zawacki <lfzawacki(a)gmail.com>: > From: Lucas Zawacki <lfzawacki(a)gmail.com> > > --- > news/pt/2013061601.xml | 9 +++++++++ > 1 file changed, 9 insertions(+) > create mode 100644 news/pt/2013061601.xml > > diff --git a/news/pt/2013061601.xml b/news/pt/2013061601.xml > new file mode 100644 > index 0000000..86bf5a8 > --- /dev/null > +++ b/news/pt/2013061601.xml > @@ -0,0 +1,9 @@ > +<news> > +<date>16 de Junho de 2013</date> > +<title>Procura-se tradutores!</title> > +<body> Just wondering: does it make sense to translate a call for translators? I mean, if a volunteer can't understand the English from the call, (s)he wouldn't be able to translate correctly anyway. Frédéric

3 2

Re: [PATCH 2/4] ddraw: Send device2_Draw* through device3_Draw*
by Henri Verbeet June 20, 2013

June 20, 2013

On 20 June 2013 12:17, Stefan Dösinger <stefan(a)codeweavers.com> wrote: > - return IDirect3DDevice7_DrawPrimitive(&device->IDirect3DDevice7_iface, device->primitive_type, > + return IDirect3DDevice3_DrawPrimitive(&device->IDirect3DDevice3_iface, device->primitive_type, > device->vertex_type, device->sysmem_vertex_buffer, device->nb_vertices, device->render_flags); This is not introduced by your patch of course, but I don't think there's much of a point in going through the vtbl here.

2 1

Re: [PATCH 1/4] ddraw: SPECULARENABLE defaults to true in IDirect3DDevice2
by Henri Verbeet June 20, 2013

June 20, 2013

On 20 June 2013 12:17, Stefan Dösinger <stefan(a)codeweavers.com> wrote: > @@ -4250,7 +4250,12 @@ static HRESULT WINAPI d3d2_CreateDevice(IDirect3D2 *iface, REFCLSID riid, > wined3d_mutex_lock(); > hr = d3d_device_create(ddraw, surface_impl, 2, &device_impl, NULL); > if (SUCCEEDED(hr)) > + { > *device = &device_impl->IDirect3DDevice2_iface; > + hr = IDirect3DDevice2_SetRenderState(*device, D3DRENDERSTATE_SPECULARENABLE, TRUE); > + if (FAILED(hr)) > + ERR("SetRenderState failed\n"); > + } Why not do this together with the other render state setup in d3d_device_init()?

1 0

Re: [PATCH 2/3] d3d9/tests: Reset texture 0 to NULL at the end of texop_test().
by Henri Verbeet June 20, 2013

June 20, 2013

On 19 June 2013 21:59, Matteo Bruni <mbruni(a)codeweavers.com> wrote: > While it doesn't really matter for the test results (it only affects the > alpha component of the following test but that isn't checked anyway) I feel > it's a good idea to properly cleanup and avoid dependencies between > tests. Yeah, though ideally we'd do that by using create_device() from individual tests instead of reusing a single device.

1 0

Re: [Wine] German translation - netstat.exe
by Joerg Schiermeier June 19, 2013

June 19, 2013

On Tuesday, 18 Jun 2013 at 19:35:02 André Hentschel wrote: > when you scroll down a bit on that page, you see everything needed to get rid of false positives. Ah, I didn't scrolled down sooo deep. :( > Oh and fgouget is Fran?ois Gouget Thanks for this hint. I will deal with it in the right manner. -- Kindly regards Joerg Schiermeier Bielefeld/Germany

1 0

Re: gdi32: Get the default character index, not its value.
by Alexandre Julliard June 19, 2013

June 19, 2013

Akihiro Sagawa <sagawa.aki(a)gmail.com> writes: > --- > dlls/gdi32/freetype.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) It doesn't work here: ../../../tools/runtest -q -P wine -M gdi32.dll -T ../../.. -p gdi32_test.exe.so font.c && touch font.ok font.c:1356: Test failed: GetGlyphIndicesW should have returned a 0080 not 0000 -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: po: Update Italian translation (try 3)
by Matteo Bruni June 19, 2013

June 19, 2013

2013/6/17 Fabian Ebner <f.ebner94(a)gmail.com>: > Typo fixed (finally) > --- > po/it.po | 40 ++++++++++++++-------------------------- > 1 file changed, 14 insertions(+), 26 deletions(-) > This one looks good to me, thanks :)

1 0

Re: mlang.dll implementation fix
by Christian Costa June 19, 2013

June 19, 2013

Hi and welcome, Patches should be sent included in the mail. Adding tests is recommended. You can follow status at http://source.winehq.org/patches/. But currently we're in code freeze so you may have to wait Wine 1.6. Christian 2013/6/19 Bartosz <gang65(a)poczta.onet.pl> > My name is Bartosz and I would like to start hacking wine. > At beginning I implemented BreakLine methods (available in mlang.dll): > http://msdn.microsoft.com/en-us/library/aa741037%28v=vs.85%29.aspx > > I prepared patch which is available at: > http://bugs.winehq.org/attachment.cgi?id=44854&action=diff > http://bugs.winehq.org/show_bug.cgi?id=32126 > > Could you please take a look at these methods and push it into git repo? > > Should I write tests for that? > > Thanks in advance > Bartosz > > >

2 1

Re: oledb32: Add more database tests
by Marvin June 19, 2013

June 19, 2013

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=26031 Your paranoid android. === WXPX64 (64 bit database) === database.c:166: Test failed: got L"Provider=SQLOLEDB.1;Data Source=dummy" === WVISTAX64 (64 bit database) === database.c:166: Test failed: got L"Provider=SQLOLEDB.1;Data Source=dummy" === W7PROX64 (64 bit database) === database.c:166: Test failed: got L"Provider=SQLOLEDB.1;Data Source=dummy" === TEST64_W7SP1 (64 bit database) === database.c:166: Test failed: got L"Provider=SQLOLEDB.1;Data Source=dummy"

1 0

Using of mac driver completely breaks tablets support?
by Andrey Upadyshev June 19, 2013

June 19, 2013

Hi All! I've installed a Wine 1.5.31 instead of 1.4.smthng and found that tablets support is broken. Some research gives me that wintab32 DLL fails to init because winemac driver doesn't implement tablet related functions (LoadTabletInfo etc) unlike x11 driver. I'm a Wine newcomer so probably I get something wrong about it... How can I switch back to x11 driver? BR, Andrey Upadyshev

4 4

Re: [bugzilla] Add descriptions for additional resolutions (try 2)
by Alexandre Julliard June 18, 2013

June 18, 2013

André Hentschel <nerv(a)dawncrow.de> writes: > + <dd> > + The problem is not a [% terms.bug %] in Wine, > + but in some upstream software Wine depends on or broken > + packages in distributions. > + Note: Most other bug trackers call this "NOTOURBUG". We should probably rename it. UPSTREAM is not a good name for the way we are using it. -- Alexandre Julliard julliard(a)winehq.org

3 4

Re: Alexandre Julliard : winefile: Avoid abbreviations in column names.
by Bruno Jesus June 18, 2013

June 18, 2013

On Tue, Jun 18, 2013 at 4:42 PM, Alexandre Julliard <julliard(a)winehq.org> wrote: > Module: wine > Branch: master > Commit: f07ff7f9819b32437624a320fe3029dd795b7a61 > URL: http://source.winehq.org/git/wine.git/?a=commit;h=f07ff7f9819b32437624a320f… > > Author: Alexandre Julliard <julliard(a)winehq.org> > Date: Tue Jun 18 11:32:45 2013 +0200 > > winefile: Avoid abbreviations in column names. Why not remove the "Date" word as they are always dates? Leave only "Creation", "Modification" and "Access". It will help to keep the column headers short. Are they used in other places? Best wishes, Bruno

2 1

Re: po: Update Spanish translation
by Bruno Jesus June 18, 2013

June 18, 2013

On Mon, Jun 17, 2013 at 6:05 PM, André Hentschel <nerv(a)dawncrow.de> wrote: > I recently started to learn spanish, so i did this easy fixes with best knowledge, though without guaranty > --- > po/es.po | 60 ++++++++++++++++++------------------------------------------ > 1 file changed, 18 insertions(+), 42 deletions(-) > #: joy.rc:64 > -#, fuzzy > -#| msgid "Available formats" > msgid "Available Effects" > -msgstr "Formatos disponibles" > +msgstr "Formatos efectos" "Efectos disponibles" makes more sense than "Formatos efectos", I guess this was a typo. #: netstat.rc:33 msgid "Sent" -msgstr "" +msgstr "Envió" "Envió" is present tense, the past tense would be "Enviado". Best wishes, Bruno

3 2

[Wine] German translation - netstat.exe
by Joerg Schiermeier June 18, 2013

June 18, 2013

Hi list, in wines call for translators is listed one 'spelling error' in the German translation which isn't one. <http://fgouget.free.fr/wine-po/> The translation is correct: <http://fgouget.free.fr/wine-po/de.spell.html> ---<cite>--- [Proto] (unchanged) 10453 -> Porto, Proton, Photo, Protest, Protzen ---</cite>--- is correct translated to 'Proto'. 'Proto' is used as an abbreviation for protocol. This appears in 'netstat.exe' (src/wine/programs/netstat). I checked this in an 'Windows 7' installation running the original command. Who is responsible for the list in <http://fgouget.free.fr/wine-po/>? You may please take this one faulty error away. -- Kindly regards Joerg Schiermeier Bielefeld/Germany

2 1

Re: [PATCH 1/2] wined3d: Pass fogstart==fogend to GL in fog table mode (try 3)
by Henri Verbeet June 18, 2013

June 18, 2013

On 18 June 2013 16:17, Stefan Dösinger <stefan(a)codeweavers.com> wrote: > diff --git a/dlls/wined3d/glsl_shader.c b/dlls/wined3d/glsl_shader.c > index e14d7ab..ad9be6a 100644 > --- a/dlls/wined3d/glsl_shader.c > +++ b/dlls/wined3d/glsl_shader.c > @@ -7107,7 +7109,7 @@ static void glsl_fragment_pipe_fog(struct wined3d_context *context, > new_source = FOGSOURCE_FFP; > } > > - if (new_source != context->fog_source) > + if (new_source != context->fog_source || fogstart == fogend) > { > context->fog_source = new_source; > state_fogstartend(context, state, STATE_RENDER(WINED3D_RS_FOGSTART)); You shouldn't need this.

2 3

Re: [PATCH 2/5] wined3d: Handle fog_start==fog_end with table fog(try 2)
by Henri Verbeet June 18, 2013

June 18, 2013

On 17 June 2013 22:53, Stefan Dösinger <stefan(a)codeweavers.com> wrote: > correctly with fixed function fog. Tested as usual on Geforce9 (OSX, > Linux), Geforce7 (Linux), r300g, r600g, i965 (OSX), r200. Which of those is affected by the quirk? I think all of those except r200 should be able to do GLSL, but you say r200 is the only one that doesn't pass the tests. You may be able to convince me that this is a good idea, but so far I don't think it's worth all the extra code. > + if (isnan(fogstart) || isinf(fogstart) || isinf(-fogstart)) You should only need one isinf() here, it's not supposed to care about the sign. > #define WINED3D_ADAPTER_CAP_XYZRHW 0x00000001 > #define WINED3D_ADAPTER_CAP_VS_CLIPPING 0x00000002 > +#define WINED3D_ADAPTER_CAP_BROKEN_FOG 0x00000008 Is this intentional?

2 4

Re: comctl32: Use default GUI font instead of System as a fallback in the tab control.
by Alexandre Julliard June 18, 2013

June 18, 2013

Dmitry Timoshkov <dmitry(a)baikal.ru> writes: > --- > dlls/comctl32/tab.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) It breaks the tests here: ../../../tools/runtest -q -P wine -M comctl32.dll -T ../../.. -p comctl32_test.exe.so tab.c && touch tab.ok tab.c:625: Test failed: with icon, set size > icon: Expected [57,30] got [54,30] tab.c:626: Test failed: with icon, set size < icon: Expected [57,20] got [54,20] tab.c:627: Test failed: with icon, min size: Expected [57,1] got [54,1] tab.c:625: Test failed: with icon, set size > icon: Expected [60,30] got [54,30] tab.c:626: Test failed: with icon, set size < icon: Expected [60,20] got [54,20] tab.c:627: Test failed: with icon, min size: Expected [60,1] got [54,1] tab.c:625: Test failed: with icon, set size > icon: Expected [63,30] got [56,30] tab.c:626: Test failed: with icon, set size < icon: Expected [63,20] got [56,20] tab.c:627: Test failed: with icon, min size: Expected [63,1] got [56,1] tab.c:625: Test failed: with icon, set size > icon: Expected [66,30] got [59,30] tab.c:626: Test failed: with icon, set size < icon: Expected [66,20] got [59,20] tab.c:627: Test failed: with icon, min size: Expected [66,1] got [59,1] etc. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: (retry) gdi32: load cached vertical fonts after non-vertical font
by Alexandre Julliard June 18, 2013

June 18, 2013

Aric Stewart <aric(a)codeweavers.com> writes: > { > - FontSubst *subst = HeapAlloc(GetProcessHeap(), 0, sizeof(*subst)); > - subst->from.name = strdupW(english_family); > - subst->from.charset = -1; > - subst->to.name = strdupW(family_name); > - subst->to.charset = -1; > - add_font_subst(&font_subst_list, subst, 0); > + if (!array_size) > + { > + array_size = 5; > + vertical_fonts = HeapAlloc(GetProcessHeap(), 0, array_size * sizeof(WCHAR*)); > + } > + if (vertical_count+1 >= array_size) > + { > + array_size *= 2; > + vertical_fonts = HeapReAlloc(GetProcessHeap(), 0, vertical_fonts, array_size * sizeof(WCHAR*)); > + } > + vertical_fonts[vertical_count] = strdupW(buffer); > + vertical_count++; That seems overly complicated. We already put the fonts in a list, it shouldn't be hard to reorder it, or use a temp list for vertical fonts, or something like that. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH 1/5] wined3d: Consolidate d3d_info BOOLs into a flags field
by Henri Verbeet June 18, 2013

June 18, 2013

We probably want this regardless of the next patch, but I'd prefer for this kind of thing to go in after the release.

1 0

Re: comctl32: Use default GUI font as a fallback instead of a (non-existent) Arial.
by Daniel Jeliński June 18, 2013

June 18, 2013

Hi Dmitry, After this patch got applied, the code looks like this: ( http://source.winehq.org/git/wine.git/blob/cc086f09ae46c77e6eb4a817ae567294… ) 1914 if (!GetObjectW((infoPtr->hFont) ? 1915 infoPtr->hFont : GetStockObject(DEFAULT_GUI_FONT), 1916 sizeof(LOGFONTW),&logfont)) 1917 GetStockObject(DEFAULT_GUI_FONT); I'm quite sure you didn't mean to ignore return value of GetStockObject and leave logfont uninitialized here. Regards, Daniel (sorry for being late to the party, I'm not subscribing wine-patches, and kind of assumed this patch wouldn't get past AJ)

3 2

Re: Web based translation tool
by Michal Čihař June 18, 2013

June 18, 2013

Hi Dne Mon, 12 Mar 2012 12:02:49 +0200 Yaron Shahrabani <sh.yaron(a)gmail.com> napsal(a): > The Wine project is thinking about using Weblate in order to translate > Wine, you can look at the following discussion and see for yourself. That sounds great! > Looking good although a litle immature, not sure about authentication as > wel... For authentication you can basically use anything what can be used in Django. Any specific concerns on this topic? > I'll talk to Michal Cihar and ask him for a roadmap for this project. Well I don't have any real roadmap right now. I urgently needed replacement for Pootle because we got hit by too many bugs there and did not manage to understand the code there to be able to fix them myself. So far I think it is in pretty good shape for day to day work (with today's 0.5 release fixing few annoying bugs). Of course there are some features missing, you can check what I plan for 1.0 release here: https://github.com/nijel/weblate/issues?state=open&milestone=1 I currently hope to have 1.0 in a month or so, but I really don't want to commit to any specific date. PS: I'm not subscribed to mailing list so please CC me on reply. -- Michal Čihař | http://cihar.com | http://blog.cihar.com

7 16

Re: po: Update Italian translation (try 2)
by Matteo Bruni June 17, 2013

June 17, 2013

2013/6/17 Fabian Ebner <f.ebner94(a)gmail.com>: > Fixed a typo and a sentence, thanks to Matteo > --- > po/it.po | 40 ++++++++++++++-------------------------- > 1 file changed, 14 insertions(+), 26 deletions(-) #: jscript.rc:39 msgid "'return' statement outside of function" -msgstr "" +msgstr "Istruzione 'return' fouri della funzione" Sorry, but this is worse...

2 1

Re: ole32: Do not fail if partial blocks are read
by Bruno Jesus June 17, 2013

June 17, 2013

On Mon, Jun 17, 2013 at 1:28 PM, Aric Stewart <aric(a)codeweavers.com> wrote: > > fixes bug 25346 Actually this patch fixes bug 33821. Bug 25346 is older than the regression and was only affected by it. > --- > dlls/ole32/storage32.c | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > Best wishes, Bruno

1 0

Re: Update Italian translation
by Matteo Bruni June 17, 2013

June 17, 2013

2013/6/16 Fabian Ebner <f.ebner94(a)gmail.com>: > --- > po/it.po | 40 ++++++++++++++-------------------------- > 1 file changed, 14 insertions(+), 26 deletions(-) #: joy.rc:42 msgid "" "After disabling or enabling a device, the connected joysticks won't be " "updated here until you restart this applet." msgstr "" +"Dopo disattivare o attivare un dispositivo, i joystick connessi non saranno " +"attualizzati qui, fino al riavvio dell'applet." That sounds like broken Italian (or a word-by-word translation), maybe something like "Dopo aver disattivato o attivato un dispositivo, i joystick connessi non saranno aggiornati fino al riavvio dell'applet."? The original English message has probably some room for improvement too. #: jscript.rc:39 msgid "'return' statement outside of function" -msgstr "" +msgstr "Istruzione 'return' fouri dalla funzione" Typo.

1 0