2015-02-18 17:59 GMT+03:00 Nikita Popov <nikita....@gmail.com>: > Alexander, I would recommend you to split this into two RFCs, one dealing > only with AST export (and maybe pretty printing) and the second one with > the compilation hooks. There's probably a few questions about the export > API that should be discussed that will be forgotten if everyone focuses on > the more complicated, and probably not yet relevant, question of > compilation hooks.
Hello, Nikita! Thanks for you thoughts! Sounds reasonable for me, because of short timeframe for PHP7. Let's go with the first part of RFC, because second part is really cumbersome and can be applied later, eg. in 7.1 I will split this RFC into two parts and mark first part for 7.0. About first part, do you agree with proposed classes and namespace `Php\Parser`? I want to propose to use classes for parser API instead of pure functions because it can give more usable OO-API for future needs. There is one more notice regarding to the zval nodes, they should be also exported as objects, not as values. I think we should export coherent nodes for everything.
