This looks like a good start.

@bu5hm4n, @ProhtMeyhet are you happy with the rebased version now?

Thanks for taking the time and making this usable! :-)

Let's take this patch as one cleanup and if we want to tackle the other patterns we can do as follow up patches.

Man, this sneaked in. Glad you found it.

git phab as trouble applying this. Could you rebase?

We have a basic exactness running in CI production for a while now.

the loop exists you mean, right? Might be an idea. I have none in my head right now. Was mostly curious if you had one.

For the record: I added a missing if: type = cron here before landing this

Glad that we can leave all the argument, env and file handling to python for this. :-)

ping review

ping review

ping review

Fixed with D11607 and D11612

Thanks for the updates!

Ah, now I understand what these sel1 and sel2 changes are doing in here. They are for the uninitialized variable reports.

For the commit message we prefer subject line prefixes. Given this is a hugfe project it helps to understand where it applies to.
For Coverity fixes it is also good style to give the IDs in the commit message so people can refer to them later on. e.g.

I have not much confidence in myself when it comes to eo. The re-flow of the code without is_obj looks correct to me, though.

Hmm, again wondering how you measure this.

This looks fine to me, but I wonder how are you measuring the cache hits and misses?

Duplicate from something we already landed?

Duplicate from something we already landed?

Duplicate from something we already landed?

Duplicate from something we already landed?

Thanks!

Passing CI testing: https://travis-ci.org/github/Enlightenment/efl/jobs/666869511

https://travis-ci.org/github/Enlightenment/efl/builds/666839615

We have a very similar construct in player.c line 1086ff it would be good to rool the change for it in here as well.

For ppc64le its working fine with these two patches: https://travis-ci.org/github/Enlightenment/efl/builds/666437724