Page MenuHomePhabricator

edje: migrate Eo API to use Eo callback instead of opaque type.
ClosedPublic

Authored by cedric on Sat, Feb 2, 8:27 AM.

Diff Detail

Repository
rEFL core/efl
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
cedric created this revision.Sat, Feb 2, 8:27 AM
cedric requested review of this revision.Sat, Feb 2, 8:27 AM
zmike requested changes to this revision.Mon, Feb 4, 10:28 AM

warnings

lib/edje/edje_program.c: In function ‘_edje_emit_cb’:
lib/edje/edje_program.c:1683:9: warning: implicit declaration of function ‘_edje_signal_callback_matches_unref’; did you mean ‘_edje_signal_callback_patterns_unref’? [-Wimplicit-function-declaration]
         _edje_signal_callback_matches_unref(m);
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
         _edje_signal_callback_patterns_unref

lib/edje/edje_signal.c: In function ‘_edje_signal_match_key_cmp’:
lib/edje/edje_signal.c:28:66: warning: pointer to a function used in subtraction [-Wpointer-arith]
             a->free_cb[i] != b->free_cb[i]) return a->free_cb[i] - b->free_cb[i];
                                                                  ^
This revision now requires changes to proceed.Mon, Feb 4, 10:28 AM
zmike added a comment.Mon, Feb 4, 10:32 AM

Seems okay other than the warnings, so I'll r+ if you fix those.

cedric updated this revision to Diff 19205.Tue, Feb 5, 10:47 PM

Rebase and fix reported warning.

zmike accepted this revision.Wed, Feb 6, 10:48 AM
This revision is now accepted and ready to land.Wed, Feb 6, 10:48 AM
This revision was automatically updated to reflect the committed changes.