June 2013 - wine-devel

Re: [PATCH 3/4] ddraw: Device2 and Device3 do not have a lighting render state
by Henri Verbeet 20 Jun '13

20 Jun '13

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 20 Jun '13

20 Jun '13

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 20 Jun '13

20 Jun '13

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 20 Jun '13

20 Jun '13

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 20 Jun '13

20 Jun '13

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 19 Jun '13

19 Jun '13

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 19 Jun '13

19 Jun '13

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 19 Jun '13

19 Jun '13

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 19 Jun '13

19 Jun '13

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 19 Jun '13

19 Jun '13

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 19 Jun '13

19 Jun '13

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 18 Jun '13

18 Jun '13

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 18 Jun '13

18 Jun '13

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 18 Jun '13

18 Jun '13

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 18 Jun '13

18 Jun '13

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 18 Jun '13

18 Jun '13

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 18 Jun '13

18 Jun '13

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 18 Jun '13

18 Jun '13

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 18 Jun '13

18 Jun '13

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 18 Jun '13

18 Jun '13

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 18 Jun '13

18 Jun '13

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ř 18 Jun '13

18 Jun '13

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 17 Jun '13

17 Jun '13

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 17 Jun '13

17 Jun '13

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 17 Jun '13

17 Jun '13

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

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 June 2013