Hi all, On Thu, Jan 29, 2015 at 5:24 PM, Yasuo Ohgaki <yohg...@ohgaki.net> wrote:
> A new API seems really cleaner. > > > Reasonable discussion. I'll prepare API version patch later. This is the patch. It's not complete as user save handler requires more work. Simply read() is called as create() now. It's a PoC patch. https://github.com/yohgaki/php-src/compare/master-session-new-api2 It's not too complex compared to parameter version. https://github.com/yohgaki/php-src/compare/master-session-new-api The issue with create() is that it enforces save handlers to implement create(). It may be implemented that to check null handler to allow save handler w/o create(), but it wouldn't look nice. Since all save handlers must be rewritten for PHP7, it would be the best to have only one save handler definition for internal save handlers while keeping user save handler compatibility. Dirty part can be hidden in mod_user.c. Regards, -- Yasuo Ohgaki yohg...@ohgaki.net
