[polypaudio-commits] r778 - in /trunk/src/modules: module-x11-bell.c module-x11-publish.c

[svnmailer-noreply at 0pointer.de]

Author: lennart
Date: Sat Apr 22 23:51:30 2006
New Revision: 778

URL: http://0pointer.de/cgi-bin/viewcvs.cgi?rev=778&root=polypaudio&view=rev
Log:
modify x11 modules to not cache the Display variable since pa_x11wrap_get_display() is now used as notification that the x11 output buffer needs flushing

Modified:
    trunk/src/modules/module-x11-bell.c
    trunk/src/modules/module-x11-publish.c

Modified: trunk/src/modules/module-x11-bell.c
URL: http://0pointer.de/cgi-bin/viewcvs.cgi/trunk/src/modules/module-x11-bell.c?rev=778&root=polypaudio&r1=777&r2=778&view=diff
==============================================================================
--- trunk/src/modules/module-x11-bell.c (original)
+++ trunk/src/modules/module-x11-bell.c Sat Apr 22 23:51:30 2006
@@ -52,7 +52,6 @@
     int xkb_event_base;
     char *sink_name;
     char *scache_item;
-    Display *display;
 
     pa_x11_wrapper *x11_wrapper;
     pa_x11_client *x11_client;
@@ -75,7 +74,7 @@
         return -1;
     }
 
-    pa_scache_play_item(u->core, u->scache_item, s, pa_cvolume_set(&cv, PA_CHANNELS_MAX, percent*PA_VOLUME_NORM/100));
+    pa_scache_play_item(u->core, u->scache_item, s, pa_cvolume_set(&cv, PA_CHANNELS_MAX, (percent*PA_VOLUME_NORM)/100));
     return 0;
 }
 
@@ -118,8 +117,6 @@
     if (!(u->x11_wrapper = pa_x11_wrapper_get(c, pa_modargs_get_value(ma, "display", NULL)))) 
         goto fail;
 
-    u->display = pa_x11_wrapper_get_display(u->x11_wrapper);
-    
     major = XkbMajorVersion;
     minor = XkbMinorVersion;
     
@@ -132,15 +129,15 @@
     minor = XkbMinorVersion;
 
 
