[pulseaudio-commits] r1768 - in /branches/lennart/src: modules/module-alsa-sink.c modules/module-alsa-source.c pulsecore/protocol-native.c pulsecore/rtpoll.c pulsecore/sound-file.c tests/interpol-test.c utils/paplay.c

svnmailer-noreply at 0pointer.de svnmailer-noreply at 0pointer.de
Tue Sep 4 13:01:20 PDT 2007


Author: lennart
Date: Tue Sep  4 22:01:19 2007
New Revision: 1768

URL: http://0pointer.de/cgi-bin/viewcvs.cgi?rev=1768&root=pulseaudio&view=rev
Log:
fix a couple of compiler warnings

Modified:
    branches/lennart/src/modules/module-alsa-sink.c
    branches/lennart/src/modules/module-alsa-source.c
    branches/lennart/src/pulsecore/protocol-native.c
    branches/lennart/src/pulsecore/rtpoll.c
    branches/lennart/src/pulsecore/sound-file.c
    branches/lennart/src/tests/interpol-test.c
    branches/lennart/src/utils/paplay.c

Modified: branches/lennart/src/modules/module-alsa-sink.c
URL: http://0pointer.de/cgi-bin/viewcvs.cgi/branches/lennart/src/modules/module-alsa-sink.c?rev=1768&root=pulseaudio&r1=1767&r2=1768&view=diff
==============================================================================
--- branches/lennart/src/modules/module-alsa-sink.c (original)
+++ branches/lennart/src/modules/module-alsa-sink.c Tue Sep  4 22:01:19 2007
@@ -135,7 +135,7 @@
             if (err == -EAGAIN)
                 return work_done;
             
-            pa_log("snd_pcm_avail_update: %s", snd_strerror(n));
+            pa_log("snd_pcm_avail_update: %s", snd_strerror(err));
             return -1;
         }
 
@@ -225,7 +225,7 @@
         int err;
                 
         if ((err = snd_pcm_status(u->pcm_handle, status)) < 0) {
-            pa_log("Failed to query DSP status data: %s", snd_strerror(t));
+            pa_log("Failed to query DSP status data: %s", snd_strerror(err));
             return -1;
         }
 

Modified: branches/lennart/src/modules/module-alsa-source.c
URL: http://0pointer.de/cgi-bin/viewcvs.cgi/branches/lennart/src/modules/module-alsa-source.c?rev=1768&root=pulseaudio&r1=1767&r2=1768&view=diff
==============================================================================
--- branches/lennart/src/modules/module-alsa-source.c (original)
+++ branches/lennart/src/modules/module-alsa-source.c Tue Sep  4 22:01:19 2007
@@ -132,7 +132,7 @@
             if (err == -EAGAIN)
                 return work_done;
             
-            pa_log("snd_pcm_avail_update: %s", snd_strerror(n));
+            pa_log("snd_pcm_avail_update: %s", snd_strerror(err));
             return -1;
         }
 
@@ -216,7 +216,7 @@
         pa_memchunk chunk;
         
         if ((err = snd_pcm_status(u->pcm_handle, status)) < 0) {
-            pa_log("Failed to query DSP status data: %s", snd_strerror(t));
+            pa_log("Failed to query DSP status data: %s", snd_strerror(err));
             return -1;
         }
 

Modified: branches/lennart/src/pulsecore/protocol-native.c
URL: http://0pointer.de/cgi-bin/viewcvs.cgi/branches/lennart/src/pulsecore/protocol-native.c?rev=1768&root=pulseaudio&r1=1767&r2=1768&view=diff
==============================================================================
--- branches/lennart/src/pulsecore/protocol-native.c (original)
+++ branches/lennart/src/pulsecore/protocol-native.c Tue Sep  4 22:01:19 2007
@@ -439,8 +439,8 @@
         const pa_sample_spec *ss,
         const pa_channel_map *map,
         const char *name,
-        size_t *maxlength,
-        size_t fragment_size,
+        uint32_t *maxlength,
+        uint32_t fragment_size,
         int corked) {
 
     record_stream *s;

Modified: branches/lennart/src/pulsecore/rtpoll.c
URL: http://0pointer.de/cgi-bin/viewcvs.cgi/branches/lennart/src/pulsecore/rtpoll.c?rev=1768&root=pulseaudio&r1=1767&r2=1768&view=diff
==============================================================================
--- branches/lennart/src/pulsecore/rtpoll.c (original)
+++ branches/lennart/src/pulsecore/rtpoll.c Tue Sep  4 22:01:19 2007
@@ -383,7 +383,8 @@
         }
     }
 
-    errno = saved_errno;
+    if (r < 0)
+        errno = saved_errno;
 
     return r;
 }

Modified: branches/lennart/src/pulsecore/sound-file.c
URL: http://0pointer.de/cgi-bin/viewcvs.cgi/branches/lennart/src/pulsecore/sound-file.c?rev=1768&root=pulseaudio&r1=1767&r2=1768&view=diff
==============================================================================
--- branches/lennart/src/pulsecore/sound-file.c (original)
+++ branches/lennart/src/pulsecore/sound-file.c Tue Sep  4 22:01:19 2007
@@ -126,7 +126,7 @@
     ptr = pa_memblock_acquire(chunk->memblock);
 
     if ((readf_function && readf_function(sf, ptr, sfinfo.frames) != sfinfo.frames) ||
-        (!readf_function && sf_read_raw(sf, ptr, l) != l)) {
+        (!readf_function && sf_read_raw(sf, ptr, l) != (sf_count_t) l)) {
         pa_log("Premature file end");
         goto finish;
     }

Modified: branches/lennart/src/tests/interpol-test.c
URL: http://0pointer.de/cgi-bin/viewcvs.cgi/branches/lennart/src/tests/interpol-test.c?rev=1768&root=pulseaudio&r1=1767&r2=1768&view=diff
==============================================================================
--- branches/lennart/src/tests/interpol-test.c (original)
+++ branches/lennart/src/tests/interpol-test.c Tue Sep  4 22:01:19 2007
@@ -137,7 +137,7 @@
             pa_gettimeofday(&now);
 
             rtc = pa_timeval_diff(&now, &start);
-            printf("%i\t%llu\t%llu\t%llu\t%llu\t%u\n", k, rtc, t, rtc-old_rtc, t-old_t, changed);
+            printf("%i\t%llu\t%llu\t%llu\t%llu\t%u\n", k, (unsigned long long) rtc, (unsigned long long) t, (unsigned long long) (rtc-old_rtc), (unsigned long long) (t-old_t), changed);
             old_t = t;
             old_rtc = rtc;
         }

Modified: branches/lennart/src/utils/paplay.c
URL: http://0pointer.de/cgi-bin/viewcvs.cgi/branches/lennart/src/utils/paplay.c?rev=1768&root=pulseaudio&r1=1767&r2=1768&view=diff
==============================================================================
--- branches/lennart/src/utils/paplay.c (original)
+++ branches/lennart/src/utils/paplay.c Tue Sep  4 22:01:19 2007
@@ -123,7 +123,7 @@
     else
         pa_xfree(data);
 
-    if (bytes < length) {
+    if (bytes < (sf_count_t) length) {
         sf_close(sndfile);
         sndfile = NULL;
         pa_operation_unref(pa_stream_drain(s, stream_drain_complete, NULL));




More information about the pulseaudio-commits mailing list