tree 9dcb95992f31709a6b6ef0d9d5e8e7e727f754d7
parent 0e4e570caee9f857d13e35a9f4c78d52343872d7
author Derek Foreman <derekf@osg.samsung.com> 1501097758 -0500
committer Derek Foreman <derekf@osg.samsung.com> 1501174806 -0500

libweston-desktop/xdg-shell: Properly properly handle ack_configure

commit 749637a8a306588964885fe6b25fda6087a84ccd
introduced this feature, but the break is outside of any conditional
so only the first item in the list is ever tested.

If a client skips a few configures and then acks the most recent
it's still operating within spec, so the break should only occur
when a match is found.

This version also adds a break after we miss the target, as a tiny
optimization (the list will be cleaned up on disconnect anyway),
as it makes the code no more difficult to read or maintain.

Signed-off-by: Derek Foreman <derekf@osg.samsung.com>
Reviewed-by: Quentin Glidic <sardemff7+git@sardemff7.net>