Re: user32/tests: SetWindowPos() propagates update region from WS_CLIPCHILDREN child to its children (try 3).

2013-03-06 Thread Sergey Guralnik

On 2013-03-04 10:39, Sergey Guralnik wrote:
This patch demonstrates the most interest case from previous 
versions.

When SetWindowPos() moves child window, that has some invalid
area, it also invalidates children of this window according to its
update region, even if moved window has WS_CLIPCHILDREN style.

Wine doesn't perform this invalidation.
One pretty large application depends on this feature, and now,
runned under Wine, it has broken output.

Hope this variant is more clear than all previous.


It stays queued since Monday without any reply.
Is something wrong with it?

--
Sergey




Re: d3dx9 [patch 1/2, try 3]: Implement D3DXSHEvalConeLight

2013-03-06 Thread Nozomi Kodama
hello

Still problems with this patch?

Nozomi.



Re: [PATCH 1/7] winemac: Implement OpenGL support.

2013-03-06 Thread Ken Thomases
On Mar 6, 2013, at 1:58 PM, C.W. Betts wrote:

> It seems like there's conflicting types for GLhandleARB, one defined by Wine, 
> the other by OS X, on Mountain Lion. I assume that the #define __gl_h_ is a 
> try to work around the issue, but OS X's gltypes.h still gets included.

Ugh.  Thanks for letting me know.  I'll have to investigate.  I'm not well set 
up to build against later SDKs at the moment, but I'll see what I can do.  Of 
course, if you can figure out a solution, that would be great.

-Ken





Re: [PATCH 1/7] winemac: Implement OpenGL support.

2013-03-06 Thread C.W. Betts
It seems like there's conflicting types for GLhandleARB, one defined by Wine, 
the other by OS X, on Mountain Lion. I assume that the #define __gl_h_ is a try 
to work around the issue, but OS X's gltypes.h still gets included.
On Mar 6, 2013, at 3:59 AM, Ken Thomases  wrote:

> ---
> dlls/winemac.drv/Makefile.in|4 +-
> dlls/winemac.drv/cocoa_opengl.h |   32 +
> dlls/winemac.drv/cocoa_opengl.m |  154 
> dlls/winemac.drv/cocoa_window.m |  187 
> dlls/winemac.drv/gdi.c  |2 +-
> dlls/winemac.drv/macdrv.h   |   10 +
> dlls/winemac.drv/macdrv_cocoa.h |   15 +
> dlls/winemac.drv/opengl.c   | 1912 +++
> dlls/winemac.drv/window.c   |   11 +-
> 9 files changed, 2322 insertions(+), 5 deletions(-)
> create mode 100644 dlls/winemac.drv/cocoa_opengl.h
> create mode 100644 dlls/winemac.drv/cocoa_opengl.m
> create mode 100644 dlls/winemac.drv/opengl.c
> 
> <0001-winemac-Implement-OpenGL-support.patch>





Re: ntdll: make NtDelayExecution a bit more efficient

2013-03-06 Thread Michael Stefaniuc
On 03/06/2013 07:10 PM, Graham wrote:
> Alexandre wrote:
>> That's what the existing code already does.
> 
> Indeed. I don't know what I was thinking...
> 
>>> 2. If you're about to block on select(), then I don't see any point in
>>> preceding that with a call to sched_yield().
>>
>> This was added for a reason; most likely you'll have to write tests.
> 
> i.e. commit 8099c2b9. JW says "... to more closely resemble Windows
> behavior.  The key is to yield in a Sleep..."
JW is Jeremy White so us old timers chuckle now ;)

> I haven't yet figured out how he came up with this analysis, but I
> think it's safe to assume that he is correct, and my patch is garbage.
> Lesson learned: consult history.
"Date:   Tue Nov 2 19:32:03 2004 +" and the comment mentions Win2K.
If the ancient wisdom isn't backed by tests there's a fair chance that
it might not be applicable today. Or that it was a wrong theory back
then too.

