Bug 80119 - dead/unreachable code at line 643 & 648 of simple-egl.c
Summary: dead/unreachable code at line 643 & 648 of simple-egl.c
Status: RESOLVED INVALID
Alias: None
Product: Wayland
Classification: Unclassified
Component: weston (show other bugs)
Version: 1.4.0
Hardware: ARM Linux (All)
: medium normal
Assignee: Wayland bug list
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-06-17 07:04 UTC by Rohit Nandan
Modified: 2018-06-04 09:25 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments

Description Rohit Nandan 2014-06-17 07:04:21 UTC
I have double checked seat_handle_capabilities function used in several of wayland clients which never calls wl_pointer_destroy/wl_keyboard_destroy/wl_touch_destroy with every combination possible. Though touch capability is not present it still call wl_seat_get_touch.


Thanks.
Comment 1 Pekka Paalanen 2014-09-10 12:42:55 UTC
Sorry, what is the problem here?

Whenever a wl_seat changes capabilities, e.g. a the first keyboard gets hot-plugged, or the last pointer device gets hot-unplugged, wl_seat sends a capability notification with the new feature bits appropriately set or cleared. Clients should destroy e.g. wl_pointer if the pointer capability disappears.

To me it looks like the code does exactly that.

But it *is* missing calls like wl_pointer_release, but I am unsure if that is the bug you are reporting here.
Comment 2 Daniel Stone 2018-06-04 09:25:18 UTC
Not sure what the actual bug is here.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.