<div dir="ltr">Hi,<div><br></div><div>Can someone review my patch for the bug: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=70096">https://bugs.freedesktop.org/show_bug.cgi?id=70096</a></div><div><br></div><div>Regards,</div>
<div>Kiran</div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Oct 7, 2013 at 9:43 AM, Kiran Krishnappa <span dir="ltr"><<a href="mailto:kiraank@gmail.com" target="_blank">kiraank@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">PA_SAMPLE_24NE generated in pa_sndfile_read_sample_spec is not<br>
handled in pa_sndfile_readf_function. paplay used to get aborted<br>
for 24bit depth wav files<br>
---<br>
 src/pulsecore/sndfile-util.c |    3 +++<br>
 1 file changed, 3 insertions(+)<br>
<br>
diff --git a/src/pulsecore/sndfile-util.c b/src/pulsecore/sndfile-util.c<br>
index 0820ee4..46689af 100644<br>
--- a/src/pulsecore/sndfile-util.c<br>
+++ b/src/pulsecore/sndfile-util.c<br>
@@ -386,6 +386,9 @@ pa_sndfile_readf_t pa_sndfile_readf_function(const pa_sample_spec *ss) {<br>
         case PA_SAMPLE_ALAW:<br>
             return NULL;<br>
<br>
+        case PA_SAMPLE_S24NE:<br>
+            return NULL;<br>
+<br>
         default:<br>
             pa_assert_not_reached();<br>
     }<br>
<span class="HOEnZb"><font color="#888888">--<br>
1.7.9.5<br>
<br>
</font></span></blockquote></div><br></div>