diff options
-rw-r--r-- | libs/jpegrecoverymap/Android.bp | 13 | ||||
-rw-r--r-- | libs/jpegrecoverymap/include/jpegrecoverymap/jpegrerrorcode.h | 1 | ||||
-rw-r--r-- | libs/jpegrecoverymap/include/jpegrecoverymap/recoverymap.h | 1 | ||||
-rw-r--r-- | libs/jpegrecoverymap/jpegdecoder.cpp | 2 | ||||
-rw-r--r-- | libs/jpegrecoverymap/jpegencoder.cpp | 2 | ||||
-rw-r--r-- | libs/jpegrecoverymap/recoverymap.cpp | 6 | ||||
-rw-r--r-- | libs/jpegrecoverymap/tests/Android.bp | 2 |
7 files changed, 20 insertions, 7 deletions
diff --git a/libs/jpegrecoverymap/Android.bp b/libs/jpegrecoverymap/Android.bp index 54af7c912d..c9bed70714 100644 --- a/libs/jpegrecoverymap/Android.bp +++ b/libs/jpegrecoverymap/Android.bp @@ -21,7 +21,7 @@ package { default_applicable_licenses: ["frameworks_native_license"], } -cc_library_static { +cc_library { name: "libjpegrecoverymap", host_supported: true, @@ -37,15 +37,18 @@ cc_library_static { shared_libs: [ "libimage_io", "libjpeg", - "libutils", + "libjpegencoder", + "libjpegdecoder", ], } -cc_library_static { +cc_library { name: "libjpegencoder", + host_supported: true, shared_libs: [ "libjpeg", + "liblog", ], export_include_dirs: ["include"], @@ -55,11 +58,13 @@ cc_library_static { ], } -cc_library_static { +cc_library { name: "libjpegdecoder", + host_supported: true, shared_libs: [ "libjpeg", + "liblog", ], export_include_dirs: ["include"], diff --git a/libs/jpegrecoverymap/include/jpegrecoverymap/jpegrerrorcode.h b/libs/jpegrecoverymap/include/jpegrecoverymap/jpegrerrorcode.h index 6995762ea2..699c0d3ca1 100644 --- a/libs/jpegrecoverymap/include/jpegrecoverymap/jpegrerrorcode.h +++ b/libs/jpegrecoverymap/include/jpegrecoverymap/jpegrerrorcode.h @@ -38,6 +38,7 @@ enum { ERROR_JPEGR_RESOLUTION_MISMATCH = JPEGR_IO_ERROR_BASE - 3, ERROR_JPEGR_BUFFER_TOO_SMALL = JPEGR_IO_ERROR_BASE - 4, ERROR_JPEGR_INVALID_COLORGAMUT = JPEGR_IO_ERROR_BASE - 5, + ERROR_JPEGR_INVALID_TRANS_FUNC = JPEGR_IO_ERROR_BASE - 6, JPEGR_RUNTIME_ERROR_BASE = -20000, ERROR_JPEGR_ENCODE_ERROR = JPEGR_RUNTIME_ERROR_BASE - 1, diff --git a/libs/jpegrecoverymap/include/jpegrecoverymap/recoverymap.h b/libs/jpegrecoverymap/include/jpegrecoverymap/recoverymap.h index 35979032be..ae15d247bd 100644 --- a/libs/jpegrecoverymap/include/jpegrecoverymap/recoverymap.h +++ b/libs/jpegrecoverymap/include/jpegrecoverymap/recoverymap.h @@ -30,6 +30,7 @@ typedef enum { // Transfer functions as defined for XMP metadata typedef enum { + JPEGR_TF_UNSPECIFIED = -1, JPEGR_TF_LINEAR = 0, JPEGR_TF_HLG = 1, JPEGR_TF_PQ = 2, diff --git a/libs/jpegrecoverymap/jpegdecoder.cpp b/libs/jpegrecoverymap/jpegdecoder.cpp index 7dc3df9d71..6fbc6b0118 100644 --- a/libs/jpegrecoverymap/jpegdecoder.cpp +++ b/libs/jpegrecoverymap/jpegdecoder.cpp @@ -16,7 +16,7 @@ #include <jpegrecoverymap/jpegdecoder.h> -#include <cutils/log.h> +#include <utils/Log.h> #include <errno.h> #include <setjmp.h> diff --git a/libs/jpegrecoverymap/jpegencoder.cpp b/libs/jpegrecoverymap/jpegencoder.cpp index 1997bf9ea3..627dcdf6ee 100644 --- a/libs/jpegrecoverymap/jpegencoder.cpp +++ b/libs/jpegrecoverymap/jpegencoder.cpp @@ -16,7 +16,7 @@ #include <jpegrecoverymap/jpegencoder.h> -#include <cutils/log.h> +#include <utils/Log.h> #include <errno.h> diff --git a/libs/jpegrecoverymap/recoverymap.cpp b/libs/jpegrecoverymap/recoverymap.cpp index ee68043c79..6b46d4001c 100644 --- a/libs/jpegrecoverymap/recoverymap.cpp +++ b/libs/jpegrecoverymap/recoverymap.cpp @@ -673,6 +673,9 @@ status_t RecoveryMap::generateRecoveryMap(jr_uncompressed_ptr uncompressed_yuv_4 hdrInvOetf = pqInvOetf; hdr_white_nits = kPqMaxNits; break; + case JPEGR_TF_UNSPECIFIED: + // Should be impossible to hit after input validation. + return ERROR_JPEGR_INVALID_TRANS_FUNC; } ColorTransformFn hdrGamutConversionFn = getHdrConversionFn( @@ -771,6 +774,9 @@ status_t RecoveryMap::applyRecoveryMap(jr_uncompressed_ptr uncompressed_yuv_420_ case JPEGR_TF_PQ: hdrOetf = pqOetf; break; + case JPEGR_TF_UNSPECIFIED: + // Should be impossible to hit after input validation. + return ERROR_JPEGR_INVALID_TRANS_FUNC; } for (size_t y = 0; y < height; ++y) { diff --git a/libs/jpegrecoverymap/tests/Android.bp b/libs/jpegrecoverymap/tests/Android.bp index b509478e73..39445f81ab 100644 --- a/libs/jpegrecoverymap/tests/Android.bp +++ b/libs/jpegrecoverymap/tests/Android.bp @@ -30,10 +30,10 @@ cc_test { ], shared_libs: [ "libjpeg", + "libimage_io", "liblog", ], static_libs: [ - "libimage_io", "libgmock", "libgtest", "libjpegdecoder", |