Page MenuHomePhabricator

elm_entry: add null check logic on elm_entry_text_set API
ClosedPublic

Authored by bowonryu on Aug 29 2019, 12:16 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.
bowonryu created this revision.Aug 29 2019, 12:16 AM

It seems that this patch has no reviewers specified. If you are unsure who can review your patch, please check this wiki page and see if anyone can be added: https://phab.enlightenment.org/w/maintainers_reviewers/

bowonryu requested review of this revision.Aug 29 2019, 12:16 AM
zmike requested changes to this revision.Aug 29 2019, 11:35 AM
zmike added a subscriber: zmike.

Can you just change this entire conditional to eina_streq so it becomes more readable?

This revision now requires changes to proceed.Aug 29 2019, 11:35 AM
bowonryu updated this revision to Diff 24635.Aug 29 2019, 11:12 PM

changed entire conditions to eina_streq

zmike accepted this revision.Aug 30 2019, 5:43 AM

Thanks!

This revision is now accepted and ready to land.Aug 30 2019, 5:43 AM
This revision was automatically updated to reflect the committed changes.