bramp / ffmpeg-cli-wrapper

Java wrapper around the FFmpeg command line tool
BSD 2-Clause "Simplified" License
1.72k stars 413 forks source link

Fix #264: Add additional CodecTypes to FFmpegStream #265

Closed Euklios closed 2 years ago

Euklios commented 2 years ago

I've added the required CodecTypes for SUBTITLE and ATTACHMENT. Additionally, I've added the DATA CodecType.

@bramp: These are all CodecTypes according to Documentation. Therefore, I fink we could also remove the corresponding TODO comment.

Fix #264 Fix #259

bramp commented 2 years ago

Sgtm. Sorry where are the to-do comments that we can remove?

Euklios commented 2 years ago

L12 in FFmpegStream the first line above the change that GitHub does not show.

Added a second commit