bye
michael




Re: ntdll: make NtDelayExecution a bit more efficient

2013-03-06 Thread Graham
Alexandre wrote:
> That's what the existing code already does.

Indeed. I don't know what I was thinking...

>> 2. If you're about to block on select(), then I don't see any point in
>> preceding that with a call to sched_yield().
>
> This was added for a reason; most likely you'll have to write tests.

i.e. commit 8099c2b9. JW says "... to more closely resemble Windows
behavior.  The key is to yield in a Sleep..."

I haven't yet figured out how he came up with this analysis, but I
think it's safe to assume that he is correct, and my patch is garbage.
Lesson learned: consult history.

-- graham




Re: Use accented letters in their names.

2013-03-06 Thread Frédéric Delanoy
> On 3/4/13, Frédéric Delanoy  wrote:
>> On Fri, Mar 1, 2013 at 5:01 PM, Tae Wong  wrote:
>>> The authors file is outdated and you want this updated with the
>>> missing authors. Andrej Znidarsic should have Slovenian accented
>>> letters for his last name.
>>>
>>> On 2/4/13, Tae Wong  wrote:
 You will want to use accented letters in their authors names.

 Caolan McNamara
 Ferenc Wagner
 Marko Nikolic

 Every once and a while the GNOME Live website is down with an error on
 Netscape called “The operation timed out when attempting to contact .”
 - This paragraph should have a website name. You want to be unbanned
 in Amaterasu Translations and the DOS ain’t dead forums.
>>
>> Feel free to send a patch to wine-patches fixing those issues.
>>
On Wed, Mar 6, 2013 at 3:02 PM, Tae Wong  wrote:
> You’ve sent a patch to wine-patches but never got them fixed!

I didn't send a patch, but you did (well, sort of, this wasn't really
a patch, but a mere email)

See http://wiki.winehq.org/SubmittingPatches for instructions
(http://source.winehq.org/patches/ helps you see what is the status of
your patch, in this case 94687)

Note however that this authors list is likely autogenerated from
git.("John Doo ") email addresses, so it's up to the
patch submitter to correctly spell his/her own name to begin with.
Rewriting the history to fix this seems overkill to me.
Hence, if you fix the AUTHORS file, your changes might be overwritten
the next time the file is generated.

Frédéric




Re: [1/3] Make mac driver the default on OS X

2013-03-06 Thread Ken Thomases
On Mar 6, 2013, at 8:24 AM,  
 wrote:

> Bug report:
> Toying around with built-in notepad, clock, winhlp32, I noticed that
> notepad and winhlp32 are not resizable, whereas the system
> preferences "control" is.  By comparison, with the x11 driver,
> notepad's window is resizable even though it has no triangle
> widget at the bottom left.

Works here™.  I can resize notepad and winhlp32 just fine.


> Also, the virtual desktop seems gone or not yet implemented.  This is a key
> requirement for enough apps that do not handle today's huge resolutions.

There is no mechanism for one Mac process to draw into the windows of another.  
That's key to how the virtual desktop is implemented.  So, it's not really 
possible as things stand.  Alexandre and I have considered some sort of 
shared-memory mechanism, but it's barely into the conceptualization stage, let 
alone design or implementation.

Alternatively, we'd have to recreate X11 – a protocol for shuttling drawing 
operation requests from one process to another – to fix this.

I'm not sure it's worth it.  In my experience, the virtual desktop has more 
often been used to work around X window manager limitations.  The hope is that 
we have greater control with the Mac driver.  If all else fails, the X11 driver 
is still available.


> Another annoyance was that there's no HIDE function or short cut and some
> dialogues do not provide the orange button (middle one) to iconify the window.
> As a result, those pesky windows clutter your screen.
> That alone is enough reason to me a reason not to use that driver. It is not
> acceptable that one cannot get rid of a set of windows!
> Example: wine control, then navigate to the root CA certificates.

