Skip to content

Conversation

@mkver
Copy link

@mkver mkver commented Jul 11, 2025

Mostly removal of dead encode checks.

mkver added 5 commits July 11, 2025 18:52
It is supposed to be unused by encoders.

Signed-off-by: Andreas Rheinhardt <[email protected]>
Can be readded if needed (likely never).

Signed-off-by: Andreas Rheinhardt <[email protected]>
The dimensions have already been checked during init.

Signed-off-by: Andreas Rheinhardt <[email protected]>
@mkver
Copy link
Author

mkver commented Jul 11, 2025

/submit

@ffmpeg-codebot
Copy link

Submitted as [email protected]

To fetch this version into FETCH_HEAD:

git fetch https://github.com/ffstaging/FFmpeg pr-ffstaging-107/mkver/encode_checks-v1

To fetch this version to local tag pr-ffstaging-107/mkver/encode_checks-v1:

git fetch --no-tags https://github.com/ffstaging/FFmpeg tag pr-ffstaging-107/mkver/encode_checks-v1

@ffmpeg-codebot
Copy link

On the FFmpeg mailing list, Andreas Rheinhardt wrote (reply to this):

ffmpegagent:
> Mostly removal of dead encode checks.
> 
> Andreas Rheinhardt (5):
>   avcodec/av1dec,libdav1d,wbmpdec: Avoid direct access to GetByteContext
>   avcodec/encode: Simplify pixel format validity check
>   avcodec/encode: Ignore coded_{width,height}
>   avcodec/encode: Remove dead code
>   avcodec/encode: Remove redundant av_image_check_size2()
> 
>  libavcodec/av1dec.c   |  2 +-
>  libavcodec/encode.c   | 28 ++++++----------------------
>  libavcodec/libdav1d.c |  2 +-
>  libavcodec/wbmpdec.c  |  3 ++-
>  4 files changed, 10 insertions(+), 25 deletions(-)
> 
> 
> base-commit: 3ce348063c9433e33a5cb1ac79ac1efa37c21621
> Published-As: https://github.com/ffstaging/FFmpeg/releases/tag/pr-ffstaging-107%2Fmkver%2Fencode_checks-v1
> Fetch-It-Via: git fetch https://github.com/ffstaging/FFmpeg pr-ffstaging-107/mkver/encode_checks-v1
> Pull-Request: https://github.com/ffstaging/FFmpeg/pull/107

Will apply this patchset tomorrow unless there are objections.

- Andreas

_______________________________________________
ffmpeg-devel mailing list
[email protected]
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
[email protected] with subject "unsubscribe".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant