Re: [Musicpd-dev-team] [PATCH] Add support for DSF files to DSDIFF decoder - v3

2012-04-23 Thread Jurgen Kramer
On Mon, 2012-04-23 at 19:09 +0200, Max Kellermann wrote: On 2012/04/21 13:29, Jurgen Kramer gtmkra...@xs4all.nl wrote: - Adhered to max 80 colom width This is not doxygen syntax, and is barely readable. Please use doxygen syntax for documenting struct attributes. - Fixed indents

Re: [Musicpd-dev-team] [PATCH] Add support for DSF files to DSDIFF decoder - v3

2012-04-23 Thread Max Kellermann
On 2012/04/21 13:29, Jurgen Kramer gtmkra...@xs4all.nl wrote: - Adhered to max 80 colom width This is not doxygen syntax, and is barely readable. Please use doxygen syntax for documenting struct attributes. - Fixed indents (hopefully correctly) What did you fix? Indentation

Re: [Musicpd-dev-team] [PATCH/RFC] Fix DSD-over-USB playback

2012-04-23 Thread Max Kellermann
On 2012/04/06 12:52, Jurgen Kramer gtmkra...@xs4all.nl wrote: The attached patch fixes playback for DSD-over-USB. Without this patch MPD uses a frame size of 2 (stereo files) while the actual needed frame size for the used output sample format is 8 (sample formats S24_P32 and S32). Maybe

Re: [Musicpd-dev-team] [PATCH/RFC] Fix DSD-over-USB playback

2012-04-23 Thread adelias
On 23/4/2012 10:56 μμ, Max Kellermann wrote: On 2012/04/06 12:52, Jurgen Kramergtmkra...@xs4all.nl wrote: The attached patch fixes playback for DSD-over-USB. Without this patch MPD uses a frame size of 2 (stereo files) while the actual needed frame size for the used output sample format is 8