| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
| |
Looks like this relied on undefined behavior.
|
| |
|
|
|
|
|
| |
Yep, smb_username/password were unused since forever, even in MPlayer.
Removal untested. (Does anyone even use smb://?)
|
|
|
|
|
| |
Get rid of the stupid and error-prone buffer size calculations, use
snprintf instead of strcpy.
|
|
|
|
|
|
| |
The only thing that used mp_load_per_file_config() was inside
configfiles.c too, so remove the declaration from core.h and move the
function before its use.
|
|
|
|
|
|
| |
Code for loading "[vo.vdpau]" profiles and similar. The messages printed
on loading change, but other than that, everything should behave about
the same.
|
|
|
|
| |
Used in the following commit.
|
|
|
|
| |
mp_is_url("://") returned true.
|
|
|
|
|
|
|
|
|
| |
Including the "." in the returned extension was too inconvenient. I
think originally, the semantics were supposed to work like in Python,
but screw this.
Also, return NULL instead of "" on failure (which is what its only user
actually seems to expect).
|
|
|
|
| |
Signed-off-by: wm4 <wm4@nowhere>
|
|
|
|
|
|
|
|
| |
How embarrassing...
This code is inactive for all VOs other than vo_vdpau. For vo_vdpau,
this caused various issues, such as stuttering after about an hour of
running mpv; see github issue #403.
|
|
|
|
|
|
| |
Note that this will print a difference even with perfect sync, because
the code queues the frames _between_ vsync, probably for error margin
(though I don't understand why it uses the exact values chosen).
|
|
|
|
|
|
|
|
|
| |
I find this annoying. It's the reason common/version.c exists at all.
options.c did this for the user agent, which contains the version
number. Because not including version.h means you can't build the user
agent and use it in mp_default_opts anymore, do something rather awkward
in main.c to initialize the default user agent.
|
|
|
|
|
| |
Avoid global state (reload cookie file every time), actually free
loaded cookies, use mp_get_user_path() for cookie file.
|
| |
|
| |
|
|
|
|
| |
Prevents race conditions (which can happen only in theory anyway).
|
|
|
|
|
|
| |
Make it so --msglevel extends previous --msglevel uses, instead of
overwriting them. Do this by literally appending the --msglevel option
value to the previous one.
|
|
|
|
| |
Still no OSX here.
|
|
|
|
| |
Untested... no OSX here.
|
|\
| |
| |
| |
| | |
This branch changes mp_msg() so that it doesn't require global context.
The changes are pretty violent.
|
| | |
|
| | |
|
| |
| |
| |
| |
| | |
So the user can specifically mute or unmute the status line with
--msglevel. Accidentally removed with commit 5e0c4ec3.
|
| |
| |
| |
| | |
Add a separate mp_log instance for this purpose.
|
| |
| |
| |
| | |
Same for companion functions.
|
| |
| |
| |
| | |
Also get rid of MSGL_HINT and the many MSGL_DBG* levels.
|
| | |
|
| | |
|
| |
| |
| |
| | |
Apparently this should never be run anyway.
|
| |
| |
| |
| | |
Worthless anyway.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
There's a single mp_msg() in path.c, but all path lookup functions seem
to depend on it, so we get a rat-tail of stuff we have to change. This
is probably a good thing though, because we can have the path lookup
functions also access options, so we could allow overriding the default
config path, or ignore the MPV_HOME environment variable, and such
things.
Also take the chance to consistently add talloc_ctx parameters to the
path lookup functions.
Also, this change causes a big mess on configfiles.c. It's the same
issue: everything suddenly needs a (different) context argument. Make it
less wild by providing a mp_load_auto_profiles() function, which
isolates most of it to configfiles.c.
|
| |
| |
| |
| |
| |
| | |
Since mp_find_user_config_file() is going to get a context argument,
which would be annoying to do in the audio chain (actually I'm just
lazy).
|
| | |
|
| | |
|
| |
| |
| |
| | |
We also drop some slave mode stuff from stream_vcd.
|
| |
| |
| |
| |
| |
| |
| | |
The TV code pretends to be part of stream/, but it's actually demuxer
code too. The audio_in code is shared between the TV code and
stream_radio.c, so stream_radio.c needs a small hack until stream.c is
converted.
|
| | |
|
| | |
|
| |
| |
| |
| | |
Miss two mp_msg calls, because these conflict with future commits.
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| | |
This is pretty nasty, because FFmpeg/Libav is yet another library with a
global message callback. We do something with mutexes trying to get it
done, but of course we can't actually solve this problem. If more than
one library in a process use FFmpeg/Libav, only one of them will get log
messages.
|
| | |
|
| |
| |
| |
| |
| | |
And also convert a bunch of other code, especially ao_wasapi and
ao_portaudio.
|
| |
| |
| |
| |
| |
| |
| |
| | |
Always pass around mp_log contexts in the option parser code. This of
course affects all users of this API as well.
In stream.c, pass a mp_null_log, because we can't do it properly yet.
This will be fixed later.
|
| |
| |
| |
| |
| | |
Includes some semi-crappy hacks to avoid changing too much code for this
conversion (allowing NULL log argument for m_property_do()).
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Until now, there were two functions to add input sources (stuff like
stdin input, slave mode, lirc, joystick). Unify them to a single
function (mp_input_add_fd()), and make sure the associated callbacks
always have a context parameter.
Change the lirc and joystick code such that they take store their state
in a context struct (probably worthless), and use the new mp_msg
replacements (the point of this refactoring).
Additionally, get rid of the ugly USE_FD0_CMD_SELECT etc. ifdeffery in
the terminal handling code.
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
| |
| |
| |
| | |
We print these before calling abort(), which is deadly unclean anyway.
Avoids having to add log contexts.
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The mp_select_decoders() function doesn't have a log context (and I
don't want to give it one), so get rid of the mp_msg error message by
enhancing the semantics such that the syntax error is replaced by a new
feature. Now doing "--ad=something" will enable all decoders in the
"something" module, same as "--ad=something:*". Pretty useless, but gets
rid of the annoyance.
|
| | |
|
| |
| |
| |
| | |
Saves a little bit of pain.
|
| | |
|
| |
| |
| |
| |
| |
| |
| | |
Remove the nonsensical print_lock too.
Things that are called from the option validator are not converted yet,
because the option parser doesn't provide a log context yet.
|
| |
| |
| |
| | |
Same deal as with video/fmt-conversion.c.
|
| | |
|
| | |
|
| |
| |
| |
| |
| | |
Using m_property_do() is more complicated, and will have to be changed
later for mp_msg conversions.
|
| | |
|
| |
| |
| |
| |
| | |
Replacement for MSGT_IDENTIFY. Can't kill it off completely yet; certain
people would complain to me personally.
|
| |
| |
| |
| | |
Too bad.
|
| |
| |
| |
| |
| |
| | |
This is printed right before abort(), which is bad style anyway.
Converting this to mp_msg will help nobody, and passing through a mp_log
is effort.
|
| |
| |
| |
| |
| |
| | |
This ended up a little bit messy. In order to get a mp_log everywhere,
mostly make use of the fact that va_surface already references global
state anyway.
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This removes the messages printed on unknown pixel format messages.
Passing a mp_log to them would be too messy. Actually, this is a good
change, because in the past we often had trouble with these messages
printed too often (causing terminal spam etc.), and printing warnings or
error messages on the caller sides is much cleaner.
vd_lavc.c had a change earlier to print an error message if a decoder
outputs an unsupported pixel format.
|
| |
| |
| |
| | |
Adds an awkward mp_log argument for error messages.
|
| |
| |
| |
| |
| |
| | |
This requires the caller to provide a mp_log in order to see error
messages. Unfortunately we don't do this in most places, but I guess we
have to live with it.
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
lcms2 has a global message callback for error reporting. If you don't
set this, these error messages are silently thrown away. I think we
still want the error messages, so we have to do dumb stuff to avoid
clashes. This doesn't handle the case if another library in the same
process sets the message callback, but at least this should exclude
possible memory errors when running multiple instances of mpv.
|
| |
| |
| |
| | |
Will be needed for other parts (especially in gl_lcms.c).
|
|