Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix QT compile-time warnings #990

Merged
merged 9 commits into from
Dec 4, 2024
Merged

Conversation

flipmcf
Copy link
Contributor

@flipmcf flipmcf commented Nov 18, 2024

Fixes this compile-time warning:

./rddialog.h:33:65: warning: ‘constexpr QFlags<T>::QFlags(QFlags<T>::Zero) [with Enum = Qt::WindowType; QFlags<T>::Zero = int QFlags<Qt::WindowType>::Private::*]’ is deprecated: Use default constructor instead [-Wdeprecated-declarations]
   33 |   RDDialog(RDConfig *config,QWidget *parent=0,Qt::WindowFlags f=0);
      |                                                                 ^
In file included from /usr/include/x86_64-linux-gnu/qt5/QtCore/qglobal.h:1299,
                 from /usr/include/x86_64-linux-gnu/qt5/QtGui/qtguiglobal.h:43,
                 from /usr/include/x86_64-linux-gnu/qt5/QtWidgets/qtwidgetsglobal.h:43,
                 from /usr/include/x86_64-linux-gnu/qt5/QtWidgets/qapplication.h:43,
                 from /usr/include/x86_64-linux-gnu/qt5/QtWidgets/QApplication:1,
                 from rdcart_dialog.cpp:24:
/usr/include/x86_64-linux-gnu/qt5/QtCore/qflags.h:123:80: note: declared here
  123 |     QT_DEPRECATED_X("Use default constructor instead") Q_DECL_CONSTEXPR inline QFlags(Zero) noexcept : i(0) {}

@flipmcf
Copy link
Contributor Author

flipmcf commented Nov 21, 2024

Fixes #992

@flipmcf flipmcf closed this Nov 21, 2024
@flipmcf
Copy link
Contributor Author

flipmcf commented Nov 21, 2024

There's more I want to add... standby

@flipmcf flipmcf reopened this Nov 22, 2024
@flipmcf flipmcf changed the title Fix QFlags initialization warning - use constructor Fix QT compile-time warnings Nov 25, 2024
@ElvishArtisan ElvishArtisan merged commit 81eeccc into ElvishArtisan:v4 Dec 4, 2024
@ElvishArtisan
Copy link
Owner

Merged to 'v4'. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants