<div dir="ltr"><div>Hi,</div><div><br></div><div>Using digikam_globasl.h with a preprocessor definition as QT_ENDL depending of Qt version can be a solution.</div><div><br></div><div>Gilles<br></div><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Le mer. 16 juin 2021 à 07:41, Maik Qualmann <<a href="mailto:metzpinguin@gmail.com">metzpinguin@gmail.com</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">It's all correct with the Qt version check. My only thought was whether we <br>
could find another option for Qt::endl, as there would be a lot of Qt version <br>
checks. <br>
<br>
Maik<br>
<br>
Am Dienstag, 15. Juni 2021, 23:09:28 CEST schrieb Anjani Kumar:<br>
> What should I do? Keep doing this or not?<br>
> <br>
> Thanks<br>
> Anjani<br>
> <br>
> On Jun 16 2021, at 2:23 am, Anjani Kumar <<a href="mailto:anjanik012@gmail.com" target="_blank">anjanik012@gmail.com</a>> wrote:<br>
> > Something like this<br>
> > <br>
> > <br>
> > #if (QT_VERSION >= QT_VERSION_CHECK(5, 14, 0))<br>
> > <br>
> > QStringList tagsList = hierarchy.split(QLatin1Char('/'),<br>
> > Qt::SkipEmptyParts);<br>
> > <br>
> > #else<br>
> > <br>
> > QStringList tagsList = hierarchy.split(QLatin1Char('/'),<br>
> > QString::SkipEmptyParts);<br>
> > <br>
> > #endif<br>
> > I haven't pushed it yet though.<br>
> > <br>
> > On Jun 16 2021, at 2:19 am, Anjani Kumar <<a href="mailto:anjanik012@gmail.com" target="_blank">anjanik012@gmail.com</a>> wrote:<br>
> > > I already started this pre-processor branching. Not for Qt::endl but I<br>
> > > have done it for QString::split() which is also present from Qt 5.14.> > <br>
> > > On Jun 16 2021, at 2:16 am, Maik Qualmann <<a href="mailto:metzpinguin@gmail.com" target="_blank">metzpinguin@gmail.com</a>> wrote:<br>
> > > > But we already have a problem with Qt::endl, this only exists with<br>
> > > > Qt-5.14. We should find another way of doing this than with<br>
> > > > pre-processeur branching. Maybe define a macro.<br>
> > > > <br>
> > > > Maik<br>
> > > > <br>
> > > > Am Dienstag, 15. Juni 2021, 13:55:53 CEST schrieb Gilles Caulier:<br>
> > > > > Hi,<br>
> > > > > <br>
> > > > > No. We need to be compatible at least with Qt 5.12 LTS.<br>
> > > > > Please make pre-processeur branching in source code by checking Qt<br>
> > > > > version,<br>
> > > > > as it already do in other place in digiKam.<br>
> > > > > <br>
> > > > > exemple :<br>
> > > > > <a href="https://invent.kde.org/graphics/digikam/-/blob/master/core/app/date/" rel="noreferrer" target="_blank">https://invent.kde.org/graphics/digikam/-/blob/master/core/app/date/</a><br>
> > > > > ddatepic ker.cpp#L547<br>
> > > > > <br>
> > > > > Best<br>
> > > > > <br>
> > > > > Gilles Caulier<br>
> > > > > <br>
> > > > > Le mar. 15 juin 2021 à 13:52, Anjani Kumar <<a href="mailto:anjanik012@gmail.com" target="_blank">anjanik012@gmail.com</a>> a <br>
écrit :<br>
> > > > > > The recent changes to<br>
> > > > > > <a href="https://invent.kde.org/graphics/digikam/-/tree/gsoc21-qt6-port" rel="noreferrer" target="_blank">https://invent.kde.org/graphics/digikam/-/tree/gsoc21-qt6-port</a><br>
> > > > > > makes it<br>
> > > > > > compile only on newer Qt versions. So I it should be best Bump Qt<br>
> > > > > > version<br>
> > > > > > to 5.15.2. Should I make the change?<br>
> > > > > > <br>
> > > > > > Thanks,<br>
> > > > > > Anjani<br>
<br>
<br>
<br>
<br>
</blockquote></div>