From 21f07b9fa980ffa181a0e88865236fc0c5bac9e7 Mon Sep 17 00:00:00 2001 From: Cedric Bail Date: Thu, 31 Oct 2019 13:20:45 -0400 Subject: [PATCH] evas: deprecate evas_device_parent_set. Summary: It is unlikely that the code was working before and it was a bad idea anyway. There is no user of this API in EFL. Depends on D10490 Reviewers: zmike, raster, bu5hm4n, Hermet Reviewed By: zmike Subscribers: #reviewers, #committers Tags: #efl Maniphest Tasks: T8321 Differential Revision: https://phab.enlightenment.org/D10491 --- src/lib/evas/Evas_Common.h | 2 +- src/lib/evas/canvas/evas_device.c | 17 ++--------------- 2 files changed, 3 insertions(+), 16 deletions(-) diff --git a/src/lib/evas/Evas_Common.h b/src/lib/evas/Evas_Common.h index f68f544dd3..632260fbe0 100644 --- a/src/lib/evas/Evas_Common.h +++ b/src/lib/evas/Evas_Common.h @@ -1159,7 +1159,7 @@ EAPI const char *evas_device_description_get(const Evas_Device *dev); * * @since 1.8 */ -EAPI void evas_device_parent_set(Evas_Device *dev, Evas_Device *parent); +EAPI void evas_device_parent_set(Evas_Device *dev, Evas_Device *parent) EINA_DEPRECATED; /** * Get the parent of a device diff --git a/src/lib/evas/canvas/evas_device.c b/src/lib/evas/canvas/evas_device.c index af95619044..1652e79100 100644 --- a/src/lib/evas/canvas/evas_device.c +++ b/src/lib/evas/canvas/evas_device.c @@ -330,25 +330,12 @@ evas_device_description_get(const Evas_Device *dev) } EAPI void -evas_device_parent_set(Evas_Device *dev, Evas_Device *parent) +evas_device_parent_set(Evas_Device *dev EINA_UNUSED, Evas_Device *parent EINA_UNUSED) { // Note: This function should be deprecated. parent_set doesn't make sense // unless the parent is a seat device. Parent shouldn't be changed after // creation. - - SAFETY_CHECK(dev, EFL_INPUT_DEVICE_CLASS); - - if (parent) - { - SAFETY_CHECK(parent, EFL_INPUT_DEVICE_CLASS); - } - else if (efl_parent_get(dev)) - { - efl_ref(dev); - } - - efl_parent_set(dev, parent); - evas_event_callback_call(efl_input_device_evas_get(dev), EVAS_CALLBACK_DEVICE_CHANGED, dev); + ERR("It is not advised and possible anymore to changed the parent of an Evas_Device."); } EAPI const Evas_Device *