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