[PATCH] Add override specifier
José Abílio Matos
jamatos at lyx.org
Mon Oct 5 19:38:56 UTC 2020
On Monday, October 5, 2020 6:09:40 PM WEST Jean-Marc Lasgouttes wrote:
> Le 05/10/2020 à 18:43, José Abílio Matos a écrit :
> > On Monday, October 5, 2020 5:19:46 PM WEST Jean-Marc Lasgouttes wrote:
> >> Fixed
> >>
> >> JMarc
> >
> > Now, if I may ask what about the following kind of warning:
> >
> > In file included from ./ui_TabularUi.h:28:
> > /home/jamatos/lyx/lyx/src/frontends/qt/GuiSetBorder.h:30:59: warning:
> > 'QFlags' is deprecated: Use default constructor instead
> > [-Wdeprecated-declarations] GuiSetBorder(QWidget * parent = nullptr,
> > Qt::WindowFlags fl =
> > nullptr);
>
> Are you sure that your ui_TabularUi.h has been built by your current qt
> version? It is weird to see Qt complain about code that it generated.
Yes.
I created a new directory. And then ran configure on that directory using the
CC and CXX environment variables to force clang.
This is what I get. I am using qt-5.15.1 FWIW:
/home/jamatos/lyx/lyx/src/frontends/qt/GuiView.cpp:221:11: warning: 'width' is
deprecated: Use QFontMetrics::horizontalAdvance [-Wdeprecated-declarations]
if (fm.width(*sit) > wline)
^
/usr/include/qt5/QtGui/qfontmetrics.h:105:5: note: 'width' has been explicitly
marked deprecated here
QT_DEPRECATED_X("Use QFontMetrics::horizontalAdvance")
^
/usr/include/qt5/QtCore/qglobal.h:294:33: note: expanded from macro
'QT_DEPRECATED_X'
# define QT_DEPRECATED_X(text) Q_DECL_DEPRECATED_X(text)
^
/usr/include/qt5/QtCore/qcompilerdetection.h:675:55: note: expanded from macro
'Q_DECL_DEPRECATED_X'
# define Q_DECL_DEPRECATED_X(text) __attribute__ ((__deprecated__(text)))
^
Probably is more a question of the qt version.
> > There is another type but this seems to be the single instance:
> > In file included from /home/jamatos/lyx/lyx/src/EnchantChecker.cpp:14:
> > /usr/include/enchant/enchant++.h:55:25: warning:
> > 'enchant::Exception::what'
> > hides overloaded virtual function [-Woverloaded-virtual]
> > virtual const char * what () throw() {
>
> Do you have enchant2 installed? If you do and LyX does not use it, then
> it is a bug. I see that we check for enchant-2 with pkg-config, but it
> might be that enchant2 is better for you.
For some reason I did not had enchant2-devel installed. After the installation
it seems to be used and the warning goes away.
> JMarc
Best regards,
--
José Abílio
More information about the lyx-devel
mailing list