Add support for images embedded in the .acbf file
ClosedPublic

Authored by jessep on Jan 26 2017, 12:47 PM.

Details

Summary
  • Add support for data and binary elements.
  • Support loading images from embedded data.
Test Plan

Can read comics where the images are embedded in the .acbf file.

Diff Detail

Repository
R157 Peruse
Branch
acbf-embedded-data
Lint
No Linters Available
Unit
No Unit Test Coverage
jessep updated this revision to Diff 10586.Jan 26 2017, 12:47 PM
jessep retitled this revision from to Add support for images embedded in the .acbf file.
jessep updated this object.
jessep edited the test plan for this revision. (Show Details)
jessep added a reviewer: Peruse.
jessep set the repository for this revision to R157 Peruse.
jessep added a project: Peruse.
leinir accepted this revision.Jan 27 2017, 10:43 AM
leinir added a reviewer: leinir.
leinir added a subscriber: leinir.

With that minor caveat in the archive image provider, this looks good to me! :)

src/qtquick/ArchiveImageProvider.cpp
63

A comment to describe the logic here would be good :) (i know it's just because that's how ACBF does references internally, but it'd be good to make that explicit, perhaps with a link to the standard?)

This revision is now accepted and ready to land.Jan 27 2017, 10:43 AM
jessep updated this revision to Diff 10623.Jan 27 2017, 2:40 PM
jessep updated this object.
jessep edited edge metadata.

Added comment to ArchiveImageProvider.cpp

This revision was automatically updated to reflect the committed changes.
jessep marked an inline comment as done.