aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
* msg: remove legacy stuffGravatar wm42013-12-21
|
* vo_opengl_old: remove leftoversGravatar wm42013-12-21
|
* macosx_application: replace mp_msg with stderr for semi-fatal errorGravatar wm42013-12-21
| | | | Apparently this should never be run anyway.
* cpudetect: remove mp_msg callsGravatar wm42013-12-21
| | | | Worthless anyway.
* path lookup functions: mp_msg conversionsGravatar wm42013-12-21
| | | | | | | | | | | | | | | | | 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.
* af_export: require filename argumentGravatar wm42013-12-21
| | | | | | 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).
* quvi: mp_msg conversionsGravatar wm42013-12-21
|
* charset_conv: mp_msg conversionsGravatar wm42013-12-21
|
* stream: mp_msg conversionsGravatar wm42013-12-21
| | | | We also drop some slave mode stuff from stream_vcd.
* demux: mp_msg conversionsGravatar wm42013-12-21
| | | | | | | 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.
* playlist_parser: mp_msg conversionGravatar wm42013-12-21
|
* x11: mp_msg conversion for fstype help outputGravatar wm42013-12-21
|
* encode_lavc: mp_msg conversionsGravatar wm42013-12-21
| | | | Miss two mp_msg calls, because these conflict with future commits.
* options: some more mp_msg conversionsGravatar wm42013-12-21
|
* av_log: mp_msg conversionGravatar wm42013-12-21
| | | | | | | | 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.
* m_option: add mp_log context to sub-module print_help callbackGravatar wm42013-12-21
|
* m_option: add mp_log callback to OPT_STRING_VALIDATE optionsGravatar wm42013-12-21
| | | | | And also convert a bunch of other code, especially ao_wasapi and ao_portaudio.
* m_option, m_config: mp_msg conversionsGravatar wm42013-12-21
| | | | | | | | 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.
* m_property: mp_msg conversionsGravatar wm42013-12-21
| | | | | Includes some semi-crappy hacks to avoid changing too much code for this conversion (allowing NULL log argument for m_property_do()).
* input: rework how input sources are addedGravatar wm42013-12-21
| | | | | | | | | | | | | | 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.
* m_config: add custom context to includefunc callbackGravatar wm42013-12-21
|
* sub/osd: mp_msg conversionsGravatar wm42013-12-21
|
* find_subfiles: mp_msg conversionsGravatar wm42013-12-21
|
* player/timeline: mp_msg conversionsGravatar wm42013-12-21
|
* demux: use fprintf() for printing fatal errorsGravatar wm42013-12-21
| | | | | We print these before calling abort(), which is deadly unclean anyway. Avoids having to add log contexts.
* codecs: avoid having to print error messageGravatar wm42013-12-21
| | | | | | | | | 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.
* codecs: mp_msg conversionGravatar wm42013-12-21
|
* av_common: abuse av_log to print message instead of mp_msgGravatar wm42013-12-21
| | | | Saves a little bit of pain.
* ao: some missing mp_msg conversionsGravatar wm42013-12-21
|
* ao_wasapi: mp_msg conversionsGravatar wm42013-12-21
| | | | | | | 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.
* audio/fmt-conversion.c: remove unknown audio format messagesGravatar wm42013-12-21
| | | | Same deal as with video/fmt-conversion.c.
* audio: mp_msg conversionsGravatar wm42013-12-21
|
* mixer: mp_msg conversionsGravatar wm42013-12-21
|
* lua: minor simplificationGravatar wm42013-12-21
| | | | | Using m_property_do() is more complicated, and will have to be changed later for mp_msg conversions.
* osdep/priority: drop message output when setting priorityGravatar wm42013-12-21
|
* player: use MSGL_SMODE for some slave-mode stuffGravatar wm42013-12-21
| | | | | Replacement for MSGT_IDENTIFY. Can't kill it off completely yet; certain people would complain to me personally.
* img_format: drop message about unknown pixel formatsGravatar wm42013-12-21
| | | | Too bad.
* bitmap_packer: use printf() for fatal messageGravatar wm42013-12-21
| | | | | | 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.
* vaapi: mp_msg conversionsGravatar wm42013-12-21
| | | | | | 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.
* vdpau: mp_msg conversionsGravatar wm42013-12-21
|
* video/fmt-conversion.c: remove unknown pixel format messagesGravatar wm42013-12-21
| | | | | | | | | | | 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.
* image_writer: mp_msg conversionsGravatar wm42013-12-21
| | | | Adds an awkward mp_log argument for error messages.
* sws_utils: mp_msg conversionsGravatar wm42013-12-21
| | | | | | 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.
* gl_lcms: use global lock to deal with crappy lcms2 message callbackGravatar wm42013-12-21
| | | | | | | | | 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.
* video/out: pass along global contextGravatar wm42013-12-21
| | | | Will be needed for other parts (especially in gl_lcms.c).
* video/decode: mp_msg conversionsGravatar wm42013-12-21
| | | | Doesn't cover vdpau/vaapi parts yet, because these are a bit messier.
* video/filter: mp_msg conversionsGravatar wm42013-12-21
|
* x11: don't set global error handlerGravatar wm42013-12-21
| | | | | | | | | This has similar problems as the ALSA message callback, though in theory we could use the Display handle to find the right mpv instance from the global callback. It still wouldn't work if another library happens to set the error handler at the same time. There doesn't seem much of an advantage overriding the error handler (though it used to be required), so remove it.
* ao_alsa: don't set ALSA message callbackGravatar wm42013-12-21
| | | | | | | This could output additional, potentially useful error messages. But the callback is global and not library-safe, and would require us to add additional state. Remove it, because it's obviously too much of a pain. Also, it seems ALSA prints stuff to stderr anyway.
* msg: handle vsnprintf errorsGravatar wm42013-12-20
| | | | | | | I don't know under which circumstances this can error (other than a broken format string). It seems it won't return an error code on I/O errors, so maybe broken format strings are the only case. Either way, don't continue if an error is returned.