This particular patch, yes. Was just something I caught while working in
the branch and it just stayed there.
On 03/29/2018 10:48 AM, Mike Blumenkrantz wrote:
This seems like it could be merged at any time instead of waiting in a
branch?
On Thu, Mar 29, 2018 at 10:01 AM Christopher Michael <[email protected]>
wrote:
devilhorns pushed a commit to branch feature/wayland/multi-output.
http://git.enlightenment.org/core/enlightenment.git/commit/?id=cac8cce54242afe26191532ade55a0ef488eb34d
commit cac8cce54242afe26191532ade55a0ef488eb34d
Author: Chris Michael <[email protected]>
Date: Thu Mar 29 09:53:05 2018 -0400
conf_randr: Use eina_stringshare_replace
Instead of deleteing a stringshare and re-adding it, just use
eina_stringshare_replace
Signed-off-by: Chris Michael <[email protected]>
---
src/modules/conf_randr/e_int_config_randr2.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/src/modules/conf_randr/e_int_config_randr2.c
b/src/modules/conf_randr/e_int_config_randr2.c
index 1530c214a..e30cf08b8 100644
--- a/src/modules/conf_randr/e_int_config_randr2.c
+++ b/src/modules/conf_randr/e_int_config_randr2.c
@@ -571,12 +571,11 @@ _cb_rel_to_set(void *data, Evas_Object *obj, void
*event)
E_Randr2_Screen *s = _screen_config_id_find(cs2->id);
if (s)
{
- printf("SEt to %p [%s]\n", cs, cs->id);
+ printf("Set to %p [%s]\n", cs, cs->id);
printf("find s = %p\n", s);
printf("s id = %s\n", s->id);
elm_object_text_set(obj, s->info.name);
- eina_stringshare_del(cs->rel_to);
- cs->rel_to = eina_stringshare_add(s->id);
+ eina_stringshare_replace(&cs->rel_to, s->id);
}
}
e_config_dialog_changed_set(cfdata->cfd, EINA_TRUE);
--
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
enlightenment-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
