Hi, This is more or less final proposed patch for 5.4
http://pastebin.com/ceiWWD4N It fixes implementation mistakes and makes the whole implementation much simpler. I hope I didn't introduce new bugs :) Of course I checked it with PHP test suite, but it would be great to test it with real life applications that use traits. The patch keeps binary compatibility, and changes only error messages and behavior in error situations. It must not affect any applications. I like to commit the patch in 5.4 and above on next week. I would simplify the trait implementation much more, but the conflict resolution rules allows to make so many tricks that all needs to be checked :( Anyway, I'll think about it a bit more. Any feedback is welcome. Thanks. Dmitry. On Tue, Dec 18, 2012 at 3:46 PM, Stefan Marr <p...@stefan-marr.de> wrote: > Hi Dmitry: > > On 18 Dec 2012, at 12:37, Dmitry Stogov wrote: > > > I'm going to take a deep look into trait implementation and provide a > > better solution for 5.5. > > The current implementation is really wired and makes a lot of troubles > for > > maintenance and each new fix, makes new troubles :( > Sorry, that's mostly me lacking understanding of the PHP internals. > And there are many wired corner cases that have to be covered. > > > I'm going to work on it with top priority during last few days and then > > send a patch. > > Thanks for looking into it. > I'll be able to test things over christmas. > > Best regards > Stefan > > > -- > Stefan Marr > Software Languages Lab > Vrije Universiteit Brussel > Pleinlaan 2 / B-1050 Brussels / Belgium > http://soft.vub.ac.be/~smarr > Phone: +32 2 629 2974 > Fax: +32 2 629 3525 > >
