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 7B00745F97 for ; Thu, 22 Jun 2023 14:20:12 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id AD86968C196; Thu, 22 Jun 2023 17:20:07 +0300 (EEST) Received: from mail-oo1-f41.google.com (mail-oo1-f41.google.com [209.85.161.41]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id BDE5068BE2D for ; Thu, 22 Jun 2023 17:20:01 +0300 (EEST) Received: by mail-oo1-f41.google.com with SMTP id 006d021491bc7-55e11f94817so4437940eaf.2 for ; Thu, 22 Jun 2023 07:20:01 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20221208; t=1687443599; x=1690035599; h=content-transfer-encoding:in-reply-to:from:references:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=T0J6+ACo3zkR3Yyw+GE/uBMEVsH0iA4ftQXh+sHRdiA=; b=hijX+JAnmOGowfz+raP0kPeMLy+VhdH/M0NRAacOLKxBiCV20YLiogqmx5pPwGuDRl uKLwqF+wneqK3/HXBK577copQkvggbQpjYLrdP2ylSEpJ3IhJBJWufwuncfitKb0+nyJ izyKeP+RLdAMYAdyC5VYaYhO39SskD+k1QMmdEnR1CBTfl3hpz5MVmcDhuZXwK3+xT5u uToodgpd4SXeJs6ad1Vl1DpablKJwkKVOve/RNTK9Gr7+uZlUn/7qMpPcIWkKfV5zrn7 d0GDJZR454r4oFv3ilTVdwwKAGt0w13PNawSt0BzKHN3aI74gBzjh/5uJ0Tr4YfPW2HE PwWg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1687443599; x=1690035599; h=content-transfer-encoding:in-reply-to:from:references:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=T0J6+ACo3zkR3Yyw+GE/uBMEVsH0iA4ftQXh+sHRdiA=; b=C0FPLUbWco/5ebWzo/EnjVUYWuV18hQQ0+ixo7uAGOoUqHcKWrSU1ys1ZM6rRuhC3e rS0ZFrHVOmrvNOVnIU3jwhO8QqMfPH2+AlNkqelrCrQabnZdTkuPzSLaEfmNCFAzyVSC 9DO5/FEqav1Gi+N2UAce8N0d4Sr+rg+s3eBebEcoXc0Nu0fjFuxTbft3AnxOm9aietYn WY9HU7ndIoDE/CIo6A5m/XwOY6UPTgEBj2QHIRaghDjrwawwXD+IhfHNKwqd4bY/aY3O Z+HH9i89UbmXLHsiIclwEXutk1MYz5i0DSS3lvNI8r2/8TjjZucbOFCwKV+CVcUcQUfQ U+9w== X-Gm-Message-State: AC+VfDwVWqW7gXR5oiQBQelqUavaH/b9TnXkeaMqkzRbtcjrs+ZHCOoO CxOyYKLjX8tZZ38seYDLwB24/ENVEzs= X-Google-Smtp-Source: ACHHUZ4aS/t7hdzANSt9DtjxMsMtmQHPFUBlJT5xQ3P7ZUAIanN54RJrdRgZV5YvXKDyHaCRxyFQCA== X-Received: by 2002:a05:6808:3082:b0:3a0:5800:38aa with SMTP id bl2-20020a056808308200b003a0580038aamr3415083oib.24.1687443599370; Thu, 22 Jun 2023 07:19:59 -0700 (PDT) Received: from [192.168.0.12] (host197.190-225-105.telecom.net.ar. [190.225.105.197]) by smtp.gmail.com with ESMTPSA id o13-20020a0568080f8d00b003a034e62741sm3215315oiw.2.2023.06.22.07.19.58 for (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 22 Jun 2023 07:19:58 -0700 (PDT) Message-ID: <77d7cc53-4047-fc9a-d0c0-6ad165b68fb1@gmail.com> Date: Thu, 22 Jun 2023 11:19:57 -0300 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.12.0 Content-Language: en-US To: ffmpeg-devel@ffmpeg.org References: <20230622004313.69697-1-leo.izen@gmail.com> <20230622004313.69697-2-leo.izen@gmail.com> From: James Almer In-Reply-To: <20230622004313.69697-2-leo.izen@gmail.com> Subject: Re: [FFmpeg-devel] [PATCH v2 1/3] avcodec/jpegxl_parser: add 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 6/21/2023 9:43 PM, Leo Izen wrote: > +static int jpegxl_parse(AVCodecParserContext *s, AVCodecContext *avctx, > + const uint8_t **poutbuf, int *poutbuf_size, > + const uint8_t *buf, int buf_size) > +{ > + JXLParseContext *ctx = s->priv_data; > + int ret; > + > + *poutbuf_size = 0; > + *poutbuf = NULL; > + > + if (!ctx->parsed_header) { > + if (AV_RL64(buf) == FF_JPEGXL_CONTAINER_SIGNATURE_LE) { > + int copied; > + uint8_t codestream_header[4096]; > + ret = ff_jpegxl_collect_codestream_header(buf, buf_size, codestream_header, > + sizeof(codestream_header), &copied); > + if (ret < 0) > + return ret; > + /* copied may be larger than the bufsize if stuff was skipped */ > + copied = FFMIN(copied, sizeof(codestream_header)); > + ret = ff_jpegxl_parse_codestream_header(codestream_header, copied, 0, &ctx->meta); > + if (ret < 0) > + return ret; > + } else { > + ret = ff_jpegxl_parse_codestream_header(buf, buf_size, 1, &ctx->meta); > + if (ret < 0) > + return ret; > + } > + avctx->width = ctx->meta.width; > + avctx->height = ctx->meta.height; Set s->width and s->height instead. The generic code will set the avctx fields if required. _______________________________________________ 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".