Hello.

On 04/18/2018 04:13 AM, YEONGJONG LEE wrote:
> Hello,
>
> I'm sure this is right.
> See https://phab.enlightenment.org/D5953

OK, I can see now what happened.

You did actually do everything correct. Having two patches for two different 
issues you found.
Resulting in D5952 and D5953.

SangHyeon, please do not ask to have such things merged together in one patch.
These have been two different issues (free call and const attribute) and they 
should not be
lumped together just because they are in the same file.

We want atomic patches in this project. One issue handled per patch.

regards
Stefan Schmidt

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to