Page MenuHomePhabricator

Evas: add jxl loader and saver
AbandonedPublic

Authored by raster on Dec 21 2021, 12:22 PM.

Details

Reviewers
vtorri
Summary

add jxl loader and saver to Evas.

Test Plan

Entice for loading, animated jxl files or not
some conformances files : https://github.com/libjxl/conformance/tree/master/testcases

Diff Detail

Repository
rEFL core/efl
Branch
vtorri_jxl
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 17590
Build 11852: arc lint + arc unit
vtorri created this revision.Dec 21 2021, 12:22 PM
vtorri requested review of this revision.Dec 21 2021, 12:22 PM

i need to change a bit the data() cb

so wait a bit before applying

vtorri updated this revision to Diff 31537.Apr 26 2022, 12:43 AM

Formatting

vtorri updated this revision to Diff 31538.Apr 26 2022, 5:49 AM

formatting

vtorri updated this revision to Diff 31539.Apr 26 2022, 6:46 AM

formatting

vtorri updated this revision to Diff 31540.Apr 28 2022, 6:09 AM

factorize data callback

raster requested changes to this revision.Apr 28 2022, 8:12 AM

a few minor changes (inline comments :)

src/modules/evas/image_loaders/jxl/evas_image_load_jxl.c
268

there is some junk character between JXL_BIG_ENDIAN and ; (i see a box here - some whitespace problem?)

398

eek bad indenting here. :)

This revision now requires changes to proceed.Apr 28 2022, 8:12 AM
vtorri updated this revision to Diff 31541.Apr 28 2022, 8:44 AM

formatting and remove junk character

vtorri edited the test plan for this revision. (Show Details)Apr 28 2022, 8:46 AM
vtorri marked 2 inline comments as done.

@raster done

raster commandeered this revision.Apr 28 2022, 9:07 AM
raster edited reviewers, added: vtorri; removed: raster.

accepted

raster abandoned this revision.Apr 28 2022, 9:07 AM