| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
| |
Without the ogg_stream_reset() call, the "e_o_s" flag never gets
reset, and libogg writes EOS packets over and over.
|
|\
| |
| |
| |
| |
| |
| |
| | |
Conflicts:
NEWS
configure.ac
src/output/jack_plugin.c
src/update.c
|
| | |
|
| |
| |
| |
| |
| |
| | |
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.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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.
|
| | |
|
| | |
|
| |
| |
| |
| |
| |
| |
| | |
This makes FreeBSD detect libogg correctly. The '==' operator is an
undocumented GNU extension to test(1) and cannot be relied upon to
exist and do the right thing. POSIX mandates string comparisons to be
done using "test foo = bar".
|
| |
| |
| |
| | |
See code comment.
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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)
|
| |
| |
| |
| | |
this needs to be done for the end of songs to be detected.
|
| | |
|
| |
| |
| |
| |
| | |
When the configure options were moved around for 0.16, the order was
changed, and the Tremor check broke.
|
| | |
|
| | |
|
| |
| |
| |
| |
| | |
A bit of automake magic (see info automake "Per-Object Flags").
Compile-tested.
|
|\|
| |
| |
| |
| |
| |
| | |
Conflicts:
NEWS
configure.ac
src/directory.h
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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.
|
| | |
|
| | |
|
| |
| |
| |
| |
| |
| | |
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.
|
| |
| |
| |
| |
| |
| |
| |
| | |
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.
|
| |
| |
| |
| |
| |
| |
| | |
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.
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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.
|
| |
| |
| |
| | |
It is known to crash instantly.
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| | |
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
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
| |
| |
| |
| | |
Loop over all frames with a specific id, and import all of them - not
just the first one (index 0).
|
| |
| |
| |
| | |
One APE tag may contain more than one value, separated by null bytes.
|
| |
| |
| |
| | |
Based on the APE reader.
|
|\|
| |
| |
| |
| |
| | |
Conflicts:
NEWS
configure.ac
|
| | |
|
| |
| |
| |
| |
| |
| | |
After popular demand, I've switched the order of "artist" and "title"
in the stream title. There is no standard, and there is no reliable
way to parse those from the stream title.
|
| |
| |
| |
| |
| | |
The assertion added in MPD 0.15.14 was too much, it failed when the
MIME type of a stream was NULL.
|
| | |
|
|\|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Conflicts:
NEWS
configure.ac
src/decoder_control.c
src/decoder_control.h
src/input/rewind_input_plugin.c
src/output_control.c
src/output_thread.c
src/player_thread.c
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
When one song is played twice, and the decoder is working on the
second "instance", but the first should be seeked, the check in
player_seek_decoder() may assume that it can reuse the decoder without
exchanging pipes. The last thing was the mistake: the pipe pointer
was different, which led to an assertion failure. This patch adds
another check which exchanges the player pipe.
|
| |
| |
| |
| |
| | |
Duplicate the "mime" attribute of the inner input_stream object,
instead of copying the pointer.
|
| |
| |
| |
| | |
Same as ".m4a".
|
| |
| |
| |
| |
| |
| | |
Change the assertion on "fail_timer==NULL" in OPEN to a runtime check.
This assertion crashed when the output thread failed while the player
thread was calling audio_output_open().
|
| | |
|
| | |
|
| |
| |
| |
| |
| | |
This makes the plugin more responsive to control commands, because it
will listen to control events while waiting.
|