On 9/9/2024 8:33 AM, Anton Khirnov wrote:
Quoting James Almer (2024-09-08 15:26:51)On 9/6/2024 8:44 AM, Anton Khirnov wrote:Quoting James Almer (2024-08-31 18:31:10)+typedef struct AVStreamGroupLCEVC { + const AVClass *av_class; + + /** + * Width of the final stream for presentation. + */ + int width; + /** + * Height of the final image for presentation. + */ + int height;What's the point of exporting these separately when they are the same as the enhancement AVStream's dimensions?The enhancement stream is of data type, so width/height are undefined in it. Hence including these here.Why not define them then? Seems simpler and more straightforward to me.
I personally don't think re-defining the AVStream API in order to have two fields start meaning something for DATA type streams for the sake of only one codec_id is better than just having said fields defined in the specific stream group.
OpenPGP_signature.asc
Description: OpenPGP digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
