Skip to content

videoio: added bad parameters handling to VideoWriter#14156

Merged
alalek merged 7 commits intoopencv:masterfrom
mshabunin:videowriter-bad-test
Mar 29, 2019
Merged

videoio: added bad parameters handling to VideoWriter#14156
alalek merged 7 commits intoopencv:masterfrom
mshabunin:videowriter-bad-test

Conversation

@mshabunin
Copy link
Copy Markdown
Contributor

@mshabunin mshabunin commented Mar 26, 2019

This pullrequest changes

We expect no throw behavior for incorrect VideoWriter parameters, added test to check this.

Changes in GStreamer backend:

  • replaced abort() calls with error handling

Changes in AVFoundation VideoWriter backend:

  • added support to UTF-8 filenames (not tested)
  • removed fallback to mp4/h264 codec in case when extension or FourCC are not supported (supported extensions: mov, qt, mp4, m4v, supported FourCC codes: JPEG, MJPEG, jpeg, mjpeg, H264, avc1)
  • added basic error handling and cleanup

Changes in MediaSDK backend:

  • added soft fps check and modified
force_builders=Linux AVX2
docker_image:Linux AVX2=ubuntu:18.04

@mshabunin mshabunin added this to the 4.1.0 milestone Mar 26, 2019
@mshabunin mshabunin requested a review from alalek March 26, 2019 12:22
@alalek
Copy link
Copy Markdown
Member

alalek commented Mar 26, 2019

We definitely should ban all "assert()" calls in OpenCV code, including tests.

Copy link
Copy Markdown
Member

@alalek alalek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@alalek alalek merged commit 52cd1da into opencv:master Mar 29, 2019
@mshabunin mshabunin deleted the videowriter-bad-test branch July 14, 2020 17:18
a-sajjad72 pushed a commit to a-sajjad72/opencv that referenced this pull request Mar 30, 2023
* videoio: added bad parameters handling to VideoWriter

* AVFoundation/Writer: support UTF-8, check input parameters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: videoio port/backport done Label for maintainers. Authors of PR can ignore this

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants