'FirstTimeWizard::initializePage' hides overloaded virtual function #117

Closed
opened 1 year ago by duke · 3 comments
duke commented 1 year ago
Owner

It would be nice to deal with this compile warning, which maybe is related to some buggy behavior :

src/firsttimewizard.h:33:18: warning: 'FirstTimeWizard::initializePage' hides overloaded virtual function [-Woverloaded-virtual]
    virtual void initializePage();
It would be nice to deal with this compile warning, which maybe is related to some buggy behavior : ``` src/firsttimewizard.h:33:18: warning: 'FirstTimeWizard::initializePage' hides overloaded virtual function [-Woverloaded-virtual] virtual void initializePage(); ```
duke commented 1 year ago
Poster
Owner

I only see this with clang LLVM version 9.1.0 (clang-902.0.39.1)

It doesn't happen with gcc 8.3.0

I only see this with clang LLVM version 9.1.0 (clang-902.0.39.1) It doesn't happen with gcc 8.3.0
onryo added the
build
label 1 year ago
fekt commented 1 year ago
Collaborator

I recall seeing this and looking into it awhile back. It seemed like it was a warning that could be ignored and gets thrown by clang for any virtual functions as a precaution. QT defines it that way: https://doc.qt.io/qt-5/qwizardpage.html#initializePage

I recall seeing this and looking into it awhile back. It seemed like it was a warning that could be ignored and gets thrown by clang for any virtual functions as a precaution. QT defines it that way: https://doc.qt.io/qt-5/qwizardpage.html#initializePage
duke commented 1 year ago
Poster
Owner

@fekt ok, I don't think this is worth worrying about. Closing

@fekt ok, I don't think this is worth worrying about. Closing
duke closed this issue 1 year ago
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.