Re: winex11.drv: Don't add owned windows to the taskbar

2008-03-31 Thread Alex Villací­s Lasso
Dmitry Timoshkov escribió:
> "Alex Villací­s Lasso" <[EMAIL PROTECTED]> wrote:
>
>   
>>> The problem with IDA is that the button you see in the Windows task bar
>>> does not belong to the main IDA window, it belongs to a zero sized visible
>>> window with the same caption. Since it's zero sized Wine doesn't map it all.
>>> So, while formally it's a regression, technically it's not: main IDA window
>>> has never been added to the task bar before my patch. This all can be easily
>>> discovered with spy++ and a simple test app.
>>>
>>>
>>>   
>> This also affects many, if not all, VB6 applications (and possibly many
>> Delphi applications). Personally, I consider this a rather serious
>> regression and propose backing out of commit
>> cb3ff54789c652b02d852f40db71cafd9fd67ca9 as a fix.
>>
>> BTW, this reminds me of bug 1598. If the explanation of the bug for IDA
>> above is correct, then the "buggy" behavior of not assigning
>> minimize/maximize buttons to the window that appears in the panel is
>> sort of consistent with this regression. Maybe the behavior should be
>> that visible windows owned by an invisible window should get mapped in
>> the panel.
>> 
>
> 1598 is closed as fixed, and I have no idea how missing minimize/maximize
> buttons are related to owned windows, and why you think that it's somehow
> related to the visibility of an owner.
>
> What you need to be fixed is making a zero sized window with WS_VISIBLE
> style set mapped, but X11/WMs doesn't handle that gracefully.
>
>   
Bug 1598 (even if closed and fixed) is relevant because it was also 
triggered by having an invisible owner window, as my own analysis in 
comment #16 suggested. This bug (1598) also made the window button 
disappear from the taskbar, with the same application that now exhibited 
the regression.

I was just preparing a patch to fix this very issue over the weekend, 
but it seems Jacek Caban beat me to it. I am currently recompiling my 
tree to test the patch.

-- 
perl -e '$x=2.4;print sprintf("%.0f + %.0f = %.0f\n",$x,$x,$x+$x);'





Re: winex11.drv: Don't add owned windows to the taskbar

2008-03-29 Thread Dmitry Timoshkov
"Alex Villací­s Lasso" <[EMAIL PROTECTED]> wrote:

>> The problem with IDA is that the button you see in the Windows task bar
>> does not belong to the main IDA window, it belongs to a zero sized visible
>> window with the same caption. Since it's zero sized Wine doesn't map it all.
>> So, while formally it's a regression, technically it's not: main IDA window
>> has never been added to the task bar before my patch. This all can be easily
>> discovered with spy++ and a simple test app.
>>
>>
> This also affects many, if not all, VB6 applications (and possibly many
> Delphi applications). Personally, I consider this a rather serious
> regression and propose backing out of commit
> cb3ff54789c652b02d852f40db71cafd9fd67ca9 as a fix.
>
> BTW, this reminds me of bug 1598. If the explanation of the bug for IDA
> above is correct, then the "buggy" behavior of not assigning
> minimize/maximize buttons to the window that appears in the panel is
> sort of consistent with this regression. Maybe the behavior should be
> that visible windows owned by an invisible window should get mapped in
> the panel.

1598 is closed as fixed, and I have no idea how missing minimize/maximize
buttons are related to owned windows, and why you think that it's somehow
related to the visibility of an owner.

What you need to be fixed is making a zero sized window with WS_VISIBLE
style set mapped, but X11/WMs doesn't handle that gracefully.

-- 
Dmitry. 





Re: winex11.drv: Don't add owned windows to the taskbar