More items can be added to the menus easily enough.  However, I'm loathe to 
assign the usual keyboard shortcuts.  The problem is that the Command key is 
used to generate Alt keystrokes and it's relatively likely that a Windows 
program will want to receive Alt-H for its menus.  Or, perhaps, a game uses Alt 
for a player action (e.g. crouch) and H for another action and the user would 
end up pressing them simultaneously without even considering them a key 
combination.

My principle has been that only keyboard shortcuts that combine Command and 
Option should be used, since that's unlikely to be pressed by accident.  In the 
case of the Hide menu item, though, Command-Option-H would ordinarily mean Hide 
Others, just the opposite of Hide.

In any case, for the meantime, you can hide a Wine app from its Dock menu.


> The menu bar sometimes was refreshed badly, causing some texts therein
> to be displayed twice, shifted by a few pixels, or with varying fonts.

I'm not familiar with that, but it's not likely to be the fault of the Mac 
driver.  The Mac driver is not really a "graphics" driver.  It doesn't draw 
anything.  It uses Wine's relatively new client-side graphics implementation.  
User32 arranges for the DIB engine to do all of the drawing and then just 
deliver bitmaps to the Mac driver for it to blit to the window.

Are you sure you aren't see the same thing with the X11 driver?

-Ken





ntdll: make NtDelayExecution a bit more efficient

2013-03-06 Thread Joerg-Cyril.Hoehle
You may try and find that MSDN or blog page where people explain the
subtle differences of Sleep(0) and Sleep(1) on multi-core machines.

Regards,
 Jörg Höhle



[1/3] Make mac driver the default on OS X

2013-03-06 Thread Joerg-Cyril.Hoehle
Hi,

