On 09/06/16 23:06, Felipe Magno de Almeida wrote: > On Thu, Jun 9, 2016 at 12:23 PM, Cedric BAIL <cedric.b...@free.fr> wrote: >> On Jun 9, 2016 03:14, "Tom Hacohen" <t...@osg.samsung.com> wrote: > > [snip] > >>> If rejected is set, the chunk is rejected no matter what the strbuf >>> value is. > > [snip] > >> My main concern is actually what happen if there is more than one event >> handler registered. I think it become pretty complex to chain them with all >> this parameter. What if you just pass the text embedded in the strbuf >> already and you just look at it for all behavior at the end ? > > Kolesa and I have argued that this will be prohibited and that > no aliasing will be considered. So for example, JS will copy the > boolean value and when the boolean is modified it will not > impact the original struct of the event. > > This is the only sane thing to do and to avoid random crashes > in languages that are suppose to be safe from UB. > > IMO. Just use an override function for this.
That makes sense, I guess. It seems like I'll have to do something similar to what jpeg did with input events and pass an object for text filtering. I'll do this unless I hear objections (second try!). :) > >> Cedric >> >>> This is what I'm going to go with unless I hear any strong objections by >>> Monday. > > I strongly object to it. It's only you between me and my happiness! I strongly agree with you though. That's why I asked, I knew it didn't feel right. -- Tom. ------------------------------------------------------------------------------ What NetFlow Analyzer can do for you? Monitors network bandwidth and traffic patterns at an interface-level. Reveals which users, apps, and protocols are consuming the most bandwidth. Provides multi-vendor support for NetFlow, J-Flow, sFlow and other flows. Make informed decisions using capacity planning reports. https://ad.doubleclick.net/ddm/clk/305295220;132659582;e _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
