Page MenuHomePhabricator

edje: Use EINA_LIST_FREE instead of hand coded variant
ClosedPublic

Authored by ManMower on Jan 18 2019, 11:20 AM.

Details

Summary

Minor simplification, should be no functional change.

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.
ManMower created this revision.Jan 18 2019, 11:20 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/

ManMower requested review of this revision.Jan 18 2019, 11:20 AM
zmike accepted this revision.Jan 18 2019, 11:21 AM
This revision is now accepted and ready to land.Jan 18 2019, 11:21 AM
cedric requested changes to this revision.Jan 18 2019, 11:33 AM
cedric added inline comments.
src/lib/edje/edje_var.c
243–244

You can move that declaration in the upper scope and avoid this unnecessary temporary variable.

This revision now requires changes to proceed.Jan 18 2019, 11:33 AM
ManMower updated this revision to Diff 18618.Jan 18 2019, 11:56 AM

Remove temp variable

cedric accepted this revision.Jan 18 2019, 12:09 PM
This revision is now accepted and ready to land.Jan 18 2019, 12:09 PM
This revision was automatically updated to reflect the committed changes.