-    if (!XkbQueryExtension(u->display, NULL, &u->xkb_event_base, NULL, &major, &minor)) {
+    if (!XkbQueryExtension(pa_x11_wrapper_get_display(u->x11_wrapper), NULL, &u->xkb_event_base, NULL, &major, &minor)) {
         pa_log(__FILE__": XkbQueryExtension() failed");
         goto fail;
     }
 
-    XkbSelectEvents(u->display, XkbUseCoreKbd, XkbBellNotifyMask, XkbBellNotifyMask);
+    XkbSelectEvents(pa_x11_wrapper_get_display(u->x11_wrapper), XkbUseCoreKbd, XkbBellNotifyMask, XkbBellNotifyMask);
     auto_ctrls = auto_values = XkbAudibleBellMask;
-    XkbSetAutoResetControls(u->display, XkbAudibleBellMask, &auto_ctrls, &auto_values);
-    XkbChangeEnabledControls(u->display, XkbUseCoreKbd, XkbAudibleBellMask, 0);
+    XkbSetAutoResetControls(pa_x11_wrapper_get_display(u->x11_wrapper), XkbAudibleBellMask, &auto_ctrls, &auto_values);
+    XkbChangeEnabledControls(pa_x11_wrapper_get_display(u->x11_wrapper), XkbUseCoreKbd, XkbAudibleBellMask, 0);
 
     u->x11_client = pa_x11_client_new(u->x11_wrapper, x11_event_callback, u);
     

Modified: trunk/src/modules/module-x11-publish.c
URL: http://0pointer.de/cgi-bin/viewcvs.cgi/trunk/src/modules/module-x11-publish.c?rev=778&root=polypaudio&r1=777&r2=778&view=diff
==============================================================================
--- trunk/src/modules/module-x11-publish.c (original)
+++ trunk/src/modules/module-x11-publish.c Sat Apr 22 23:51:30 2006
@@ -66,7 +66,6 @@
 struct userdata {
     pa_core *core;
     pa_x11_wrapper *x11_wrapper;
-    Display *display;
     char *id;
     uint8_t auth_cookie[PA_NATIVE_COOKIE_LENGTH];
     int auth_cookie_in_property;
@@ -123,28 +122,26 @@
     if (!(u->x11_wrapper = pa_x11_wrapper_get(c, pa_modargs_get_value(ma, "display", NULL)))) 
         goto fail;
 
-    u->display = pa_x11_wrapper_get_display(u->x11_wrapper);
-
     if (!(l = pa_property_get(c, PA_NATIVE_SERVER_PROPERTY_NAME)))
         goto fail;
 
     s = pa_strlist_tostring(l);
-    pa_x11_set_prop(u->display, "POLYP_SERVER", s);
+    pa_x11_set_prop(pa_x11_wrapper_get_display(u->x11_wrapper), "POLYP_SERVER", s);
     pa_xfree(s);
     
     if (!pa_get_fqdn(hn, sizeof(hn)) || !pa_get_user_name(un, sizeof(un)))
         goto fail;
     
     u->id = pa_sprintf_malloc("%s@%s/%u", un, hn, (unsigned) getpid());
-    pa_x11_set_prop(u->display, "POLYP_ID", u->id);
+    pa_x11_set_prop(pa_x11_wrapper_get_display(u->x11_wrapper), "POLYP_ID", u->id);
 
     if ((t = pa_modargs_get_value(ma, "source", NULL)))
-        pa_x11_set_prop(u->display, "POLYP_SOURCE", t);
+        pa_x11_set_prop(pa_x11_wrapper_get_display(u->x11_wrapper), "POLYP_SOURCE", t);
 
     if ((t = pa_modargs_get_value(ma, "sink", NULL)))
-        pa_x11_set_prop(u->display, "POLYP_SINK", t);
+        pa_x11_set_prop(pa_x11_wrapper_get_display(u->x11_wrapper), "POLYP_SINK", t);
 
-    pa_x11_set_prop(u->display, "POLYP_COOKIE", pa_hexstr(u->auth_cookie, sizeof(u->auth_cookie), hx, sizeof(hx)));
+    pa_x11_set_prop(pa_x11_wrapper_get_display(u->x11_wrapper), "POLYP_COOKIE", pa_hexstr(u->auth_cookie, sizeof(u->auth_cookie), hx, sizeof(hx)));
     
     pa_modargs_free(ma);
     return 0;
@@ -168,15 +165,15 @@
         char t[256];
 
         /* Yes, here is a race condition */
-        if (!pa_x11_get_prop(u->display, "POLYP_ID", t, sizeof(t)) || strcmp(t, u->id))
+        if (!pa_x11_get_prop(pa_x11_wrapper_get_display(u->x11_wrapper), "POLYP_ID", t, sizeof(t)) || strcmp(t, u->id))
             pa_log_warn(__FILE__": Polypaudio information vanished from X11!");
         else {
-            pa_x11_del_prop(u->display, "POLYP_ID");
-            pa_x11_del_prop(u->display, "POLYP_SERVER");
-            pa_x11_del_prop(u->display, "POLYP_SINK");
-            pa_x11_del_prop(u->display, "POLYP_SOURCE");
-            pa_x11_del_prop(u->display, "POLYP_COOKIE");
-            XSync(u->display, False);
+            pa_x11_del_prop(pa_x11_wrapper_get_display(u->x11_wrapper), "POLYP_ID");
+            pa_x11_del_prop(pa_x11_wrapper_get_display(u->x11_wrapper), "POLYP_SERVER");
+            pa_x11_del_prop(pa_x11_wrapper_get_display(u->x11_wrapper), "POLYP_SINK");
+            pa_x11_del_prop(pa_x11_wrapper_get_display(u->x11_wrapper), "POLYP_SOURCE");
+            pa_x11_del_prop(pa_x11_wrapper_get_display(u->x11_wrapper), "POLYP_COOKIE");
+            XSync(pa_x11_wrapper_get_display(u->x11_wrapper), False);
         }
     }