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 _______________________________________________ Wikitech-l mailing list Wikitech-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikitech-l
