Re: [E-devel] [Patch][edje] Prevent duplicated inherition of after

2012-11-30 Thread Bluezery
Please review this patch. When inheriting group, program.after can be duplicated currently. But I think that duplicated program.after is not needed when inherting. If user want that after can be called twice, it's better to add one more after intentionally 2012/11/8 Bluezery ohpo...@gmail.com:

Re: [E-devel] [Patch][edje] Prevent duplicated inherition of after

2012-11-11 Thread Bluezery
2012/11/9 Daniel Juyung Seo seojuyu...@gmail.com: thanks for the quick fix. btw how can i reproduce/check the bug? I don't know this is a bug or intended. if duplicated after usage is wrong case, it's a bug. Daniel Juyung Seo (SeoZ) On Nov 8, 2012 6:37 PM, Bluezery ohpo...@gmail.com

Re: [E-devel] [Patch][edje] Prevent duplicated inherition of after

2012-11-09 Thread Daniel Juyung Seo
thanks for the quick fix. btw how can i reproduce/check the bug? Daniel Juyung Seo (SeoZ) On Nov 8, 2012 6:37 PM, Bluezery ohpo...@gmail.com wrote: Oops, actually I removed automated whitespace removal by editor. Something is remained.. sorry :( I attached patch again. In the orianal

Re: [E-devel] [Patch][edje] Prevent duplicated inherition of after

2012-11-08 Thread Daniel Juyung Seo
Dear taehwan, thanks for the patch. But can you please separate formatting fix(including white space removal) and feature change/bug fix next time? Thanks. Daniel Juyung Seo (SeoZ) On Thu, Nov 8, 2012 at 5:42 PM, Bluezery ohpo...@gmail.com wrote: Dear EFL developers, When inherting group,

Re: [E-devel] [Patch][edje] Prevent duplicated inherition of after

2012-11-08 Thread Bluezery
Oops, actually I removed automated whitespace removal by editor. Something is remained.. sorry :( I attached patch again. In the orianal code, tab spaces are mixed. But I uses only space. 2012/11/8 Daniel Juyung Seo seojuyu...@gmail.com: Dear taehwan, thanks for the patch. But can you