On 04/05/2011 10:43 PM, Ilmari Karonen wrote:
On 04/05/2011 02:42 AM, Tim Starling wrote:
You can't include the class files for compiled classes, they all exist at startup and you get a redeclaration error if you try. I explained this in the documentation page on mediawiki.org which has now appeared.
http://www.mediawiki.org/wiki/HipHop
The autoloader is not used. It doesn't matter where the class_exists() is. HipHop scans the entire codebase for class_exists() at compile time, and breaks any classes it finds, whether the or not the class_exists() is reachable.
These both really sound like bugs in HipHop. I've no idea how hard it would be to fix them, but are we reporting them at least?
I reported the class_exists() thing. The fact that classes exist on startup is more of a design decision than a bug. I don't want to annoy the HipHop devs with frivolous bug reports, we need a lot of favours from them.
-- Tim Starling