On 01/03/16 09:23, Andrew Williams wrote:
> Hi Tom,
>
> Overall it looks great :)
> I'm curious, however, about why we need the worse eo_add syntax - it seems
> like a step backwards and passing a pointer and its address in the same
> line seems a little messy :(

I know, I explicitly said it's a bit of a step backwards. :)

I haven't given eo_add a lot of thought, maybe I can improve it. Part of 
the reason to give such an heads up is so people can comment and 
improve. Go ahead, comment and improve. :)

>
> Other thought - I expect that looking for the result of eo_super and
> referencing later may be a common issue as it looks legal. If it's not
> possible to support that can we trap/track it and report clearly on error
> log?

Same comment as above. I'm aware it's an issue, that's why I pointed it 
out. I'm less concerned about abusing it for super, and more about 
abusing it for part, which I suspect will be super common. We will error 
about it, but only on the second call (which is not soon enough because 
it's already illegal on the first), but the error won't be that good, 
because it may be another issue. Again, comments are welcome, I'd love 
to trap it better.

--
Tom.

>
> Good work - looking forward to it!
> Andy
> On Tue, 1 Mar 2016 at 09:06, Tom Hacohen <t...@osg.samsung.com> wrote:
>
>> Hey,
>>
>> The Eo syntax is going to be changing once more, and this time, I really
>> think/hope it'll be the last time. We plan on stabilizing Eo and all of
>> the functions on top of it in the next few months, so that doesn't leave
>> us much more time to change it again. :)
>>
>> These changes will remove the need for the eo_do family of functions.
>> Functions will now look like normal C functions (which they are). There
>> are many benefits to that, and we have many cool new ideas.
>>
>> For more info: https://phab.enlightenment.org/w/eo/
>>
>> I'm sending this email as an head's up, as I'll be starting to work on
>> migrating to the new Eo syntax (and implementing it) today. Felipe and I
>> have actually already started (needed to for the PoC), but I plan on
>> pushing my changes to master soon.
>>
>> If you have any issues/suggestions/comments with the proposal, please
>> let me know, either in pm, irc or just here.
>>
>> --
>> Tom
>>
>>
>> ------------------------------------------------------------------------------
>> Site24x7 APM Insight: Get Deep Visibility into Application Performance
>> APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
>> Monitor end-to-end web transactions and take corrective actions now
>> Troubleshoot faster and improve end-user experience. Signup Now!
>> http://pubads.g.doubleclick.net/gampad/clk?id=272487151&iu=/4140
>> _______________________________________________
>> enlightenment-devel mailing list
>> enlightenment-devel@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
>>
> ------------------------------------------------------------------------------
> Site24x7 APM Insight: Get Deep Visibility into Application Performance
> APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
> Monitor end-to-end web transactions and take corrective actions now
> Troubleshoot faster and improve end-user experience. Signup Now!
> http://pubads.g.doubleclick.net/gampad/clk?id=272487151&iu=/4140
> _______________________________________________
> enlightenment-devel mailing list
> enlightenment-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
>


------------------------------------------------------------------------------
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=272487151&iu=/4140
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to