client: Fix double locking bug
authorAnder Conselvan de Oliveira <ander.conselvan.de.oliveira@intel.com>
Thu, 11 Oct 2012 11:55:59 +0000 (14:55 +0300)
committerKristian Høgsberg <krh@bitplanet.net>
Thu, 11 Oct 2012 13:59:40 +0000 (09:59 -0400)
commitff4afd6c0c0a1dd5a49d7078c52a6a037089341a
treecc2d496c295d5321df18bfe9a9b5ae71d6b3fc7d
parenteb5fae3226fa6d0c542fda54a35e282466167e96
client: Fix double locking bug

The function wl_proxy_create_for_id() would try to acquire the display
lock, but the only call path leading to it would call it with the lock
already acquired.

This patch removes the attempt to acquire the lock and makes the
function static. It was exported before because client had to create
proxy's manually when the server sent a new object id, but since commit
9de9e39f [1] this is no longer necessary.

[1] commit 9de9e39f87adfce1ea9755e394928756254c0ba2
    Author: Kristian Høgsberg <krh@bitplanet.net>
    Date:   Thu Jun 28 22:01:58 2012 -0400

        Allocate client proxy automatically for new objects

v2: Change the right function. Previous patch changed wl_proxy_create()
    instead of wl_proxy_create_for_id().
src/wayland-client.c
src/wayland-client.h