[digikam] [Bug 327124] New: Add loadHalfPreview reading from QBuffer

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

[digikam] [Bug 327124] New: Add loadHalfPreview reading from QBuffer

Bugzilla from sine.nomine@centrum.cz
https://bugs.kde.org/show_bug.cgi?id=327124

            Bug ID: 327124
           Summary: Add loadHalfPreview reading from QBuffer
    Classification: Unclassified
           Product: digikam
           Version: unspecified
          Platform: unspecified
                OS: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: NOR
         Component: libkdcraw
          Assignee: [hidden email]
          Reporter: [hidden email]
                CC: [hidden email]

Created attachment 83331
  --> https://bugs.kde.org/attachment.cgi?id=83331&action=edit
proposed patch

During implementation of the RAW preview support in gwenview, a need for
loadHalfPreview able of loading the RAW data from QBuffer and returning JPEG
data in QByteArray arose.
I decided to implement variants of loadHalfPreview and loadRawPreview returning
data in a QByteArray (JPEG format, but it would be easily implementable to
allow passing the desired format as optional parameter for loadHalfPreview. In
loadRawPreview that would be senseless as loadEmbeddedPreview is always JPEG).

If this API change is not desired, I'll implement loadHalfPreview(QImage,
QBuffer) as a minimal variant, leaving the conversion to JPEG in QByteArray on
gwenview side.

Proposed patch is attached (fixing some typos in the documentation comments,
too).

+++ This bug was initially created as a clone of Bug #326953 +++

During implementation of the RAW preview support in gwenview, a need for
loadEmbeddedPreview able of loading the RAW data from QBuffer (not file) arose.
Temporarily, I've implemented the function within gwenview. For the sake of
better maintainability of the code it would be nice to have this feature
implemented directly in libkdcraw. Proposed patch is attached.

Reproducible: Always

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[digikam] [Bug 327124] Add loadHalfPreview reading from QBuffer

Gilles Caulier-4
https://bugs.kde.org/show_bug.cgi?id=327124

Gilles Caulier <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
   Version Fixed In|                            |4.0.0
         Resolution|---                         |FIXED
      Latest Commit|                            |http://commits.kde.org/libk
                   |                            |dcraw/af8aef2c6acc93ea53c1e
                   |                            |520fe125656902d6d9f

--- Comment #1 from Gilles Caulier <[hidden email]> ---
Git commit af8aef2c6acc93ea53c1e520fe125656902d6d9f by Gilles Caulier.
Committed on 04/11/2013 at 12:55.
Pushed by cgilles into branch 'master'.

apply patch #83331 to add new method to get load RAW preview from QBuffer
FIXED-IN: 4.0.0

M  +2    -2    CMakeLists.txt
M  +50   -17   libkdcraw/kdcraw.cpp
M  +32   -12   libkdcraw/kdcraw.h
M  +58   -4    libkdcraw/kdcraw_p.cpp
M  +2    -0    libkdcraw/kdcraw_p.h

http://commits.kde.org/libkdcraw/af8aef2c6acc93ea53c1e520fe125656902d6d9f

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel