On 29/05/2024 23:46, James Almer wrote:
> Signed-off-by: James Almer <jamrial@gmail.com>
> ---
>   libavcodec/packet.h | 14 ++++++++++++++
>   1 file changed, 14 insertions(+)
> 
> diff --git a/libavcodec/packet.h b/libavcodec/packet.h
> index a9a41576da..9dee035690 100644
> --- a/libavcodec/packet.h
> +++ b/libavcodec/packet.h
> @@ -330,6 +330,20 @@ enum AVPacketSideDataType {
>       */
>       AV_PKT_DATA_AMBIENT_VIEWING_ENVIRONMENT,
>   
> +    /**
> +     * The number of pixels to discard from the
> +     * top/bottom/left/right border of the decoded frame to obtain the sub-rectangle
> +     * intended for presentation.
> +     *
> +     * @code
> +     * u32le crop_top
> +     * u32le crop_bottom
> +     * u32le crop_left
> +     * u32le crop_right
> +     * @endcode
> +     */
> +    AV_PKT_DATA_FRAME_CROPPING,
> +
>       /**
>        * The number of side data types.
>        * This is not part of the public API/ABI in the sense that it may

Shouldn't this be propagated to the frame's crop fields, and back from 
the crop fields to the packet side-data?