efl_datetime_manager: how did that ever work?
authorMarcel Hollerbach <mail@marcel-hollerbach.de>
Mon, 9 Dec 2019 18:25:18 +0000 (19:25 +0100)
committerJongmin Lee <jm105.lee@samsung.com>
Mon, 16 Dec 2019 03:26:45 +0000 (12:26 +0900)
commit110f11775b90a13bec0cced315586f3bcc710b34
treebf74113a6b55d7cb0226c60c37ae83372266ef5f
parentff087c7f0832e8501f73ed11d65889d86685d8c1
efl_datetime_manager: how did that ever work?

seriously, it sometimes might be worth at least *reading* what code
does, this was initializating a private data struct on one single global
boolean flag. How was that ever intended to work ? How could that ever
slip through review ?

This is not the only madness in these widget arround time and date, you
cannot even select hours in 24h mode, you also cannot cannot select the
0 hour, which is kind of normal for the one or another region? the
datetimemanager (which is IMO a complete misconcept) is full of FIXMEs
and API calls that are defined and never called at all. Again what is
this ? And how did that ever get into the codebase ?!

With this commit the widget *finally* can be created more than once
without exploding and erroring one.

Reviewed-by: Cedric BAIL <cedric.bail@free.fr>
Differential Revision: https://phab.enlightenment.org/D10843
src/lib/elementary/efl_datetime_manager.c
src/tests/elementary/spec/efl_test_basics.c