We should remove that, as well.

commented out code

?

Why are you commenting this out?

If it doesn't make sense anymore, remove it :)
We can still get it back using git, if we really turn out to need it ;)

if this accept method doesn't do anything different, than the method of the base class, then there's no reason to reimplement it ;)

I think we can remove that as well :)

Please put the implementation into the cpp file. :)

We should just delete that, rather than commenting it out.

Same as above, we should delete it.

We should not include outcommented code, so we should remove it for now.
(btw, you can just use Property *property = 0; here and then don't need to set it in the constructor, that's cool new C++11 :) )

Please put the implementation into the cpp file. :)

Please put the implementation into the cpp file. :)

Please put the implementation into the cpp file. :D

Please put the implementation into the cpp file. :) (Nooooo, I'm not copy&pasting that :D)

you accidentially added a few whitespaces here ;)

Please put the implementation into the cpp file. :)

Please put the implementation into the cpp file. :)

If you add a comment yourself, that this is bad, you shouldn't do it :)
Rather add a QHash<IR::Object *, QString> m_objectToInternalName; (or alike) member variable to the GeneratorPass.

Please put the implementation into the cpp file. :)

Why is class an object?

Guess what :D

I think we need Component to inherit from Object. But your implementation is much nearer to what we need, so let's just remove this :)

Please put the implementation into the cpp file. :)

we should keep dummy the last in the list

No, this isn't the right approach here.
We're going to load the same kind of thing from QtQmlModuleLoader, but it doesn't belong into the IR, imo.

If you need it as a polyfill until the QtQmlModuleLoader is available, please put it into prettygeneratorpass.cpp, for now. :)

I think a boolean flag is enough and it seems you successfully implemented it with a boolean flag. So we can remove that code :)

I think we must subclass it from Object, because the rootObject belongs to the Component-object, not to the class "Component".
We'll need to add a flag to "Class" as well, like "isComponent" (I added that to my branch, you won't need it for now)

Please put the implementation into the cpp file. :)

Yeah, you know, what to do :D