| Summary: | Compilation of preprocesstask.cpp fails if QT_NO_CAST_FROM_ASCII is defined | ||
|---|---|---|---|
| Product: | [Applications] digikam | Reporter: | thisco |
| Component: | Portability-Compilation | Assignee: | Digikam Developers <digikam-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | ||
| Priority: | NOR | ||
| Version First Reported In: | 5.0.0 | ||
| Target Milestone: | --- | ||
| Platform: | Gentoo Packages | ||
| OS: | Linux | ||
| Latest Commit: | http://commits.kde.org/digikam/b0f728ed31de2ebebe68394f856df0be388d3d04 | Version Fixed/Implemented In: | 5.0.0 |
| Sentry Crash Report: | |||
| Attachments: | qt-no-cast-from-ascii.patch | ||
Created attachment 97347 [details]
qt-no-cast-from-ascii.patch
Added the patch as an attachment as well now. Sorry for the inconvenience, but I did not find an attachment field in the first form, hence I added it as "Additional Information" ... Git commit b0f728ed31de2ebebe68394f856df0be388d3d04 by Maik Qualmann. Committed on 22/02/2016 at 07:07. Pushed by mqualmann into branch 'master'. apply patch #97347 to correct DImg save format option FIXED-IN: 5.0.0 M +2 -2 NEWS M +1 -1 utilities/assistants/panorama/tasks/preprocesstask.cpp http://commits.kde.org/digikam/b0f728ed31de2ebebe68394f856df0be388d3d04 |
Found this problem today while trying to compile the digikam-9999 ebuild (git master) on my Gentoo system. Apparently, QT_NO_CAST_FROM_ASCII is defined on my side, causing the QString(const * char) c'tor to be private. As a consequence I was not able to compile preprocesstask.cpp without modifying it, such that QImg::save(const QString&, FORMAT, DImgLoaderObserver* const) gets called in PreProcessTask::computePreview(const QUrl& inUrl) instead. The patch is included in the "Additional Information" section. Reproducible: Always diff --git a/utilities/assistants/panorama/tasks/preprocesstask.cpp b/utilities/assistants/panorama/tasks/preprocesstask.cpp index 8efc607..49f9972 100644 --- a/utilities/assistants/panorama/tasks/preprocesstask.cpp +++ b/utilities/assistants/panorama/tasks/preprocesstask.cpp @@ -133,7 +133,7 @@ bool PreProcessTask::computePreview(const QUrl& inUrl) if (img.load(inUrl.toLocalFile())) { DImg preview = img.smoothScale(1280, 1024, Qt::KeepAspectRatio); - bool saved = preview.save(outUrl.toLocalFile(), "JPG"); + bool saved = preview.save(outUrl.toLocalFile(), Digikam::DImg::FORMAT::JPEG); // save exif information also to preview image for auto rotation if (saved)