<div>For qrand() and qsrand() I had already proposed them to replace with QRandomGenerator in the proposal and I plan to do it next week alongwith few other fixes. So it should be no problem. The other issues also seem straight forward as I have looked. So it shouldn't be a problem.</div><br><div>Also How can I run these checks locally? </div><br><div>Thanks</div><div>Anjani</div><br><div class="gmail_quote_attribution">On Jun 18 2021, at 4:57 pm, Gilles Caulier <caulier.gilles@gmail.com> wrote:</div><blockquote><div><div>Hi Anjani,</div><div><br></div><div>Please review the new cppcheck warnings here :</div><div><br></div><div><a href="https://www.digikam.org/reports/cppcheck/master/" title="https://www.digikam.org/reports/cppcheck/master/">https://www.digikam.org/reports/cppcheck/master/</a></div><div><br></div><div>cppcheck has now a specific Qt config file which reports more warnings about deprecated API.</div><div><br></div><div>Typically, i expect that qrandCalled, useStlAlgorithm, qsrandCalled, and qSortCalled entries will be fixed in your qt6 port project</div><div><br></div><div>Right ?</div><div><br></div><div>Best</div><div><br></div><div><div>Gilles Caulier</div></div></div><br><div class="gmail_quote"><div class="gmail_attr"><div>Le ven. 18 juin 2021 à 11:51, Anjani Kumar <<a href="mailto:anjanik012@gmail.com" title="mailto:anjanik012@gmail.com">anjanik012@gmail.com</a>> a écrit :</div></div><blockquote><div>I cleared all the issues above Qt 5.12 including macros for Qt::endl and Qt::SplitBehavior. Compilations is working fine on Qt 5.15.2 but I have no way to compile it on Qt 5.12.0 as of now. </div><div>Also, I have one MR <a href="https://invent.kde.org/graphics/digikam/-/merge_requests/95" title="https://invent.kde.org/graphics/digikam/-/merge_requests/95">https://invent.kde.org/graphics/digikam/-/merge_requests/95</a>. Could you review it? I have fixed some of the concerns.</div><br><div>Thanks,</div><div>Anjani</div><br><div>On Jun 17 2021, at 9:41 pm, Anjani Kumar <<a href="mailto:anjanik012@gmail.com" title="mailto:anjanik012@gmail.com">anjanik012@gmail.com</a>> wrote:</div><blockquote><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> :(</div><br><div>Should I correct it?</div><br><div>Thanks</div><div>Anjani</div><br><div>On Jun 17 2021, at 9:39 pm, Maik Qualmann <<a href="mailto:metzpinguin@gmail.com" title="mailto:metzpinguin@gmail.com">metzpinguin@gmail.com</a>> 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 <<a href="mailto:anjanik012@gmail.com" title="mailto:anjanik012@gmail.com">anjanik012@gmail.com</a>> wrote:</div><div>> > Hello,</div><div>> ></div><div>> > This should be fine I guess,</div><div>> > <a href="https://invent.kde.org/graphics/digikam/-/commit/0a1fd2c60ec38707f0e2dfea" title="https://invent.kde.org/graphics/digikam/-/commit/0a1fd2c60ec38707f0e2dfea">https://invent.kde.org/graphics/digikam/-/commit/0a1fd2c60ec38707f0e2dfea</a></div><div>> > bf6ebb45a01f9382. Let me know. Thanks,</div><div>> > Anjani</div><div>> ></div><div>> > On Jun 16 2021, at 11:23 am, Gilles Caulier <<a href="mailto:caulier.gilles@gmail.com" title="mailto:caulier.gilles@gmail.com">caulier.gilles@gmail.com</a>></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 <<a href="mailto:metzpinguin@gmail.com" title="mailto:metzpinguin@gmail.com">metzpinguin@gmail.com</a></div><div>(mailto:<a href="mailto:metzpinguin@gmail.com" title="mailto:metzpinguin@gmail.com">metzpinguin@gmail.com</a>)> 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 <<a href="mailto:anjanik012@gmail.com" title="mailto:anjanik012@gmail.com">anjanik012@gmail.com</a></div><div>(mailto:<a href="mailto:anjanik012@gmail.com" title="mailto:anjanik012@gmail.com">anjanik012@gmail.com</a>)> 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 <<a href="mailto:anjanik012@gmail.com" title="mailto:anjanik012@gmail.com">anjanik012@gmail.com</a></div><div>(mailto:<a href="mailto:anjanik012@gmail.com" title="mailto:anjanik012@gmail.com">anjanik012@gmail.com</a>)> 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 <<a href="mailto:metzpinguin@gmail.com" title="mailto:metzpinguin@gmail.com">metzpinguin@gmail.com</a></div><div>(mailto:<a href="mailto:metzpinguin@gmail.com" title="mailto:metzpinguin@gmail.com">metzpinguin@gmail.com</a>)> 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>> > > > > > > > > <a href="https://invent.kde.org/graphics/digikam/-/blob/master/core/a" title="https://invent.kde.org/graphics/digikam/-/blob/master/core/a">https://invent.kde.org/graphics/digikam/-/blob/master/core/a</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>> > > > > > > > > <<a href="mailto:anjanik012@gmail.com" title="mailto:anjanik012@gmail.com">anjanik012@gmail.com</a> (mailto:<a href="mailto:anjanik012@gmail.com" title="mailto:anjanik012@gmail.com">anjanik012@gmail.com</a>)> a> > ></div><div>> > > > écrit :</div><div>> > > > > > > > > > The recent changes to</div><div>> > > > > > > > > > <a href="https://invent.kde.org/graphics/digikam/-/tree/gsoc21-qt6-" title="https://invent.kde.org/graphics/digikam/-/tree/gsoc21-qt6-">https://invent.kde.org/graphics/digikam/-/tree/gsoc21-qt6-</a>> > > > > > > > > > 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></blockquote></blockquote></div></blockquote>