Failing qobject_cast
-
So by following the process above I found my error. It was this stray include lying in my runtime subsystem. (there was a time where I turned off the plugin system and did things locally)
@
#include "../lib_surfacemesh/SurfaceMeshModel.h" /// TO BE REMOVED
@I wish I could offer more intuition on WHY it was causing the whole dynamic casting to fail.. But I seriously have no clue!
NOTE: this fixed the cast/uncast ONLY when I was trying my test code. If an object is created from a plugin, casted to the interface object and casted back to the subclass in a second plugin... the cast still fails... triple sigh...
-
Let me use a small diagram to describe what is happening:
"Diagram":https://docs.google.com/drawings/d/1Jp-wpPuFTCvsC-3ovRmZXlRj93U0TfS7urxpfkLZPEc/editThe core runtime loads the plugins1/2
Everything else is statically linked to Core Runtime
Core runtime calls plugin1 to generate an instance of SurfaceMeshModel
(note: plugin1 casts this instance to the more general Model which is the only thing runtime knows about as the diagram shows)
Core runtime calls plugin2, which receives Model* and needs to cast it back down to SurfaceMeshModel <=== This is the failure point
What I mentioned above is that now the cast/uncast of an object within a single plugin works well. But if I create an object and pass it to another plugin... that's where the casting fails...
Do you see any fault with my design?
Andrea
-
Your SurfaceMeshModel must be defined ONLY once in your application. If Plugin1 and Plugin2 need to know and use SurfaceMeshModel and even more pass instances of it, then you need an extra shared library that will offer it to the plugins. You must then link both your plugins to that library.
If you compile the SurfaceMeshModel in each plugin, it will work locally in that plugin but not outside of the plugin boundaries using qobject_cast.
The reason is that each plugin will have its own QMetaObject (because of the MOCed Q_OBJECT macro) for that SurfaceMeshModel class. And from a Qt standpoint two classes with different QMetaObject instances are not the same class, even if they have the exact same interface and all.
However, I am sure it would work flawlessly with a static_cast as the compiler has generated the same code and memory layout in both plugins but this is not a good practice in general.
You need an extra shared library (that your Core runtime does not need to know as far as it is concerned). -
Hey rcari, fact is.. I did create an extra shared library doing exactly what you say!
I am building some example code to show you guys. But as you say, within both plugins I have a "pri" file that loads the SurfaceMeshModel class:@
macx:POST_TARGETDEPS += $$SOURCE_ROOT/libraries/lib_surfacemesh.a
macx:LIBS += -L$$SOURCE_ROOT/libraries -l_surfacemesh
@And yes, the core library is unaware of this shared (static) library used by both plugins.
Let me see if the static_cast will work, just for the heck of it :) -
You can't use it as a static lib, it breaks the whole point. The static lib will be fully included in each plugin.
You need a SHARED library (in opposite to STATIC, not as a conceptual shared compilation unit).
Using a SHARED library, when one of the plugins requiring it is loaded, this extra library is loaded as well, when the second plugin is loaded, the extra library is already loaded and they SHARE that code.
See http://en.wikipedia.org/wiki/Library_(computing) -
Here you go :) Rcari I think you solved the problem. What you are saying makes perfect sense!
Indeed, static_cast did work (pointer was not null), but when I then tried to use the casted object I got some wacky runtime error!!
I will let you know if changing to dynamic library will solve it. Thank you so much for your input :)
-
LaCie? I think I have a usb key in the shape of a real key from your company :)
-
@
class LOCSurfaceMeshModel : public Model, public Surface_mesh{
Q_OBJECT
Q_INTERFACES(Model)
public:
LOCSurfaceMeshModel(){}
};
@
Seems a bit weird. I think the Q_INTERFACES(Model) declaration is wrong (and just unneeded). -
Actually I already run into this before :)
It is needed, basically you can only qobject_cast to what you have declared as being a Q_INTERFACES. Anything else will fail :) -
Well, that is highly probable :)
[quote author="tallia1" date="1319872616"]LaCie? I think I have a usb key in the shape of a real key from your company :)[/quote] -
[quote author="tallia1" date="1319874092"]Actually I already run into this before :)
It is needed, basically you can only qobject_cast to what you have declared as being a Q_INTERFACES. Anything else will fail :)[/quote]Well that is true for non QObject classes. For QObject you can safely go up the inheritance tree using qobject_cast without declaring interfaces. I agree that this interface thing seems unneeded. However because of your architecture, I don't know for sure if that is still true across plugins boundaries...
-
[quote author="tallia1" date="1319874092"]Actually I already run into this before :)
It is needed, basically you can only qobject_cast to what you have declared as being a Q_INTERFACES. Anything else will fail :)[/quote]That is only true for non-QObjects, AFAIK. As your model is already a QObject, there is no need for it.