2008-03-28 Thread Alex Villací­s Lasso
Dmitry Timoshkov escribio':
> "Alexander Dorofeyev" <[EMAIL PROTECTED]> wrote:
>
>   
>> There's a regression in IDA Pro disassembler. Nothing is added to the 
>> taskbar at 
>> all, furthermore, when you are starting debugger, before there was a window 
>> appearing that asks pass control to the application or not. Now it's not 
>> appearing, and can't be found because there's nothing on the taskbar, which 
>> is 
>> pretty bad because the app doesn't respond to any other input at that 
>> moment. 
>> Minimizing  the main window makes the app disappear altogether, so it's 
>> pretty 
>> unusable.
>> 
>
> The problem with IDA is that the button you see in the Windows task bar
> does not belong to the main IDA window, it belongs to a zero sized visible
> window with the same caption. Since it's zero sized Wine doesn't map it all.
> So, while formally it's a regression, technically it's not: main IDA window
> has never been added to the task bar before my patch. This all can be easily
> discovered with spy++ and a simple test app.
>
>   
This also affects many, if not all, VB6 applications (and possibly many 
Delphi applications). Personally, I consider this a rather serious 
regression and propose backing out of commit 
cb3ff54789c652b02d852f40db71cafd9fd67ca9 as a fix.

BTW, this reminds me of bug 1598. If the explanation of the bug for IDA 
above is correct, then the "buggy" behavior of not assigning 
minimize/maximize buttons to the window that appears in the panel is 
sort of consistent with this regression. Maybe the behavior should be 
that visible windows owned by an invisible window should get mapped in 
the panel.

-- 
perl -e '$x=2.4;print sprintf("%.0f + %.0f = %.0f\n",$x,$x,$x+$x);'





Re: winex11.drv: Don't add owned windows to the taskbar

2008-03-25 Thread Dmitry Timoshkov
"Alexander Dorofeyev" <[EMAIL PROTECTED]> wrote:

> There's a regression in IDA Pro disassembler. Nothing is added to the taskbar 
> at 
> all, furthermore, when you are starting debugger, before there was a window 
> appearing that asks pass control to the application or not. Now it's not 
> appearing, and can't be found because there's nothing on the taskbar, which 
> is 
> pretty bad because the app doesn't respond to any other input at that moment. 
> Minimizing  the main window makes the app disappear altogether, so it's 
> pretty 
> unusable.

The problem with IDA is that the button you see in the Windows task bar
does not belong to the main IDA window, it belongs to a zero sized visible
window with the same caption. Since it's zero sized Wine doesn't map it all.
So, while formally it's a regression, technically it's not: main IDA window
has never been added to the task bar before my patch. This all can be easily
discovered with spy++ and a simple test app.

-- 
Dmitry.




Re: winex11.drv: Don't add owned windows to the taskbar

2008-03-25 Thread Alexander Dorofeyev
Hi.

There's a regression in IDA Pro disassembler. Nothing is added to the taskbar 
at 
all, furthermore, when you are starting debugger, before there was a window 
appearing that asks pass control to the application or not. Now it's not 
appearing, and can't be found because there's nothing on the taskbar, which is 
pretty bad because the app doesn't respond to any other input at that moment. 
Minimizing  the main window makes the app disappear altogether, so it's pretty 
unusable.

I've KDE 3.5.

You can get free version (that has same symptoms) from 
http://www.datarescue.com/ to help tweaking this.

Dmitry Timoshkov wrote:
> Hello,
> 
> this patch fixes the bug 12160. According to my tests explorer in XP
> doesn't add owned windows to the taskbar.
> 
> Changelog:
> winex11.drv: Don't add owned windows to the taskbar.
> ---





Re: winex11.drv: Don't add owned windows to the taskbar

2008-03-24 Thread Alexandre Julliard
Dmitry Timoshkov <[EMAIL PROTECTED]> writes:

> this patch fixes the bug 12160. According to my tests explorer in XP
> doesn't add owned windows to the taskbar.

Do we really want to remove them from the pager too?

-- 
Alexandre Julliard
[EMAIL PROTECTED]