Warning when closing an undocked qwidget

Mwoua_Leddar

Just run basic tests with version 5.11.3 and 5.12.0.
Looks like I dont have the issue with any of them

Christian Ehrlicher

Can you check latest 5.12.x?

Mwoua_Leddar

I have managed to reproduce it with a really basic example:

#include <QApplication>
#include <QMainWindow>
#include <QLabel>
#include <QDockWidget>


int main(int argc, char* argv[])
{
    QApplication lApp(argc, argv);

    QMainWindow a;
    a.setCentralWidget(new QLabel("Main"));
    a.show();

    auto lWindow = new QDockWidget(&a);
    lWindow->setFloating(true);
    lWindow->show();

    return lApp.exec();
}

and the cmake file im using:

cmake_minimum_required(VERSION 3.16.0)
set(CMAKE_VERBOSE_MAKEFILE ON CACHE BOOL "ON" FORCE)

project(test)

#Qt libs and macro
SET(CMAKE_PREFIX_PATH "C:/Qt/5.15.1/msvc2019_64")

set(CMAKE_AUTOMOC ON)
set(CMAKE_AUTORCC ON)
set(CMAKE_AUTOUIC ON)

find_package(Qt5 COMPONENTS Widgets Gui REQUIRED)

set(Src
    ${CMAKE_CURRENT_LIST_DIR}/main.cpp
)

add_executable(Test ${Src})
target_link_libraries(Test PRIVATE Qt5::Widgets Qt5::Gui)

Mwoua_Leddar

Ran more test with this simple example, the issue is actually there with qt 5.11.3 ,5.12.0 and 5.12.9.

Edit:
I confirm the issue is not here with 5.7.1

Should I test with more versions?
Raise a bug?

Do you know if there is a workaround?

Christian Ehrlicher

So you can also reproduce it with 5.11? Then it's not the change I thought.
It would be nice when you find some time to investigate with which version the issue came in.

jsulm

This post is deleted!

Mwoua_Leddar

The issue is there in 5.9.

Cant test between 5.7.1 and 5.9, versions are not available for download

Christian Ehrlicher

Thx for the information. I would suggest you to create a bug reports with all your findings. Sadly I can't compile such an old version on my system anymore so I can't hep figuring out which exact commit creates this issue.

Mwoua_Leddar

Bug report created: https://bugreports.qt.io/browse/QTBUG-88157
(Cant figure out how to properly format the code in the report)

Christian Ehrlicher

Thx, I reformatted the code, the CMakeLists.txt is not needed.