On Tue, 18 Feb 2014 17:32:47 +0100 Stefan Schmidt <ste...@datenfreihafen.org> said:
> Hello. > > On Tue, 2014-02-18 at 08:32, Carsten Haitzler wrote: > > On Mon, 17 Feb 2014 14:59:51 +0100 Stefan Schmidt > > <ste...@datenfreihafen.org> said: > > > > > Hello. > > > > > > On Fri, 2014-02-14 at 00:35, Sung W. Park wrote: > > > > sung pushed a commit to branch master. > > > > > > > > http://git.enlightenment.org/core/efl.git/commit/?id=2b4a7b438def96d01a5c0bb85d2c9636773d333d > > > > > > > > commit 2b4a7b438def96d01a5c0bb85d2c9636773d333d > > > > Author: Sung W. Park <sung...@gmail.com> > > > > Date: Sat Feb 15 02:33:47 2014 +0900 > > > > > > > > evas gl: added partial rendering support for qcom extension > > > > > > > > When running in direct rendering mode, properly support partial > > > > rendering if the extension is properly supported. > > > > > > > > Also, fixed the SwapBufferwWithDamage rectangle coordinate bug. > > > > It wasn't properly y-inverted before. > > > > --- > > > > src/modules/evas/engines/gl_common/evas_gl_api.c | 28 ++++- > > > > .../evas/engines/gl_common/evas_gl_common.h | 9 +- > > > > .../evas/engines/gl_common/evas_gl_context.c | 58 +++++++-- > > > > src/modules/evas/engines/gl_common/evas_gl_core.c | 132 ++++++++++++++ > > > > ++++ +-- src/modules/evas/engines/gl_common/evas_gl_core.h | 13 +- > > > > .../evas/engines/gl_common/evas_gl_core_private.h | 10 +- > > > > src/modules/evas/engines/gl_x11/evas_engine.c | 86 ++++++++++---- > > > > src/modules/evas/engines/wayland_egl/evas_engine.c | 14 ++- > > > > 8 files changed, 298 insertions(+), 52 deletions(-) > > > > > > After reading through the whole backlog of commits that happened over > > > the last week this commit jumps out. It hit git 5 days after the merge > > > window closed. > > > > > > While the commit message mentions that this also fixes a bug it > > > clearly describes a new feature added. Partial rendering support. What > > > worries me most is that it touches core code in the evas gl engine. > > > > > > Did you realize that that we have been in a stabilization phase which > > > would exclude new features? If yes, why did you decide to put it in? > > > > > > Its a candidate for reverting for me but I want to seek some opinions > > > first. > > > > > > regards > > > Stefan Schmidt > > > > we already had support for qcomm partial rendering - it was just half-baked. > > With this definition we could bring in tons of features to finish > half-baked things we have. :) > > I close my eyes and let it slip through hoping it will not come back > to us after the release. by half baked - i mean it worked in some cases, and not others. -- ------------- 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=121054471&iu=/4140/ostg.clktrk _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
