On Tue, 4 Feb 2014 00:39:24 +0900 Daniel Juyung Seo <seojuyu...@gmail.com> said:
> On Mon, Feb 3, 2014 at 8:34 PM, Cedric BAIL <cedric.b...@free.fr> wrote: > > > On Mon, Feb 3, 2014 at 5:54 PM, Daniel Juyung Seo <seojuyu...@gmail.com> > > wrote: > > > Hi Cedric, you took Rajeev's credit. > > > The patch was from Rajeev. > > > > Yeah, I don't know what my editor did here, but something went wrong > > when doing this commit. I am not sure why. > > > > Oh... The same thing happened to me. > https://git.enlightenment.org/core/elementary.git/commit/?id=a6d73966de31b996b79129a3ffb4007751af2f5e > This patch was from Shilpa... > It was working fine until several hours ago with other's patches. > > Maybe.. Phab is broken? > > Daniel Juyung Seo (SeoZ) there is some odd thing with arc/phab where SOME patches from SOME people lose their original author info. most seem not to, but some do. the problem is not malice of us stealing authorship - the tools fail and we don't see/notice it manually UNLESS we do git log before we commit (which after reviewing the patch already is kind of just double-work) and then actually happen to notice the author has changed. i sometimes catch these then spend some time in a git rebase -i and amend... it's common enough where i wrote a small script to amend the author of the last commit locally, but again - i need to notice. right now i don't know WHY this happens, but it does. maybe it's missing data in the phab user? missing email or email hasn't been verified? > > > Daniel Juyung Seo (SeoZ) > > > > > > > > > On Mon, Feb 3, 2014 at 4:20 PM, Cedric BAIL <cedric.b...@free.fr> wrote: > > > > > >> cedric pushed a commit to branch master. > > >> > > >> > > >> > > http://git.enlightenment.org/core/efl.git/commit/?id=90cd40753dc1e59622c174585282210cacbcd30e > > >> > > >> commit 90cd40753dc1e59622c174585282210cacbcd30e > > >> Author: Cedric BAIL <cedric.b...@samsung.com> > > >> Date: Mon Feb 3 15:35:36 2014 +0900 > > >> > > >> Patcna: fix wrong widgth and height in eina_rectangle computation > > >> logic. > > >> > > >> Reviewers: cedric, raster, seoz, Hermet, bluezery > > >> > > >> Reviewed By: cedric > > >> > > >> Differential Revision: https://phab.enlightenment.org/D500 > > >> > > >> Signed-off-by: Cedric BAIL <cedric.b...@samsung.com> > > >> --- > > >> AUTHORS | 1 + > > >> src/lib/eina/eina_inline_rectangle.x | 4 ++-- > > >> 2 files changed, 3 insertions(+), 2 deletions(-) > > >> > > >> diff --git a/AUTHORS b/AUTHORS > > >> index 35068f1..dd4e277 100644 > > >> --- a/AUTHORS > > >> +++ b/AUTHORS > > >> @@ -49,6 +49,7 @@ Patryk Kaczmarek <patry...@samsung.com> > > >> Igor Murzov <e-m...@date.by> > > >> Vladislav Brovko <v.bro...@samsung.com> > > >> ChunEon Park (Hermet) <her...@hermet.pe.kr> > > >> +Rajeev Ranjan (Rajeev) <rajee...@samsung.com> <rajeev.jn...@gmail.com> > > >> > > >> Eet > > >> --- > > >> diff --git a/src/lib/eina/eina_inline_rectangle.x > > >> b/src/lib/eina/eina_inline_rectangle.x > > >> index d89d7f1..e25942b 100644 > > >> --- a/src/lib/eina/eina_inline_rectangle.x > > >> +++ b/src/lib/eina/eina_inline_rectangle.x > > >> @@ -173,7 +173,7 @@ eina_rectangle_union(Eina_Rectangle *dst, const > > >> Eina_Rectangle *src) > > >> } > > >> /* right */ > > >> if ((dst->x + dst->w) < (src->x + src->w)) > > >> - dst->w = src->x + src->w; > > >> + dst->w = src->x + src->w - dst->x; > > >> /* top */ > > >> if (dst->y > src->y) > > >> { > > >> @@ -182,7 +182,7 @@ eina_rectangle_union(Eina_Rectangle *dst, const > > >> Eina_Rectangle *src) > > >> } > > >> /* bottom */ > > >> if ((dst->y + dst->h) < (src->y + src->h)) > > >> - dst->h = src->y + src->h; > > >> + dst->h = src->y + src->h - dst->y; > > >> } > > >> > > >> /** > > >> > > >> -- > > >> > > >> > > >> > > > > > ------------------------------------------------------------------------------ > > > Managing the Performance of Cloud-Based Applications > > > Take advantage of what the Cloud has to offer - Avoid Common Pitfalls. > > > Read the Whitepaper. > > > > > http://pubads.g.doubleclick.net/gampad/clk?id=121051231&iu=/4140/ostg.clktrk > > > _______________________________________________ > > > enlightenment-devel mailing list > > > enlightenment-devel@lists.sourceforge.net > > > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > > > > > > > > > > > -- > > Cedric BAIL > > > > > > ------------------------------------------------------------------------------ > > Managing the Performance of Cloud-Based Applications > > Take advantage of what the Cloud has to offer - Avoid Common Pitfalls. > > Read the Whitepaper. > > > > http://pubads.g.doubleclick.net/gampad/clk?id=121051231&iu=/4140/ostg.clktrk > > _______________________________________________ > > enlightenment-devel mailing list > > enlightenment-devel@lists.sourceforge.net > > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > > > ------------------------------------------------------------------------------ > Managing the Performance of Cloud-Based Applications > Take advantage of what the Cloud has to offer - Avoid Common Pitfalls. > Read the Whitepaper. > http://pubads.g.doubleclick.net/gampad/clk?id=121051231&iu=/4140/ostg.clktrk > _______________________________________________ > enlightenment-devel mailing list > enlightenment-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > -- ------------- Codito, ergo sum - "I code, therefore I am" -------------- The Rasterman (Carsten Haitzler) ras...@rasterman.com ------------------------------------------------------------------------------ Managing the Performance of Cloud-Based Applications Take advantage of what the Cloud has to offer - Avoid Common Pitfalls. Read the Whitepaper. http://pubads.g.doubleclick.net/gampad/clk?id=121051231&iu=/4140/ostg.clktrk _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
