@bu5hm4n , shouldn't this #endif and label be after the if block below?
Just like the other methods that notify the manager.
I agree, we can calculate the length before creating the request and avoid one allocation cycle for nothing completely.
Updated by moving the variables and calculations used only in VIEWPORT_ENABLE actually inside the ifdef
Ah no, this is incorrect. The idea is to refetch all the object after the deleted one. This patch seems to disable that completely.
Update by only allocating the request if it will actually be used.
In efl we tend to declare variable at the top of the function.
Couldn't this be request_length?
Update after cedric's comments.