As I said in the ticket, we can get back and discuss if we need review
for things like this.
But i have to say, even reviewing those tiny things can be good. I can
have weird things in mind for big commits or small commits, so reviewing
all of them is a good idea IMO, Independent from their size. The only
difference is, this here is very easy and fast to review, while a bigger
commit takes more time.
Further more, i don't really believe that this is that much more work,
fetching a stack of commits can be easily done via git phab apply. It
actually takes about 5 sec. I think everyone could have those 5 seconds
to make sure, we dont land "upis" in master ... :)
Greetings,
buh5m4n
On 06/25/2018 11:01 AM, Carsten Haitzler (The Rasterman) wrote:
On Mon, 25 Jun 2018 14:48:54 +0900 Hermet Park <hermetp...@gmail.com> said:
Hmm... do we really need to get a review for this kind of small fix?
At least, committers are proved, definitely know some patches are just
about trivial fix, not necessary get a review.
if they wish to make work for themselves and others ... they are free to do so.
review is not a requirement in efl or e etc. if you have commit access.
On Mon, Jun 25, 2018 at 2:26 PM, Marcel Hollerbach <
m...@marcel-hollerbach.de> wrote:
hermet pushed a commit to branch master.
http://git.enlightenment.org/core/efl.git/commit/?id=
c44c1e2ea077dc689f52239ff341b546e95f2480
commit c44c1e2ea077dc689f52239ff341b546e95f2480
Author: Marcel Hollerbach <m...@marcel-hollerbach.de>
Date: Mon Jun 25 14:25:52 2018 +0900
efl_gfx_path: make counters unsigned
Summary:
we are comparing to unsigned number, and the number are moving strongly
from 0 up.
Depends on D6380
Reviewers: devilhorns
Subscribers: Hermet, cedric, #committers, zmike
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D6381
---
src/lib/efl/interfaces/efl_gfx_path.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/lib/efl/interfaces/efl_gfx_path.c
b/src/lib/efl/interfaces/efl_gfx_path.c
index ca3a842b99..97439c0fee 100644
--- a/src/lib/efl/interfaces/efl_gfx_path.c
+++ b/src/lib/efl/interfaces/efl_gfx_path.c
@@ -335,7 +335,7 @@ _efl_gfx_path_interpolate(Eo *obj, Efl_Gfx_Path_Data
*pd,
{
double *to_pts = to_pd->points;
double *from_pts = from_pd->points;
- int i, j;
+ unsigned int i, j;
for (i = 0; cmds[i] != EFL_GFX_PATH_COMMAND_TYPE_END;
i++)
for (j = 0; j < _efl_gfx_path_command_length(cmds[i]);
j++)
--
--
Regards, Hermet
------------------------------------------------------------------------------
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
------------------------------------------------------------------------------
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