Ken Thomases wrote:
>I think that the Mac driver should not be made the default
>until OpenGL and clipboard support are in.
Exactly :-(

Following Josh DuBois explanation about
HKCU/Software/Wine/Drivers "Graphics"="mac,x11"
I decided to give it a try over the week-end and
was very disappointed to see that seemingly *not any* app would run.
It took me some time to understand that they all want graphics.
Please don't waste the user's time when such "basics" don't work.

What you can do is post to wine-devel and the user forum,
asking for people to test your work. That would be polite.

Josh DuBois wrote:
>I propose an environment variable, WINEDRIVERLOADORDER
Too generic. Include "DISPLAY" into the name, if at all,
e.g. WINEDISPLAYDRIVER, or "GRAPHICS" as in the registry.
The winepulse driver might face similar issues. I have not looked
how Maarten thinks that it may coexist nicely with winealsa.


Bug report:
Toying around with built-in notepad, clock, winhlp32, I noticed that
notepad and winhlp32 are not resizable, whereas the system
preferences "control" is.  By comparison, with the x11 driver,
notepad's window is resizable even though it has no triangle
widget at the bottom left.

Also, the virtual desktop seems gone or not yet implemented.  This is a key
requirement for enough apps that do not handle today's huge resolutions.

Another annoyance was that there's no HIDE function or short cut and some
dialogues do not provide the orange button (middle one) to iconify the window.
As a result, those pesky windows clutter your screen.
That alone is enough reason to me a reason not to use that driver. It is not
acceptable that one cannot get rid of a set of windows!
Example: wine control, then navigate to the root CA certificates.

The menu bar sometimes was refreshed badly, causing some texts therein
to be displayed twice, shifted by a few pixels, or with varying fonts.

Regards,
 Jörg Höhle



Re: [1/3] Make mac driver the default on OS X

2013-03-06 Thread Josh DuBois

On 3/6/13 12:24 AM, Ken Thomases wrote:
As I believe you're aware, I think that the Mac driver should not be 
made the default until OpenGL and clipboard support are in. So, maybe 
this was just submitted in preparation for that time, but there should 
probably have been a note to that effect. 

Sure.

Also, this patch seems exceedingly complex for a relatively simple problem.  
Just because it has historically been user32 that reported failure to load the 
driver doesn't mean it must remain so, at least to my way of thinking.  Why not 
just print out the diagnostics directly in gdi32 as each attempt to load a 
driver fails?
I figured it was that way for a reason, so I was trying to reduce noise 
in the case that fall-through drivers fail.  Even with making mac the 
first thing we try to load, it seemed that a valid use was setting by 
hand to, e.g., x11,mac, with working X11 libraries and DISPLAY unset.  
But yes, it's more complicated: I don't care where they're printed.  
(The same is true for the ERR->TRACE bits in the third chunk - I 
certainly don't mind them both staying ERRs, or WARNs.  I had 
interpreted the failure of those things differently until the point at 
which a window creation was attempted, but maybe that's wrong.)

+case ERROR_DLL_INIT_FAILED:
+load_err->err_msg = "Make sure you have permission to create \na window 
or check for errors with Console.app. \n(The Mac driver cannot do remote display: try X11 if 
you need that.)";
+break;

Probably it's just best to say that the Mac driver is running in a non-GUI 
session such as a remote login.
Well, if something in Apple's code fails, it's still likely they'll 
print to the console, yes?  At any rate, I'm not attached to the error 
message.  As for whether to print errors in gdi32 or user32, I don't 
feel strongly about that personally, either, but I'm curious what others 
think.





Re: winegcc: Add support for -Ttext-segment linker flag to set image base of a dll. (try 2)

2013-03-06 Thread Alexandre Julliard
Alexandre Julliard  writes:

> Tijl Coosemans  writes:
>
>> On 03-03-2013 17:44, André Hentschel wrote:
>>> You would rather call it PLATFORM_FREEBSD
>>
>> Well, the flag isn't FreeBSD specific. It works on all ELF targets
>> (including Linux which then wouldn't need prelink).
>
> Then it should be used by default, and you don't need to add a new
> platform for it.

Actually, I looked into this and it isn't widely supported at all, so it
can't be the default. Ideally it would need a run-time check to see if
the option works.

-- 
Alexandre Julliard
julli...@winehq.org




Re: winegcc: Add support for -Ttext-segment linker flag to set image base of a dll. (try 2)

2013-03-06 Thread Alexandre Julliard
Tijl Coosemans  writes:

> On 03-03-2013 17:44, André Hentschel wrote:
>> You would rather call it PLATFORM_FREEBSD
>
> Well, the flag isn't FreeBSD specific. It works on all ELF targets
> (including Linux which then wouldn't need prelink).

Then it should be used by default, and you don't need to add a new
platform for it.

-- 
Alexandre Julliard
julli...@winehq.org




Re: ntdll: make NtDelayExecution a bit more efficient

2013-03-06 Thread Alexandre Julliard
Graham  writes:

> This patch is vaguely related to bug 24558. It eliminates a few
> syscalls in NtDelayExecution:
>
> 1. If the caller requests a zero-wait yield, then do just that, and
> nothing more.

That's what the existing code already does.

> 2. If you're about to block on select(), then I don't see any point in
> preceding that with a call to sched_yield().

This was added for a reason; most likely you'll have to write tests.

-- 
Alexandre Julliard
julli...@winehq.org




Re: qcap: Fix compilation on systems that have v4l1 but not VIDIOCMCAPTURE & co.

2013-03-06 Thread Alexandre Julliard
Francois Gouget  writes:

> In particular this fixes compilation on FreeBSD 8.1.
>
>  configure   |   74 
> +++
>  configure.ac|   18 +
>  dlls/qcap/v4l.c |4 +--
>  include/config.h.in |3 +++
>  4 files changed, 97 insertions(+), 2 deletions(-)

It's not very useful to add a configure check for a #define, you can
check for it directly.

-- 
Alexandre Julliard
julli...@winehq.org