Loading...

XML

Word

Printable

Details

Type: Bug
Resolution: Fixed
Priority: P2: Important
Fix Version/s: 6.8.2, 6.9.0 Beta2, 6.10.0 FF
Affects Version/s: 6.5, 6.8, 6.9
Component/s: Multimedia
Labels:
None

Epic Link:
OKR - Media Recording
Commits:
d477efb46 (dev), 4d8f2826d (dev), 02d196a54 (dev), 083de80cd (dev), 3fa293394 (dev), 867ce56b9 (dev), 768af4c3c (dev), 9296ed458 (dev), 0368f04d3 (dev), b1ed83374 (dev), b31c85c92 (dev), 784b9c722 (6.8), 7209970b2 (dev), c49c2de58 (dev), f04da6010 (dev)
Sprint:
Multimedia wk 47-48, Multimedia wk 49-50, Multimedia wk 51-2

Description

~~QTBUG-130478~~ reveals that QMediaFormat::isSupported only returns true with a small subset of combinations and may return false even if Qt Multimedia with the FFmpeg media backend is able to encode and decode the file.

Investigate the logic QMediaFormat::isSupported, and attemt to improve the logic.

Attachments

Issue Links

relates to

QTBUG-126174 [gstreamer] QMediaFormat does not properly populate supported file formats

Reported

Gerrit Reviews

- Issue Only
- Show All Reviews
- Show Open Reviews
- Show All Issues
- Show Open Issues

No reviews matched the request. Check your Options in the drop-down menu of this sections header.

Activity

People

Assignee:: Jøger Hansegård

Reporter:: Jøger Hansegård

Votes:: 0 Vote for this issue

Watchers:: 3 Start watching this issue

Dates

Created:: 14 Nov '24 11:16

Updated:: 17 Jan '25 22:06

Resolved:: 17 Jan '25 13:21

Gerrit Reviews

There are no open Gerrit changes

Show There are 58 closed Gerrit changes

Hide There are 58 closed Gerrit changes

Move free functions that operate on codecs to their own file: Gerrit Review:

Rename QFFmpeg::Codec to QFFmpeg::CodecContext: Gerrit Review:

Use QSharedData base as base class for explicitly shared data: Gerrit Review:

Tidy the includes in qffmpegcodeccontext_p.h: Gerrit Review:

Introduce Codec class convert free functions into member functions: Gerrit Review:

Extend Codec class, and switch to using Codec class where applicable: Gerrit Review:

Rename QFFmpeg::Codec to QFFmpeg::CodecContext: Gerrit Review:

Delete unused code: Gerrit Review:

Use std::optional to represent invalid/nullptr codecs: Gerrit Review:

Use QSharedData base as base class for explicitly shared data: Gerrit Review:

Tidy the includes in qffmpegcodeccontext_p.h: Gerrit Review:

Move free functions that operate on codecs to their own file: Gerrit Review:

Return QSpan instead of raw pointers from Codec class: Gerrit Review:

Return optional from findBestAVValue when predicate is not fulfilled: Gerrit Review:

Introduce Codec class convert free functions into member functions: Gerrit Review:

Extend Codec class, and switch to using Codec class where applicable: Gerrit Review:

Delete unused code: Gerrit Review:

Use std::optional to represent invalid/nullptr codecs: Gerrit Review:

Use std::find_if to search in spans with predicate, and rename functions: Gerrit Review:

Reduce use of nested predicates in findAVPixelFormat: Gerrit Review:

Add forEachAVPixelFormat() that evaluates a function for all formats: Gerrit Review:

Return QSpan instead of raw pointers from Codec class: Gerrit Review:

Return optional from findBestAVValue when predicate is not fulfilled: Gerrit Review:

Use std::find_if to search in spans with predicate, and rename functions: Gerrit Review:

Replace hwConfig(int) with a new hwConfigs() function to QFFmpeg::Codec: Gerrit Review:

Remove unused findAVEncoder function: Gerrit Review:

Remove inter-layer functions used to implement findDecoderWithHwAccel: Gerrit Review:

Remove inter-layer findAVCodec function: Gerrit Review:

Make QFFmpegMediaFormatInfo easier to read: Gerrit Review:

Document isCodecValid function implementation: Gerrit Review:

Rename QFFmpeg::Codec to QFFmpeg::CodecContext: Gerrit Review:

Reduce use of nested predicates in findAVPixelFormat: Gerrit Review:

Add forEachAVPixelFormat() that evaluates a function for all formats: Gerrit Review:

Replace hwConfig(int) with a new hwConfigs() function to QFFmpeg::Codec: Gerrit Review:

Remove unused findAVEncoder function: Gerrit Review:

Remove inter-layer functions used to implement findDecoderWithHwAccel: Gerrit Review:

Remove inter-layer findAVCodec function: Gerrit Review:

Use QSharedData base as base class for explicitly shared data: Gerrit Review:

Tidy the includes in qffmpegcodeccontext_p.h: Gerrit Review:

Move free functions that operate on codecs to their own file: Gerrit Review:

Make QFFmpegMediaFormatInfo easier to read: Gerrit Review:

Document isCodecValid function implementation: Gerrit Review:

Introduce Codec class convert free functions into member functions: Gerrit Review:

Extend Codec class, and switch to using Codec class where applicable: Gerrit Review:

Delete unused code: Gerrit Review:

Use std::optional to represent invalid/nullptr codecs: Gerrit Review:

Only call avcodec_get_supported_config if codec type supports config: Gerrit Review:

Only call avcodec_get_supported_config if codec type supports config: Gerrit Review:

Add regression test for QMediaFormat::resolveForEncoding(): Gerrit Review:

Test that QMediaFormat::resolveForEncoding() gives supported format: Gerrit Review:

Test that resolveForEncoding does not change already supported formats: Gerrit Review:

Test that QMediaFormat::resolveForEncoding() gives supported format: Gerrit Review:

Test that resolveForEncoding does not change already supported formats: Gerrit Review:

Test that QMediaFormat::resolveForEncoding() gives supported format: Gerrit Review:

Test that resolveForEncoding does not change already supported formats: Gerrit Review:

Improve documentation for QMediaRecorder::mediaFormat: Gerrit Review:

Improve documentation for QMediaRecorder::mediaFormat: Gerrit Review:

Improve documentation for QMediaRecorder::mediaFormat: Gerrit Review: