[graphics/digikam] core/dplugins/generic/tools/printcreator/wizard: cppcheck++: constify

Gilles Caulier <[email protected]>
Newsgroups gmane.comp.kde.cvs
Message-ID <[email protected]>
Git commit 4a409f041ce33814e970bcb4e957f6e20fb766c6 by Gilles Caulier.
Committed on 09/08/2026 at 21:56.
Pushed by cgilles into branch 'master'.

cppcheck++: constify

M  +6    -6    core/dplugins/generic/tools/printcreator/wizard/advprintphotopage.cpp

https://invent.kde.org/graphics/digikam/-/commit/4a409f041ce33814e970bcb4e957f6e20fb766c6

diff --git a/core/dplugins/generic/tools/printcreator/wizard/advprintphotopage.cpp b/core/dplugins/generic/tools/printcreator/wizard/advprintphotopage.cpp
index 1bd2bc4f5c..c8ceda1fac 100644
--- a/core/dplugins/generic/tools/printcreator/wizard/advprintphotopage.cpp
+++ b/core/dplugins/generic/tools/printcreator/wizard/advprintphotopage.cpp
@@ -851,13 +851,13 @@ int AdvPrintPhotoPage::getPageCount() const
     {
         // get the selected layout
 
-        AdvPrintPhotoSize* const s = d->settings->photosizes.at(d->photoUi->ListPhotoSizes->currentRow());
+        const AdvPrintPhotoSize* const s = d->settings->photosizes.at(d->photoUi->ListPhotoSizes->currentRow());
 
         // how many pages?  Recall that the first layout item is the paper size
 
-        int photosPerPage   = s->m_layouts.count() - 1;
-        int remainder       = photoCount % photosPerPage;
-        int emptySlots      = 0;
+        int photosPerPage                = (s->m_layouts.count() - 1);
+        int remainder                    = (photoCount % photosPerPage);
+        int emptySlots                   = 0;
 
         if (remainder > 0)
         {
@@ -917,7 +917,7 @@ void AdvPrintPhotoPage::slotListPhotoSizesSelected()
     AdvPrintPhotoSize* s = nullptr;
     QSizeF size, sizeManaged;
 
-    // TODO FREE STYLE
+    // @todo add free style.
     // check if layout is managed by templates or free one
     // get the selected layout
 
@@ -1099,7 +1099,7 @@ void AdvPrintPhotoPage::slotListPhotoSizesSelected()
             // Atckin's layout
         }
 
-        // TODO not for Atckin's layout
+        // @todo not for Atckin's layout
 
         iconpreview.end();
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.