From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTP id 8E73B46F01 for ; Fri, 21 Jul 2023 13:50:15 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 3F3AB68C635; Fri, 21 Jul 2023 16:50:13 +0300 (EEST) Received: from mail-qv1-f52.google.com (mail-qv1-f52.google.com [209.85.219.52]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 9E6EB68C1B8 for ; Fri, 21 Jul 2023 16:50:06 +0300 (EEST) Received: by mail-qv1-f52.google.com with SMTP id 6a1803df08f44-63721c1bfa2so3545986d6.1 for ; Fri, 21 Jul 2023 06:50:06 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20221208; t=1689947405; x=1690552205; h=content-transfer-encoding:in-reply-to:from:content-language :references:to:subject:user-agent:mime-version:date:message-id:from :to:cc:subject:date:message-id:reply-to; bh=LTHL04iwHsDlkb84Bk90BcoB7b3ioZmPFv85dnCC7NE=; b=O1f5MW9tVUPp53PZeFDjUVrXf1AbtGG9dTLvgxaieMAxB2f8n8yQ4yihLMg2NC3CJF S+IGMn9Nh5rr1GcLCkmRqmSJXI7fcQiOGI4MAfI6GvZH2RiuI2BtrrF89hXMfZ3DIU5x FO0DxSE49+feJmuAQIN5B+MMDW39qcSp/ZBDybXrsGVoE6qh9oHdFlSKJUVHyxz2n0rU qk8ocYqTDyCjPg9DpLHnlhqKNjCYPWbgxQyS90s+mTkfdCeR1Bzf8lNOBNlehNSoYg4V pvgQvkwNSZtUA+FPhisHxKe3OuXz4nuwwsZ5xbOQDx6tNoXBAihocRsSDW+1bvRT1qRo kFRw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1689947405; x=1690552205; h=content-transfer-encoding:in-reply-to:from:content-language :references:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=LTHL04iwHsDlkb84Bk90BcoB7b3ioZmPFv85dnCC7NE=; b=QDuTSaqF84b6UrOpx8sq4meOW9v5jewajwEV9FM7buR2bzMcAJrb2wOOIMjuGUo9Dr etVHomfMugKTOg0Fpnm0vF6gvcdE7AHfZpkxX5qGgybtHnrwE0vh2PLidhR9v79lcwFp SO3Y4mGyQtVWd4MIS8u1AboD+rErATpdE73KmmXuuY1jUaoozt6wVBkqMYw5UC8wBs8j Pv2JqQn2NdTx/yOzHzxCxn9Q2kRbPFaAM0vhT4JAoADuXpktN3HHfzK8PwtLn2ODADoo yFkasJCNDn5H4ts40Q9LOLE4gnpws3KQE7Ika/2vDV17dGlPt219BQJR3Yto6cA/Esz5 aVdg== X-Gm-Message-State: ABy/qLbtBCb6zuXLZovK5iErqeQMtHNWhmGaWuO7U2Qo3/Sbwfyj6RvI HHLqGMjBsd3/Lt+zxDUiDwYr4yqepDA= X-Google-Smtp-Source: APBJJlFMoesBhMtaaTswr9Q/rQFf7aSW9XayNAYYpfVd3Xm/pue/wV4OywzdzpJ5u0Pz7uG3Yk22gw== X-Received: by 2002:a0d:d641:0:b0:577:4b1:c7a with SMTP id y62-20020a0dd641000000b0057704b10c7amr931955ywd.0.1689947384213; Fri, 21 Jul 2023 06:49:44 -0700 (PDT) Received: from [192.168.1.35] (c-68-56-149-176.hsd1.mi.comcast.net. [68.56.149.176]) by smtp.gmail.com with ESMTPSA id w67-20020a818646000000b005773b750d95sm889317ywf.28.2023.07.21.06.49.43 for (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 21 Jul 2023 06:49:43 -0700 (PDT) Message-ID: Date: Fri, 21 Jul 2023 09:49:43 -0400 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.13.0 To: FFmpeg Development References: <20230710235744.30513-1-leo.izen@gmail.com> Content-Language: en-US-large From: Leo Izen In-Reply-To: <20230710235744.30513-1-leo.izen@gmail.com> Subject: Re: [FFmpeg-devel] [PATCH v5 0/4] JPEG XL Parser X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: On 7/10/23 19:57, Leo Izen wrote: > Changes from v4: > > - Added an entropy decoder and full parser, which finds the > boundaries between files correctly > - Removed unnecessary logging in libjxldec > > Changes from v3: > > - Don't remove AV_CODEC_CAP_DR1 from libjxldec > - jpegxl_parse.o added to STLIBOBJS in avcodec/Makefile > - add pipe demuxer to avformat/Makefile's SHLIBOBJS > > Changes from v2: > > - Fix libjxldec to work with packets that are smaller than one frame > - Change how code is shared between libavcodec and libavformat to be more sensible. > - Fix the parser to work with large headers that proceed the codestream in a container format > (for example, if several-KB Exif boxes preceed the codestream.) > - Modify the parser to set width/height instead of avctx > Note: avctx->pix_fmt and s->format are both set, because otherwise the CLI tools won't print > the pixel format without libjxl enabled. > - Update the fate test based on the new parser's packetization > > This test relies on FATE samples that haven't been uploaded yet. To test, unzip > the following zipfile[1] in the FATE_SAMPLES directory, placing the test images in jxl/. > > [1]: https://buzo.us/y.zip > > sha256sum: 43a2eeb0dfdf471b47a9fdfb1653974fa156ceceb776891cc137569a8ebf0e75 > signature: https://buzo.us/R.asc > > Leo Izen (4): > avcodec/libjxldec: use internal AVFrame as buffered space > avcodec/jpegxl_parser: add JPEG XL parser > avformat/jpegxl: remove jpegxl_probe, instead call > avcodec/jpegxl_parse > fate/jpegxl_anim: add demuxer fate test for jpegxl_anim > > libavcodec/Makefile | 3 + > libavcodec/jpegxl.h | 94 ++ > libavcodec/jpegxl_parse.c | 520 ++++++ > libavcodec/jpegxl_parse.h | 72 + > libavcodec/jpegxl_parser.c | 1455 +++++++++++++++++ > libavcodec/libjxldec.c | 36 +- > libavcodec/parsers.c | 1 + > libavcodec/version.h | 2 +- > libavformat/Makefile | 6 +- > libavformat/img2dec.c | 4 +- > libavformat/jpegxl_anim_dec.c | 115 +- > .../{jpegxl_probe.h => jpegxl_parse.c} | 21 +- > libavformat/jpegxl_probe.c | 412 ----- > libavformat/version.h | 2 +- > tests/Makefile | 1 + > tests/fate/jxl.mak | 16 + > tests/ref/fate/jxl-anim-demux-belgium | 6 + > tests/ref/fate/jxl-anim-demux-icos4d | 6 + > tests/ref/fate/jxl-anim-demux-lenna256 | 6 + > tests/ref/fate/jxl-anim-demux-newton | 6 + > 20 files changed, 2238 insertions(+), 546 deletions(-) > create mode 100644 libavcodec/jpegxl.h > create mode 100644 libavcodec/jpegxl_parse.c > create mode 100644 libavcodec/jpegxl_parse.h > create mode 100644 libavcodec/jpegxl_parser.c > rename libavformat/{jpegxl_probe.h => jpegxl_parse.c} (55%) > delete mode 100644 libavformat/jpegxl_probe.c > create mode 100644 tests/fate/jxl.mak > create mode 100644 tests/ref/fate/jxl-anim-demux-belgium > create mode 100644 tests/ref/fate/jxl-anim-demux-icos4d > create mode 100644 tests/ref/fate/jxl-anim-demux-lenna256 > create mode 100644 tests/ref/fate/jxl-anim-demux-newton > Bumping for review, thanks. - Leo Izen _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".