On Fri, Jan 27, 2012 at 8:15 AM, Anselmo L. S. Melo
<[email protected]> wrote:
> On Fri, Jan 27, 2012 at 4:51 AM, Samuel Rødal <[email protected]> wrote:
>> On 01/26/2012 08:21 PM, ext Anselmo L. S. Melo wrote:
>>> Hi,
>>>
>>> A window is a top level window if it has no parent, correct? Code and 
>>> comments in qwindow.cpp also confirm that.
>>>
>>> With that in mind, it seems reasonable that 
>>> QGuiApplication::topLevelWindows() should return a list of QWindows with 
>>> parent()==0.
>>> However, the returned list contains all QWindows created, including the 
>>> ones that don't respect this condition.
>>>
>>> So, it seems we need to either adjust QGuiApplication::topLevelWindows() or 
>>> control which windows are added to
>>> QGuiApplicationPrivate::window_list - currently, all windows are prepended 
>>> to this list in QWindow's ctors.
>>
>> Agreed, maybe just make QGuiApplication::topLevelWindows() filter the
>> window_list, since it would take more work to keep track of a
>> top_level_window_list in qwindow.cpp.
>>
>>> Also, would it be useful to have QGuiApplication::allWindows()?
>>
>> Yep, it might be.
>
> Ok, I'll work on that when I arrive at the office.

This patch was submitted here: http://codereview.qt-project.org/#change,14492

>
> Regards,
> Anselmo
>
>
> --
> Anselmo L. S. Melo
> openBossa / INdT
> http://www.indt.org



-- 
--
--
Anselmo L. S. Melo
openBossa / INdT
http://www.indt.org
http://www.anselmolsm.org
_______________________________________________
Development mailing list
[email protected]
http://lists.qt-project.org/mailman/listinfo/development

Reply via email to