Good patch :)
Thanks in SVN!
http://trac.enlightenment.org/e/changeset/78221

But you don't need to remove white spaces anymore.
As we already discussed this a lot in this mailing list, removing
white spaces are not necessary and not important.

But you should keep the patch simple and clean.
So you don't need to include white space removal next time.
http://trac.enlightenment.org/e/wiki/PatchReview

Thanks.

Daniel Juyung Seo (SeoZ)

On Fri, Oct 19, 2012 at 12:30 PM, Bluezery <ohpo...@gmail.com> wrote:
> Dear all,
>
> There is a bug, invalid data assign into ecore_job.
> Trivial whilte space removal is included also.  it's automated in my editor 
> :D.
>
> Please review this patch.
>
> --
> BRs,
> Kim.
>
> ------------------------------------------------------------------------------
> Everyone hates slow websites. So do we.
> Make your web apps faster with AppDynamics
> Download AppDynamics Lite for free today:
> http://p.sf.net/sfu/appdyn_sfd2d_oct
> _______________________________________________
> enlightenment-devel mailing list
> enlightenment-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
>

------------------------------------------------------------------------------
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_sfd2d_oct
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to