aboutsummaryrefslogtreecommitdiffstats
path: root/src (follow)
Commit message (Collapse)AuthorAgeFilesLines
* sticker: fix a memory leakJonathan Neuschäfer2011-07-181-1/+3
|
* ape: add missing g_free in error pathJonathan Neuschäfer2011-07-181-1/+3
|
* decoder/flac: validate the sample rate when scanning the tagMax Kellermann2011-07-032-1/+5
| | | | | Don't calculate the song duration when the sample rate is 0 (division by zero crash).
* ffmpeg: workaround for semantic API change in recent ffmpeg versionsoblique2011-07-031-2/+2
|
* directory: allow directories with just playlistsMax Kellermann2011-05-092-1/+8
| | | | | Keep those when scanning for empty directories. The check in playlist_vector_is_empty() was missing.
* playlist_song: fix playlist files in base music directoryMax Kellermann2011-05-091-0/+7
| | | | | g_path_get_dirname() returns "." when there is no directory name in the given path. This patch adds a workaround for that.
* playlist_song: fix NULL pointer dereferenceMax Kellermann2011-05-091-1/+1
|
* decoder/ffmpeg: use avcodec_decode_audio3() if availableMax Kellermann2011-05-091-1/+22
| | | | | avcodec_decode_audio3() has been added in libavformat 52.25.0, and the predecessor avcodec_decode_audio2() has been deprecated.
* decoder/ffmpeg: make variables more localMax Kellermann2011-05-091-27/+16
|
* decoder/ffmpeg: don't use deprecated CODEC_TYPE_AUDIO with new lavcAnton Khirnov2011-05-091-0/+4
| | | | fixes build with lavc 53.
* decoder/ffmpeg: define fallback macro AV_VERSION_INT()Max Kellermann2011-05-091-1/+5
| | | | For ffmpeg < 0.5. Copied from libavutil 0.5.
* Merge branch 'v0.15.x' into v0.16.xMax Kellermann2011-04-121-1/+1
|\ | | | | | | | | | | | | Conflicts: NEWS configure.ac src/listen.c
| * listen: suppress "unused variable" warningMax Kellermann2011-03-231-0/+2
| |
| * command: fix return value of handle_currentsong()Max Kellermann2011-03-231-1/+1
| | | | | | | | Thanks to clang for complaining.
| * encoder/vorbis: reset the Ogg stream after flushMax Kellermann2011-03-161-0/+2
| | | | | | | | | | Without the ogg_stream_reset() call, the "e_o_s" flag never gets reset, and libogg writes EOS packets over and over.
* | decoder/flac: fix enum mismatch in flac_tell_cb()Max Kellermann2011-03-231-1/+1
| | | | | | | | Fix clang warning.
* | audio_parser: fix assertion failure in audio format mask parserMax Kellermann2011-03-231-1/+2
| | | | | | | | Use audio_format_mask_valid() to verify a mask.
* | command: print playlist load errorMax Kellermann2011-03-181-1/+1
| | | | | | | | Call print_playlist_result() instead of casting the enum implicitly.
* | output/httpd: include sys/socket.h only when building with libwrapMax Kellermann2011-03-181-1/+1
| | | | | | | | Fixes build failure on WIN32.
* | update_walk: ignore parameter "mode" on WIN32Max Kellermann2011-03-181-0/+1
| | | | | | | | Fix compiler warning.
* | audio_format, output_thread: add more audio_format_valid() assertionsMax Kellermann2011-03-164-0/+13
| |
* | encoder/vorbis: reset the Ogg stream after flushMax Kellermann2011-03-161-0/+2
| | | | | | | | | | Without the ogg_stream_reset() call, the "e_o_s" flag never gets reset, and libogg writes EOS packets over and over.
* | Merge release 0.15.16 into v0.16.xMax Kellermann2011-03-164-9/+18
|\| | | | | | | | | | | | | | | Conflicts: NEWS configure.ac src/output/jack_plugin.c src/update.c
| * output/jack: fix crash with mono playbackMax Kellermann2011-02-271-1/+1
| | | | | | | | | | | | With mono sound, jack_sample_size is smaller than frame_size (4 vs 2 bytes), and "space/jack_sample_size==0". That means mpd_jack_play() will return 0, although no error has occurred.
| * output/jack: rename variable sample_size to jack_sample_sizeMax Kellermann2011-02-251-5/+6
| |
| * Insure proper initialization of stack-allocated struct.Christopher Brannon2011-02-151-1/+4
| | | | | | | | | | | | | | | | | | | | Version 1.0.0 of the libao library added a new field to the ao_sample_format struct. It is a char * named matrix. When an ao_sample_format is allocated on the stack, this field contains garbage. The proper course is to insure that is initialized to NULL. NULL indicates that we do not want any mapping. The struct is now initialized using a static initializer, and this technique is compatible with all known versions of libao.
| * update: log all file permission problemsMax Kellermann2011-01-311-0/+6
| |
| * Fix NDEBUG testAndreas Wiese2011-01-141-1/+1
| | | | | | | | | | | | | | <stdbool.h> needs to be included unconditionally from definition of NDEBUG, since »bool« doesn't get defined otherwise. Signed-off-by: Andreas Wiese <aw-devel@meterriblecrew.net>
* | output/httpd: include sys/socket.h for AF_UNIXUlrich Spörlein2011-03-091-0/+1
| |
* | output/oss: disable 24 bit playback on FreeBSDMax Kellermann2011-02-281-0/+9
| | | | | | | | See code comment.
* | output/oss: AFMT_S24_PACKED is little-endianMax Kellermann2011-02-281-0/+12
| | | | | | | | | | | | | | | | | | According to the Solaris dsp manpage, AFMT_S24_PACKED is little-endian: http://download.oracle.com/docs/cd/E19963-01/821-1475/6nmf5baot/index.html The Minix soundcard.h header says the same.
* | general: whitespace cleanupThomas Jansen2011-02-099-29/+29
| | | | | | | | | | Remove trailing whitespace found by this command: find -name '*.[ch]' | xargs grep "[[:space:]]$"
* | output/httpd: initialize unflushed_inputThomas Jansen2011-02-091-0/+1
| | | | | | | | | | | | | | | | | | | | This fixes the following valgrind warning occuring on the first call of httpd_output_read_page: ==20124== Conditional jump or move depends on uninitialised value(s) ==20124== at 0x425E65: httpd_output_read_page (httpd_output_plugin.c:240) ==20124== by 0x426087: httpd_output_open (httpd_output_plugin.c:279) ==20124== by 0x41D862: ao_open (output_plugin.h:206) ==20124== by 0x41E133: audio_output_task (output_thread.c:590)
* | Set fadeout in gme_decoder_plugin. Due to the nature of the gme library,Tony Miller2011-02-031-0/+3
| | | | | | | | this needs to be done for the end of songs to be detected.
* | Merge branch 'v0.15.x' into v0.16.xMax Kellermann2011-01-073-2/+12
|\| | | | | | | | | | | | | Conflicts: NEWS configure.ac src/directory.h
| * encoder/lame: explicitly configure the output sample rateMax Kellermann2011-01-071-0/+7
| | | | | | | | | | | | | | | | | | When you don't explicitly set an output sample rate, liblame tries to guess an output sample rate from the input sample rate. You would think that this "guessing" consists of just setting both equal, but that is not the case. For 44.1kHz at 96kbit/s, liblame chooses 32kHz. This patch explicitly configures the output sample rate, to stop the bad guessing.
| * output/httpd: define G_LOG_DOMAIN in httpd_client.cMax Kellermann2011-01-071-0/+3
| |
| * directory: fix warning "comparison between signed and unsigned"Max Kellermann2010-12-211-2/+2
| | | | | | | | Cast the constant to dev_t, not to unsigned.
| * zeroconf-bonjour: use g_htons() instead of htons()Max Kellermann2010-12-211-1/+1
| | | | | | | | Fixes the gcc warning "implicit declaration of function 'htons'".
* | player_thread: discard empty chunks while cross-fadingMax Kellermann2011-01-071-0/+13
| | | | | | | | | | | | When a music_chunk to be crossfaded consists only of a tag, cross-fading is not possible, and led to an assertion failure. This patch just discards those, as if cross-fading was not enabled.
* | output_thread: fix double lockMax Kellermann2011-01-071-3/+0
| | | | | | | | | | | | | | | | During the whole output thread, the audio_output object is locked, and it is only unlocked while waiting for the GCond and while running a plugin method. The error handler in ao_play_chunk() attempted to lock the object again, which was code from MPD 0.15.x which should have been removed a long time ago.
* | player_thread: fix assertion failure due to early seekMax Kellermann2011-01-071-0/+4
| | | | | | | | | | | | | | Until the decoder plugin has called decoder_initialized(), the player may not submit seek commands. This however could occur with a slow decoder and a CUE file with a virtual song offset. This patch adds another check.
* | player_thread: make variables more localMax Kellermann2011-01-071-36/+19
| |
* | output/osx: fix up audio format first, then apply it to deviceMax Kellermann2011-01-071-9/+10
| | | | | | | | | | | | | | | | | | This is a MPD 0.16 regression: when playing a 24 bit file, the switch to 16 bit was made only partially, after mBytesPerPacket and mBytesPerFrame had already been applied. That means mBytesPerFrame referred to 24 bit, and mBitsPerChannel referred to 16 bits. Of course, that cannot work.
* | add void casts to suppress "result unused" warnings (clang)Max Kellermann2010-12-212-3/+3
| |
* | decoder/mad: work around build failure on SolarisAlex Viskovatoff2010-12-211-2/+2
| | | | | | | | | | | | | | | | Rename the "version" struct, because it seems to be a reserved name on Solaris: "src/decoder/mad_decoder_plugin.c", line 550: (enum) tag redeclared: version cc: acomp failed for src/decoder/mad_decoder_plugin.c
* | output/solaris: add missing parameter to open_cloexec() callAlex Viskovatoff2010-12-211-1/+1
| |
* | audio_check: fix parameter in prototypeAlex Viskovatoff2010-12-211-1/+1
| |
* | tag_id3: support multiple valuesMax Kellermann2010-12-071-10/+36
| | | | | | | | | | Loop over all frames with a specific id, and import all of them - not just the first one (index 0).
* | command: don't error when sticker list is run on song with no stickersAnton Khirnov2010-12-071-7/+3
| | | | | | | | | | | | this is inconsistent with other commands (e.g. find) and seems wrong -- a song with no stickers attached is a perfectly valid state and an empty list of stickers is also perfectly valid.