summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
* avformat/omadec: Fix { typoMichael Niedermayer2015-10-05
| | | | | | Fixes CID1324299 Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* ffplay: use correct context for av_logGanesh Ajjanagadde2015-10-05
| | | | | | | | | Recent commits c3e8de1c248f8c742dd9e61a0c71ee56bba22c28 and 8dc6e92c3dc67a85026f3010045c7a28b1c0adc8 used av_log incorrectly. This fixes such usage. Signed-off-by: Ganesh Ajjanagadde <gajjanagadde@gmail.com> Signed-off-by: Marton Balint <cus@passwd.hu>
* ffplay: more robust condition variable creationGanesh Ajjanagadde2015-10-05
| | | | | | | | | SDL_CreateCond can fail: https://wiki.libsdl.org/SDL_CreateCond. This patch makes creation more robust in one instance. Signed-off-by: Ganesh Ajjanagadde <gajjanagadde@gmail.com> Signed-off-by: Marton Balint <cus@passwd.hu>
* ffplay: log SDL error messagesGanesh Ajjanagadde2015-10-05
| | | | | | | | This logs the SDL error messages on failure of creation of SDL_CreateMutex, SDL_CreateCond, and SDL_CreateThread. Signed-off-by: Ganesh Ajjanagadde <gajjanagadde@gmail.com> Signed-off-by: Marton Balint <cus@passwd.hu>
* avutil/attributes: add av_warn_unused_resultGanesh Ajjanagadde2015-10-05
| | | | | | | | | | | | | | GCC 3.4 introduced an attribute warn_unused_result to warn when a programmer discards the return value. Applying this judiciously across the codebase can help in fixing a lot of problems. At a high level, functions which return error codes should always be checked. More concretely, consider the functions ff_add_format and the like in avfilter/formats.h. A quick examination shows that a large portion of libavfilter fails to handle the associated errors, usually AVERROR(ENOMEM). The above example was where I observed the utility of this, but it should be useful in many places across the code base. Signed-off-by: Ganesh Ajjanagadde <gajjanagadde@gmail.com> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* dnxhd: add better support for CIDs 1270 to 1274Jeremy James2015-10-05
| | | | | | | | | | | These are DNxHR profiles with the following properties: - Variable size in a profile (property added in a previous commit), requiring variable-sized macroblock table; - Variable bitdepth, up to 12 bits. - Better validation of buffer sizes and positions Signed-off-by: Christophe Gisquet <christophe.gisquet@gmail.com> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* dnxhd: add CID 1270Christophe Gisquet2015-10-05
| | | | | | | | | | This a 4:4:4 10 bits profile, where image size is not fixed by the profile, and which strays a bit outside the old frame header parsing code. Fixes ticket #4581 (DNxHR is not stricly supported, but that sequence is). Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* dnxhd: add decoder support for DNxHRJeremy James2015-10-05
| | | | | Signed-off-by: Christophe Gisquet <christophe.gisquet@gmail.com> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* isom: add support for DNxHR codec familyChristophe Gisquet2015-10-05
| | | | | | This is actually similar to DNxHD. Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* dnxhddec: Introduce DNXHD_VARIABLEChristophe Gisquet2015-10-05
| | | | | | | Currently not used, but will be used to indicate that a CIDEntry field is not set, because it is variable, and that checks should be adapted. Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* dnxhdenc: do not select 4:4:4 profilesChristophe Gisquet2015-10-05
| | | | | | The encoder can only deal with 4:2:2. Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* dnxhd: profile flagsChristophe Gisquet2015-10-05
| | | | | | | Move the 'interlaced' flag to this element (arbitrarily set to 16bits). This should allow better detection/selection of profiles. Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* dnxhddec: cleanup frame header parsingChristophe Gisquet2015-10-05
| | | | | | | Rely more on the actual syntax from the specs (also seen in the encoder code). Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* avcodec/mmaldec: Use av_assert0() instead of assert()Michael Niedermayer2015-10-05
| | | | Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* Merge commit '39f01e346cab464ef6c0d4ec58cc13b7123e60d8'Hendrik Leppkes2015-10-05
|\ | | | | | | | | | | | | * commit '39f01e346cab464ef6c0d4ec58cc13b7123e60d8': mmaldec: be more tolerant against MMAL not returning decoded output Merged-by: Hendrik Leppkes <h.leppkes@gmail.com>
| * mmaldec: be more tolerant against MMAL not returning decoded outputwm42015-10-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In some situations, MMAL won't return a decoded frame for certain input frames. This can happen if a frame fails to decode, or if a packet does not actually contain a complete frame. In these situations, we would deadlock (or actually timeout) waiting for an expected output frame, which is not ideal. On the other hand, there are situations where we definitely have to block to avoid deadlocks. (This mess is a consequence of trying to map MMAL's asynchronous and flexible dataflow to libavcodec, which is more static and rigid.) Solve this by doing a blocking wait only if the amount of buffered data is too big. The whole purpose of the blocking wait is to avoid excessive buffering of input data, so we can skip it if it appears to be low. The consequence is that libavcodec can gracefully return no frame to the API user. We want to track the number of full packets to make our heuristic work. But MMAL buffers are fixed-size, requiring splitting large packets. This is why the previous commit is needed. We use the ..._FRAME_END flag to remember packet boundaries, but MMAL does not preserve these buffer flags when returning buffers to the user. Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
* | Merge commit '65db4899fa8790049bec3af16ecdb75dd81051fd'Hendrik Leppkes2015-10-05
|\| | | | | | | | | | | | | * commit '65db4899fa8790049bec3af16ecdb75dd81051fd': mmaldec: refactor to have more context per MMAL input buffer Merged-by: Hendrik Leppkes <h.leppkes@gmail.com>
| * mmaldec: refactor to have more context per MMAL input bufferwm42015-10-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | | The next commit needs 1 bit of additional information per MMAL buffer sent to the MMAL input port. This information will be needed when the buffer is recycled (i.e. returned by the input port's callback). Normally, we could use MMAL_BUFFER_HEADER_FLAG_USER0, but that is unexpectedly not preserved. Do this by storing a pointer to FFBufferEntry in the MMAL buffer's user data, instead of an AVBufferRef. This also changes the lifetime of FFBufferEntry. Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
* | Merge commit 'eae58428bd199f7d4670bf471f56ed204430878e'Hendrik Leppkes2015-10-05
|\| | | | | | | | | | | | | * commit 'eae58428bd199f7d4670bf471f56ed204430878e': avcodec: Do not lock during open for codecs marked as having threadsafe init Merged-by: Hendrik Leppkes <h.leppkes@gmail.com>
| * avcodec: Do not lock during open for codecs marked as having threadsafe initDerek Buitenhuis2015-10-04
| | | | | | | | | | Signed-off-by: Derek Buitenhuis <derek.buitenhuis@gmail.com> Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
* | Merge commit 'abe9adfb31566c415fd830a8d4977c79512d4385'Hendrik Leppkes2015-10-05
|\| | | | | | | | | | | | | * commit 'abe9adfb31566c415fd830a8d4977c79512d4385': rangecoder: Use AV_RB16 instead of bytestream_get_be16 Merged-by: Hendrik Leppkes <h.leppkes@gmail.com>
| * rangecoder: Use AV_RB16 instead of bytestream_get_be16Vittorio Giovara2015-10-04
| | | | | | | | | | | | | | | | Silence an incompatible-pointer-types-discards-qualifiers warning from clang. rangecoder.c:58:34: warning: passing 'uint8_t **' (aka 'unsigned char **') to parameter of type 'const uint8_t **' (aka 'const unsigned char **') discards qualifiers in nested pointer types [-Wincompatible-pointer-types-discards-qualifiers]
* | Merge commit 'b8b13acd70a1c1b6c044f967d78cb82f1a53887b'Hendrik Leppkes2015-10-05
|\| | | | | | | | | | | | | * commit 'b8b13acd70a1c1b6c044f967d78cb82f1a53887b': hevc: Simplify logical check Merged-by: Hendrik Leppkes <h.leppkes@gmail.com>
| * hevc: Simplify logical checkVittorio Giovara2015-10-04
| | | | | | | | | | | | The intended meaning is "if this block is the first block in a slice then its left boundary is a slice boundary". Silence a logical-not-parentheses warning from gcc.
* | Merge commit '10aa6f9db44e96977e7155fdc3e62619e0e00a15'Hendrik Leppkes2015-10-05
|\| | | | | | | | | | | | | * commit '10aa6f9db44e96977e7155fdc3e62619e0e00a15': h264: Use the correct argument context in debug log Merged-by: Hendrik Leppkes <h.leppkes@gmail.com>
| * h264: Use the correct argument context in debug logVittorio Giovara2015-10-04
| |
* | Merge commit '4628443ca3534060888dd0015b229337eac13fd2'Hendrik Leppkes2015-10-05
|\| | | | | | | | | | | | | * commit '4628443ca3534060888dd0015b229337eac13fd2': h263: Drop uninitialized variable use from log message Merged-by: Hendrik Leppkes <h.leppkes@gmail.com>
| * h263: Drop uninitialized variable use from log messageVittorio Giovara2015-10-04
| |
* | Merge commit '1da2a20763ae9ca579d5fd20763065871ddf6311'Hendrik Leppkes2015-10-05
|\| | | | | | | | | | | | | * commit '1da2a20763ae9ca579d5fd20763065871ddf6311': dvbsubdec: Fix function return type Merged-by: Hendrik Leppkes <h.leppkes@gmail.com>
| * dvbsubdec: Fix function return typeVittorio Giovara2015-10-04
| |
* | Merge commit 'cab63a8b594cdc365bb2581a12b3ac8e6dd480b2'Hendrik Leppkes2015-10-05
|\| | | | | | | | | | | | | * commit 'cab63a8b594cdc365bb2581a12b3ac8e6dd480b2': dv: Mark internal frame reference as const Merged-by: Hendrik Leppkes <h.leppkes@gmail.com>
| * dv: Mark internal frame reference as constVittorio Giovara2015-10-04
| | | | | | | | | | | | Silence a warning due to frame assignment in dvenc. All uses of the reference in dvdec are read only, except the ones in the main decoding function, so use the frame pointer directly there.
* | Merge commit '4c160fa23996c05efcd952ccfac2359311d8a1bc'Hendrik Leppkes2015-10-05
|\| | | | | | | | | | | | | * commit '4c160fa23996c05efcd952ccfac2359311d8a1bc': dcadec: Always initialize return variable Merged-by: Hendrik Leppkes <h.leppkes@gmail.com>
| * dcadec: Always initialize return variableVittorio Giovara2015-10-04
| | | | | | | | Silence an uninitialized warning from clang.
* | Merge commit 'bb198c4997d5036f3bf91de51e44f807115677d0'Hendrik Leppkes2015-10-05
|\| | | | | | | | | | | | | * commit 'bb198c4997d5036f3bf91de51e44f807115677d0': d3d11va: make av_d3d11va_alloc_context() available at all times Merged-by: Hendrik Leppkes <h.leppkes@gmail.com>
| * d3d11va: make av_d3d11va_alloc_context() available at all timesAnton Khirnov2015-10-03
| | | | | | | | The public API should not depend on the build configuration.
* | Merge commit 'f3202871598f59b570b31b01cfeb64b8fedbd700'Hendrik Leppkes2015-10-05
|\| | | | | | | | | | | | | * commit 'f3202871598f59b570b31b01cfeb64b8fedbd700': d3d11va: check for malloc failure Merged-by: Hendrik Leppkes <h.leppkes@gmail.com>
| * d3d11va: check for malloc failureAnton Khirnov2015-10-03
| |
* | Merge commit '2c3dbff1d89bbf8300c121c06524c014cb6e0915'Hendrik Leppkes2015-10-05
|\| | | | | | | | | | | | | * commit '2c3dbff1d89bbf8300c121c06524c014cb6e0915': avutil/avstring: Inline some tiny functions Merged-by: Hendrik Leppkes <h.leppkes@gmail.com>
| * avutil/avstring: Inline some tiny functionsHenrik Gramner2015-10-03
| | | | | | | | | | | | | | They're short enough that inlining them actually reduces code size due to all the overhead associated with making a function call. Signed-off-by: Anton Khirnov <anton@khirnov.net>
* | Merge commit 'f56a08559334b7eb6b3fedbc0cc741887f6067ae'Hendrik Leppkes2015-10-05
|\| | | | | | | | | | | | | * commit 'f56a08559334b7eb6b3fedbc0cc741887f6067ae': matroskaenc: Don't write a track language tag Merged-by: Hendrik Leppkes <h.leppkes@gmail.com>
| * matroskaenc: Don't write a track language tagJohn Stebbins2015-10-03
| | | | | | | | | | | | | | | | | | "language" is not an offical matroska tag. Track languages are specified with the MATROSKA_ID_TRACKLANGUAGE ebml. Writing the tag overrides the ebml specified language during playback with libav and some other players. Signed-off-by: Anton Khirnov <anton@khirnov.net>
* | Merge commit '8bb376cf6b4ab8645daedb8becaa7163656436a4'Hendrik Leppkes2015-10-05
|\| | | | | | | | | | | | | * commit '8bb376cf6b4ab8645daedb8becaa7163656436a4': checkasm: Fix the function name sorting algorithm Merged-by: Hendrik Leppkes <h.leppkes@gmail.com>
| * checkasm: Fix the function name sorting algorithmHenrik Gramner2015-10-03
| | | | | | | | | | | | The previous implementation was behaving incorrectly in some corner cases. Signed-off-by: Anton Khirnov <anton@khirnov.net>
* | ffserver: avoid leaking pathname at exitReynaldo H. Verdejo Pinochet2015-10-04
| | | | | | | | | | | | Fixes Coverity CID 1325681 Signed-off-by: Reynaldo H. Verdejo Pinochet <reynaldo@osg.samsung.com>
* | ffserver: fix up rate units in server messagesReynaldo H. Verdejo Pinochet2015-10-04
| | | | | | | | Signed-off-by: Reynaldo H. Verdejo Pinochet <reynaldo@osg.samsung.com>
* | ffserver: wrap around bandwidth exceeded msg textReynaldo H. Verdejo Pinochet2015-10-04
| | | | | | | | Signed-off-by: Reynaldo H. Verdejo Pinochet <reynaldo@osg.samsung.com>
* | ffserver: wrap around too-busy message textReynaldo H. Verdejo Pinochet2015-10-04
| | | | | | | | Signed-off-by: Reynaldo H. Verdejo Pinochet <reynaldo@osg.samsung.com>
* | ffserver: unify exit path in _write_index()Reynaldo H. Verdejo Pinochet2015-10-04
| | | | | | | | Signed-off-by: Reynaldo H. Verdejo Pinochet <reynaldo@osg.samsung.com>
* | ffserver: fix line wrapping on function declsReynaldo H. Verdejo Pinochet2015-10-04
| | | | | | | | Signed-off-by: Reynaldo H. Verdejo Pinochet <reynaldo@osg.samsung.com>