From d6ad34fde5855dbf0121d5ff6158910cef176f50 Mon Sep 17 00:00:00 2001 From: Vignesh Date: Mon, 25 Mar 2024 15:35:13 +0530 Subject: [PATCH] Add clarifying comment on isAvifHeader --- .../com/facebook/imageformat/DefaultImageFormatChecker.kt | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/imagepipeline-base/src/main/java/com/facebook/imageformat/DefaultImageFormatChecker.kt b/imagepipeline-base/src/main/java/com/facebook/imageformat/DefaultImageFormatChecker.kt index d7fb569b29..f5338b423f 100644 --- a/imagepipeline-base/src/main/java/com/facebook/imageformat/DefaultImageFormatChecker.kt +++ b/imagepipeline-base/src/main/java/com/facebook/imageformat/DefaultImageFormatChecker.kt @@ -271,8 +271,13 @@ class DefaultImageFormatChecker : FormatChecker { (ImageFormatCheckerUtils.startsWithPattern(imageHeaderBytes, DNG_HEADER_II) || ImageFormatCheckerUtils.startsWithPattern(imageHeaderBytes, DNG_HEADER_MM)) + /** + * Checks if [imageHeaderBytes] contains 'avif'. + * + * This check may not be sufficient, though it works for most AVIF images. + * Details on AVIF can be found [here](https://aomediacodec.github.io/av1-avif/). + * */ private fun isAvifHeader(imageHeaderBytes: ByteArray): Boolean { - // This check may not enough return imageHeaderBytes .toString(Charset.forName("UTF-8")) .contains("avif")