summaryrefslogtreecommitdiff
path: root/ffplay.c
Commit message (Collapse)AuthorAgeFilesLines
* ffplay: change keyboard volume control to logarithmicMarton Balint2017-02-081-4/+6
| | | | | | The command line parameter remains linear. Signed-off-by: Marton Balint <cus@passwd.hu>
* ffplay: fix borderless mode on WindowsMarton Balint2017-02-041-1/+3
| | | | Signed-off-by: Marton Balint <cus@passwd.hu>
* ffplay: allow borderless playback windowsLucas Sandery2017-02-011-0/+4
| | | | | | | | | For a pure video tile effect, and enabling better integration of playback windows into other programs. It would improve the looks in many situations and avoid ugly hacks like this: http://stackoverflow.com/q/31465630/315024 Signed-off-by: Lucas Sandery <lucas-sandery@users.noreply.github.com> Signed-off-by: Marton Balint <cus@passwd.hu>
* ffplay: fix indentation after last commitMarton Balint2017-01-221-65/+65
| | | | Signed-off-by: Marton Balint <cus@passwd.hu>
* ffplay: do not preallocate video textureMarton Balint2017-01-221-100/+20
| | | | | | Since the uploads happen in the main display function, it does not matter much. Signed-off-by: Marton Balint <cus@passwd.hu>
* ffplay: use buffersink accessors.Nicolas George2017-01-121-8/+8
|
* ffplay: add startup volume optionGanesh Ajjanagadde2016-12-251-1/+9
| | | | | | | Fixes Ticket 5389. Reviewed-by: Marton Balint <cus@passwd.hu> Signed-off-by: Ganesh Ajjanagadde <gajjanag@alum.mit.edu>
* ffplay: fix sws_scale possible out of bounds array accessMarton Balint2016-12-101-8/+8
| | | | | | | | | | | As I used simple RGBA formats for subtitles and for the video texture if avfilter is disabled I kind of assumed that sws_scale won't access data pointers and strides above index 0, but apparently that is not the case. Fixes Coverity CID 1396737, 1396738, 1396739, 1396740. Reviewed-by: Michael Niedermayer <michael@niedermayer.cc> Signed-off-by: Marton Balint <cus@passwd.hu>
* ffplay: allow fallback to software rendererMarton Balint2016-11-201-0/+4
| | | | Signed-off-by: Marton Balint <cus@passwd.hu>
* ffplay: add support for negative RGBA linesizeMarton Balint2016-11-201-2/+12
| | | | | | This fixes the crash reported in ticket #5947. Signed-off-by: Marton Balint <cus@passwd.hu>
* Merge commit 'beb62dac629603eb074a44c44389c230b5caac7c'Hendrik Leppkes2016-10-071-6/+2
|\ | | | | | | | | | | | | * commit 'beb62dac629603eb074a44c44389c230b5caac7c': Use AVFrame.pts instead of deprecated pkt_pts. Merged-by: Hendrik Leppkes <h.leppkes@gmail.com>
* | ffplay: remove unused viddec_width/viddec_heightMarton Balint2016-10-021-9/+0
| | | | | | | | | | Reviewed-by: Josh de Kock <josh@itanimul.li> Signed-off-by: Marton Balint <cus@passwd.hu>
* | ffplay: use decoder avctx for decoded subtitle width/heightMarton Balint2016-10-021-3/+2
| | | | | | | | | | | | | | Fixes ticket #5873. Reviewed-by: Josh de Kock <josh@itanimul.li> Signed-off-by: Marton Balint <cus@passwd.hu>
* | ffplay: add SDL2 supportMarton Balint2016-09-241-348/+246
| | | | | | | | | | | | | | This commit also drops SDL1 support for ffplay. Tested-by: James Almer <jamrial@gmail.com> (Windows, mingw-w64) Signed-off-by: Josh de Kock <josh@itanimul.li>
* | cmdutils: remove the current working directory from the DLL search path on win32Hendrik Leppkes2016-08-081-0/+2
| | | | | | | | | | Reviewed-by: Matt Oliver <protogonoi@gmail.com> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* | ffplay: Fix invalid array indexMichael Niedermayer2016-08-031-1/+1
| | | | | | | | | | | | Found-by: Thomas Guilbert <tguilbert@google.com> Fixes: clusterfuzz_usan-2016-08-02 Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* | ffplay: Fix usage of private lavfi APITimo Rothenpieler2016-06-291-1/+1
| | | | | | | | Reviewed-by: Michael Niedermayer <michael@niedermayer.cc>
* | Merge commit '41ed7ab45fc693f7d7fc35664c0233f4c32d69bb'Clément Bœsch2016-06-211-1/+1
|\ \ | |/ | | | | | | | | | | * commit '41ed7ab45fc693f7d7fc35664c0233f4c32d69bb': cosmetics: Fix spelling mistakes Merged-by: Clément Bœsch <u@pkh.me>
* | ffplay: ensure that we buffer at least 1 second of contentMarton Balint2016-06-191-1/+5
| | | | | | | | | | | | | | In order to do that, we keep track of the total duration of packets in a packet queue. Signed-off-by: Marton Balint <cus@passwd.hu>
* | ffplay: factorize checking if a stream needs additional packetsMarton Balint2016-06-191-4/+10
| | | | | | | | Signed-off-by: Marton Balint <cus@passwd.hu>
* | ffplay: simplify display codeMarton Balint2016-05-311-30/+13
| | | | | | | | | | | | And get rid of frame_queue_prev. Signed-off-by: Marton Balint <cus@passwd.hu>
* | ffplay: force setting alsa buffer sizeMarton Balint2016-05-071-0/+7
| | | | | | | | Signed-off-by: Marton Balint <cus@passwd.hu>
* | ffplay: convert to codecparMarton Balint2016-04-121-22/+34
| | | | | | | | | | Tested-by: Michael Niedermayer <michael@niedermayer.cc> Signed-off-by: Marton Balint <cus@passwd.hu>
* | ffplay: fix silence insertion on error or pauseMarton Balint2016-04-041-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Insertion of silence was a bit broken since df34b700981de606ca4847e1ed0bfdf9ac3e9104 because the info whether or not the source buffer supposed to be silence must be kept between callbacks. Failing to do so causes rogue samples from the last buffer to be presented, I guess even a crash can occur under some circumstances. This patch uses a NULL audio_buf to keep the silence state across audio callbacks. Reviewed-by: Lukasz Marek <lukasz.m.luki2 at gmail.com> Signed-off-by: Marton Balint <cus@passwd.hu>
* | ffplay: remove redundant silence bufferLukasz Marek2016-03-221-7/+6
| | | | | | | | Signed-off-by: Lukasz Marek <lukasz.m.luki2@gmail.com>
* | ffplay: Remove "&& 0" from already disabled debug codeMichael Niedermayer2016-03-201-1/+1
| | | | | | | | | | Reviewed-by: Marton Balint <cus@passwd.hu> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* | ffplay: update docs after previous changes in ffplay mouse behaviourVittorio Gambaletta (VittGam)2016-01-201-1/+2
| | | | | | | | | | Signed-off-by: Vittorio Gambaletta <ffmpeg-dev@vittgam.net> Signed-off-by: Marton Balint <cus@passwd.hu>
* | ffplay: toggle full screen when double-clicking the video window with the ↵Vittorio Gambaletta (VittGam)2016-01-191-0/+10
| | | | | | | | | | | | | | | | | | | | | | left mouse button Now that the seek only happens with the right mouse button, it makes sense to toggle full screen when double-clicking with the left mouse button, like other video players do. Signed-off-by: Vittorio Gambaletta <ffmpeg-dev@vittgam.net> Signed-off-by: Marton Balint <cus@passwd.hu>
* | ffplay: seek only when pressing the right mouse button on the video windowVittorio Gambaletta (VittGam)2016-01-191-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | Seeking by clicking on the video window can be annoying, because the user might click on it accidentally while eg. trying to get focus on it, and ffplay seeks instead. This commit changes that behaviour to seek only when the right mouse button is used to click and drag on the window. Signed-off-by: Vittorio Gambaletta <ffmpeg-dev@vittgam.net> Signed-off-by: Marton Balint <cus@passwd.hu>
* | Revert "ffplay: Fix auto insertion point of rotation filter"Michael Niedermayer2015-12-281-5/+4
| | | | | | | | | | | | | | | | Revert requested by Balint Marton <cus@passwd.hu> See: [FFmpeg-devel] ffplay: insertion point of the auto rotation filter - Github ticket #141 This reverts commit 9cc1e644f39ed6a32e3a9bf5d0d6d663ca5791ab.
* | ffplay: replace rint by lrintGanesh Ajjanagadde2015-12-251-2/+2
| | | | | | | | | | | | | | | | avoids the float to integer cast, and is slightly superior in terms of rounding ("Dutch/Gauss rounding"). Reviewed-by: Ronald S. Bultje <rsbultje@gmail.com> Signed-off-by: Ganesh Ajjanagadde <gajjanagadde@gmail.com>
* | ffplay: Fix auto insertion point of rotation filterwanzhang2015-12-251-4/+5
| | | | | | | | | | | | Fixes watermark rotation for videos using auto rotation Closes #141
* | ffplay: remove existing AVPicture usageMarton Balint2015-12-191-24/+27
| | | | | | | | | | | | | | It is deprecated. Reviewed-by: Paul B Mahol <onemda@gmail.com> Signed-off-by: Marton Balint <cus@passwd.hu>
* | ffplay: use hypot()Ganesh Ajjanagadde2015-12-061-3/+3
| | | | | | | | | | Reviewed-by: Marton Balint <cus@passwd.hu> Signed-off-by: Ganesh Ajjanagadde <gajjanagadde@gmail.com>
* | Replace remaining occurances of av_free_packet with av_packet_unrefHendrik Leppkes2015-10-271-5/+5
| |
* | Merge commit 'a5d42043093a39636a1f4021a37dd9c612479f6f'Hendrik Leppkes2015-10-271-4/+0
|\ \ | |/ | | | | | | | | | | * commit 'a5d42043093a39636a1f4021a37dd9c612479f6f': avformat: Always return ref-counted AVPacket Merged-by: Hendrik Leppkes <h.leppkes@gmail.com>
* | ffplay: use a separate struct for the rescaled YUVA AVSubtitle rectanglesMarton Balint2015-10-241-22/+26
| | | | | | | | | | | | | | | | | | | | | | Current code segfaults since the deprecation of AVSubtitleRect.pict because it freed/realloced AVSubtitleRect.pict.data by itself. The new code stores the generated YUVA AVSubtitle rectangles in their own struct and keeps the original AVSubtitle structure untouched, because overwriting it is considered invalid API usage. Signed-off-by: Marton Balint <cus@passwd.hu>
* | ffplay: eliminate stream_component_close forward declarationMarton Balint2015-10-121-59/+57
| | | | | | | | | | | | No change in fuctionality. Signed-off-by: Marton Balint <cus@passwd.hu>
* | ffplay: close streams and AVFormatContext in the main threadMarton Balint2015-10-121-14/+14
| | | | | | | | | | | | To avoid race conditions. Signed-off-by: Marton Balint <cus@passwd.hu>
* | ffplay: use correct context for av_logGanesh Ajjanagadde2015-10-051-3/+3
| | | | | | | | | | | | | | | | | | 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-051-1/+4
| | | | | | | | | | | | | | | | | | 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-051-3/+10
| | | | | | | | | | | | | | | | 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>
* | ffplay: more robust mutex creationGanesh Ajjanagadde2015-10-051-0/+6
| | | | | | | | | | | | | | | | | | SDL_CreateMutex can fail: https://wiki.libsdl.org/SDL_CreateMutex. 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: more robust thread creationGanesh Ajjanagadde2015-10-041-4/+12
| | | | | | | | | | | | | | | | | | SDL_CreateThread can fail: https://wiki.libsdl.org/SDL_CreateThread. This patch makes thread creation more robust in one instance. Signed-off-by: Ganesh Ajjanagadde <gajjanagadde@gmail.com> Signed-off-by: Marton Balint <cus@passwd.hu>
* | ffplay: more robust mutex, condition variable handlingGanesh Ajjanagadde2015-10-031-4/+14
| | | | | | | | | | | | | | | | | | SDL_CreateMutex and SDL_CreateCond can fail: https://wiki.libsdl.org/SDL_CreateMutex. This patch makes handling more robust in one instance. Signed-off-by: Ganesh Ajjanagadde <gajjanagadde@gmail.com> Signed-off-by: Marton Balint <cus@passwd.hu>
* | doc/ffplay, ffplay: add information regarding volume controlGanesh Ajjanagadde2015-09-271-0/+3
| | | | | | | | | | | | | | | | | | ffplay now supports dynamic volume control. This documents the supported behavior. Signed-off-by: Ganesh Ajjanagadde <gajjanagadde@gmail.com> Reviewed-by: Stefano Sabatini <stefasab@gmail.com> Signed-off-by: Marton Balint <cus@passwd.hu>
* | ffplay: add support for interactive volume controlGanesh Ajjanagadde2015-09-271-1/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a feature heavily inspired by the mpv player. At the moment, methods for adjusting volume in ffplay are rather clumsy: either one needs to set it system-wide, or one needs to set it via the volume filter. This patch adds key bindings identical to the mpv defaults for muting/unmuting and increasing/decreasing the volume interactively without any introduction of external dependencies. TODO: doc update, possible mouse button bindings (mpv has this). Signed-off-by: Ganesh Ajjanagadde <gajjanagadde@gmail.com> Signed-off-by: Marton Balint <cus@passwd.hu>
* | ffplay: dynamically allocate filename bufferGanesh Ajjanagadde2015-09-271-2/+5
| | | | | | | | | | | | | | | | | | | | | | filename was set to an arbitrary 1024 characters. ffplay would thus be unable to play files whose name exceeds that arbitrary threshold. This patch dynamically allocates and frees the filename buffer to remove such limitations. Signed-off-by: Ganesh Ajjanagadde <gajjanagadde@gmail.com> Signed-off-by: Marton Balint <cus@passwd.hu>
* | ffplay: introduce key repeatsGanesh Ajjanagadde2015-09-271-0/+2
| | | | | | | | | | | | | | | | | | | | Key repeats have been introduced simply because they improve usability in my experience for volume, brightness, and other such controls by speeding up the time taken to go from 0 to max intensity. As a side benefit, this enables rapid seeking through a file via left/right keys. Signed-off-by: Ganesh Ajjanagadde <gajjanagadde@gmail.com> Signed-off-by: Marton Balint <cus@passwd.hu>
* | Remove left-over FF_API_AVFILTERBUFFER cruftHendrik Leppkes2015-09-051-1/+0
| |