Page MenuHomePhabricator

eo: do not over compute the hash when propagating events.

Authored by cedric on Nov 13 2019, 4:08 PM.



This should slightly improve speed in theory. In practice, I have not seen
a benchmark which would budge by 5%, so I am not sure it improve speed that

Depends on D10659

Diff Detail

rEFL core/efl
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.
cedric created this revision.Nov 13 2019, 4:08 PM
cedric requested review of this revision.Nov 13 2019, 4:08 PM
q66 requested changes to this revision.Nov 13 2019, 6:50 PM

Still makes edje_cc hang, after introduction of 7f53d9158395504151e1ff3dcae715a799d913a8.

This revision now requires changes to proceed.Nov 13 2019, 6:50 PM
q66 added a comment.Nov 13 2019, 7:24 PM

Well, I fixed the root cause: 9ca573f40f1065cc717c0c5aabb787671bab852b

Feel free to apply your changes or whatever, shouldn't matter much. For D10659 I'd keep it as an int tbh, for this one, dunno how much value there is, same for D10658.

cedric updated this revision to Diff 26899.Nov 14 2019, 9:33 AM
cedric edited the summary of this revision. (Show Details)


bu5hm4n added inline comments.Nov 17 2019, 3:02 AM

I think in this block you also do not have to update the hash ?

bu5hm4n accepted this revision.Nov 17 2019, 3:06 AM
bu5hm4n added inline comments.

NVM, we would need more logic then in the callback_call case ... lets look if we really need this later on.

This revision was not accepted when it landed; it landed in state Needs Review.Nov 17 2019, 3:12 AM
Closed by commit rEFL75301b3a1abf: eo: do not over compute the hash when propagating events. (authored by cedric, committed by Marcel Hollerbach <>). · Explain Why
This revision was automatically updated to reflect the committed changes.