https://bugs.kde.org/show_bug.cgi?id=506778

Dmitry Kazakov <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]

--- Comment #2 from Dmitry Kazakov <[email protected]> ---
Hi, Brad!

It seems like heif people have changed their API without increasing major
version number. We haven't updated to this version yet, so please provide a
patch and we will merge it. The issue looks simple, int vs size_t
incompatibility. The patch should look like that (but in more places):

```
diff --git a/plugins/impex/heif/HeifExport.cpp
b/plugins/impex/heif/HeifExport.cpp
index bb83111a62..441d692218 100644
--- a/plugins/impex/heif/HeifExport.cpp
+++ b/plugins/impex/heif/HeifExport.cpp
@@ -329,8 +329,14 @@ KisImportExportErrorCode HeifExport::convert(KisDocument
*document, QIODevice *i

                 img.add_plane(heif_channel_Y, width, height, 8);

-                int strideG = 0;
-                int strideA = 0;
+#if LIBHEIF_HAVE_VERSION(1, XXX, YYY)
+                using HeifStrideType = size_t;
+#else
+                using HeifStrideType = int;
+#endif
+
+                HeifStrideType strideG = 0;
+                HeifStrideType strideA = 0;

                 uint8_t *ptrG = img.get_plane(heif_channel_Y, &strideG);
                 uint8_t *ptrA = [&]() -> uint8_t * {
```

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to