<div>Yes it would. I did realize it after making commits otherwise. <a href="https://invent.kde.org/graphics/digikam/-/commit/8b651e62b3133908aa9500e1f7a984ec0de13531" title="https://invent.kde.org/graphics/digikam/-/commit/8b651e62b3133908aa9500e1f7a984ec0de13531">https://invent.kde.org/graphics/digikam/-/commit/8b651e62b3133908aa9500e1f7a984ec0de13531</a> <span data-emoji-typing="true">:(</span></div><br><div>Should I correct it?</div><br><div>Thanks</div><div>Anjani</div><br><div class="gmail_quote_attribution">On Jun 17 2021, at 9:39 pm, Maik Qualmann <metzpinguin@gmail.com> wrote:</div><blockquote><div><div>If everything compiles fine, yes. I think it would be a good solution for</div><div>QString::SkipEmptyParts as well.</div><br><div>Maik</div><br><div>Am Donnerstag, 17. Juni 2021, 09:23:31 CEST schrieb Anjani Kumar:</div><div>> Hello,</div><div>> I defined the macro QT_ENDL appropriately, and made the changes locally.</div><div>> Should I push them?</div><div>></div><div>> Thanks,</div><div>> Anjani</div><div>></div><div>> On Jun 16 2021, at 12:35 pm, Anjani Kumar <anjanik012@gmail.com> wrote:</div><div>> > Hello,</div><div>> ></div><div>> > This should be fine I guess,</div><div>> > https://invent.kde.org/graphics/digikam/-/commit/0a1fd2c60ec38707f0e2dfea</div><div>> > bf6ebb45a01f9382. Let me know. Thanks,</div><div>> > Anjani</div><div>> ></div><div>> > On Jun 16 2021, at 11:23 am, Gilles Caulier <caulier.gilles@gmail.com></div><div>wrote:</div><div>> > > Hi,</div><div>> > ></div><div>> > > Using digikam_globasl.h with a preprocessor definition as QT_ENDL</div><div>> > > depending of Qt version can be a solution.</div><div>> > ></div><div>> > > Gilles</div><div>> > ></div><div>> > > Le mer. 16 juin 2021 à 07:41, Maik Qualmann <metzpinguin@gmail.com</div><div>(mailto:metzpinguin@gmail.com)> a écrit :</div><div>> > > > It's all correct with the Qt version check. My only thought was</div><div>> > > > whether we</div><div>> > > > could find another option for Qt::endl, as there would be a lot of Qt</div><div>> > > > version checks.</div><div>> > > ></div><div>> > > > Maik</div><div>> > > ></div><div>> > > > Am Dienstag, 15. Juni 2021, 23:09:28 CEST schrieb Anjani Kumar:</div><div>> > > > > What should I do? Keep doing this or not?</div><div>> > > > ></div><div>> > > > > Thanks</div><div>> > > > > Anjani</div><div>> > > > ></div><div>> > > > > On Jun 16 2021, at 2:23 am, Anjani Kumar <anjanik012@gmail.com</div><div>(mailto:anjanik012@gmail.com)> wrote:</div><div>> > > > > > Something like this</div><div>> > > > > ></div><div>> > > > > ></div><div>> > > > > > #if (QT_VERSION >= QT_VERSION_CHECK(5, 14, 0))</div><div>> > > > > ></div><div>> > > > > > QStringList tagsList = hierarchy.split(QLatin1Char('/'),</div><div>> > > > > > Qt::SkipEmptyParts);</div><div>> > > > > ></div><div>> > > > > > #else</div><div>> > > > > ></div><div>> > > > > > QStringList tagsList = hierarchy.split(QLatin1Char('/'),</div><div>> > > > > > QString::SkipEmptyParts);</div><div>> > > > > ></div><div>> > > > > > #endif</div><div>> > > > > > I haven't pushed it yet though.</div><div>> > > > > ></div><div>> > > > > > On Jun 16 2021, at 2:19 am, Anjani Kumar <anjanik012@gmail.com</div><div>(mailto:anjanik012@gmail.com)> wrote:</div><div>> > > > > > > I already started this pre-processor branching. Not for Qt::endl</div><div>> > > > > > > but I</div><div>> > > > > > > have done it for QString::split() which is also present from Qt</div><div>> > > > > > > 5.14.> ></div><div>> > > > > > ></div><div>> > > > > > > On Jun 16 2021, at 2:16 am, Maik Qualmann <metzpinguin@gmail.com</div><div>(mailto:metzpinguin@gmail.com)> wrote:</div><div>> > > > > > > > But we already have a problem with Qt::endl, this only exists</div><div>> > > > > > > > with</div><div>> > > > > > > > Qt-5.14. We should find another way of doing this than with</div><div>> > > > > > > > pre-processeur branching. Maybe define a macro.</div><div>> > > > > > > ></div><div>> > > > > > > > Maik</div><div>> > > > > > > ></div><div>> > > > > > > > Am Dienstag, 15. Juni 2021, 13:55:53 CEST schrieb Gilles</div><div>Caulier:</div><div>> > > > > > > > > Hi,</div><div>> > > > > > > > ></div><div>> > > > > > > > > No. We need to be compatible at least with Qt 5.12 LTS.</div><div>> > > > > > > > > Please make pre-processeur branching in source code by</div><div>> > > > > > > > > checking Qt</div><div>> > > > > > > > > version,</div><div>> > > > > > > > > as it already do in other place in digiKam.</div><div>> > > > > > > > ></div><div>> > > > > > > > > exemple :</div><div>> > > > > > > > > https://invent.kde.org/graphics/digikam/-/blob/master/core/a</div><div>> > > > > > > > > pp/date/</div><div>> > > > > > > > > ddatepic ker.cpp#L547</div><div>> > > > > > > > ></div><div>> > > > > > > > > Best</div><div>> > > > > > > > ></div><div>> > > > > > > > > Gilles Caulier</div><div>> > > > > > > > ></div><div>> > > > > > > > > Le mar. 15 juin 2021 à 13:52, Anjani Kumar</div><div>> > > > > > > > > <anjanik012@gmail.com (mailto:anjanik012@gmail.com)> a> > ></div><div>> > > > écrit :</div><div>> > > > > > > > > > The recent changes to</div><div>> > > > > > > > > > https://invent.kde.org/graphics/digikam/-/tree/gsoc21-qt6-> > > > > > > > > > port</div><div>> > > > > > > > > > makes it</div><div>> > > > > > > > > > compile only on newer Qt versions. So I it should be best</div><div>> > > > > > > > > > Bump Qt</div><div>> > > > > > > > > > version</div><div>> > > > > > > > > > to 5.15.2. Should I make the change?</div><div>> > > > > > > > > ></div><div>> > > > > > > > > > Thanks,</div><div>> > > > > > > > > > Anjani</div></div></blockquote>