On 09/18/2013 02:51 PM, David Henningsson wrote: > On 09/18/2013 10:22 AM, David Henningsson wrote: >> Hi lynxis, >> >> I just got this compiler warning: >> >> ../../src/modules/module-tunnel-source-new.c: In function >> 'read_new_samples': >> ../../src/modules/module-tunnel-source-new.c:143:16: warning: >> declaration of 'read' shadows a global declaration [-Wshadow] >> >> Would you mind renaming the 'read' variable to something else for clarity? >> > > And also this linker error: > > modules/.libs/module_tunnel_sink_la-module-tunnel.o: In function > `module_tunnel_sink_LTX_pa__init': > /home/david/gsrc/pulseaudio/build/src/../../src/modules/module-tunnel.c:2235: > undefined reference to `xcb_disconnect' > /home/david/gsrc/pulseaudio/build/src/../../src/modules/module-tunnel.c:1987: > undefined reference to `xcb_connect' > /home/david/gsrc/pulseaudio/build/src/../../src/modules/module-tunnel.c:1990: > undefined reference to `xcb_connection_has_error' > /home/david/gsrc/pulseaudio/build/src/../../src/modules/module-tunnel.c:2253: > undefined reference to `xcb_disconnect' > /home/david/gsrc/pulseaudio/build/src/../../src/modules/module-tunnel.c:1992: > undefined reference to `xcb_disconnect' Sorry, this wasn't actually your fault. Just fixed this one in commit d909860d8d5f47cb0982797daba7762cd7db3a72 Author: David Henningsson <david.henningsson at canonical.com> Date: Wed Sep 18 15:07:37 2013 +0200 module-tunnel: Fixup broken build -- David Henningsson, Canonical Ltd. https://launchpad.net/~diwic