dose the emit self is thread safe?
-
i have a global object ,and it have one signal, and i send this signal in two worker thread . dose the emit is thread safe?
one is :
QtConcurrent::run(this{
char databuff[255]={0};
emit this->signalforsomething(databuff);
});
another is same :
QtConcurrent::run(this{
char databuff[255]={0};
emit this->signalforsomething(databuff);
}); -
QMetaObject::activatelooks to be thread safe so it should be ok. An alternative would be to use 2 separate proxy objects, connect their signals to the global object signal and emit the signals from the proxys.emitwill return immediately sodatabuffwill go out of scope before the slots can use it. It's easily solved by usingQByteArrayinstead of a rawchar []"La mort n'est rien, mais vivre vaincu et sans gloire, c'est mourir tous les jours"
~Napoleon BonaparteOn a crusade to banish setIndexWidget() from the holy land of Qt
-
QMetaObject::activatelooks to be thread safe so it should be ok. An alternative would be to use 2 separate proxy objects, connect their signals to the global object signal and emit the signals from the proxys.emitwill return immediately sodatabuffwill go out of scope before the slots can use it. It's easily solved by usingQByteArrayinstead of a rawchar []It's easily solved by using QByteArray instead of a raw char []
Ah, I certainly did not know that! I presume it's because
QByteArrayhasQ_OBJECTor something? Could you point me to docs which explain the persistence of whatever Qt types instead of the C++ ones across signals/slots so that I can at least read up? My situation is different because I use Python/PyQt, but I'd like to understand how it works from C++. Thanks. -
It's easily solved by using QByteArray instead of a raw char []
Ah, I certainly did not know that! I presume it's because
QByteArrayhasQ_OBJECTor something? Could you point me to docs which explain the persistence of whatever Qt types instead of the C++ ones across signals/slots so that I can at least read up? My situation is different because I use Python/PyQt, but I'd like to understand how it works from C++. Thanks.@JonB said in dose the emit self is thread safe?:
but I'd like to understand how it works from C++
This is very C++ centric. raw arrays passed to a function are not copied, they just get passed as pointers to the original memory so if the original memory is destroyed the pointer becomes meaningless.
Any declared and registered metatype with a copy constructor can be passed by value or const reference to a signal without any overhead.
I suggested
QByteArrayas it's the container version Qt offers forchar[]butstd::array<char,255>would work as well once you register it with the meta object system -
yes i am mistake, char databuff[255]={0}; this should be QByteArray ..fix:
one is :
QtConcurrent::run(this{
QByteArray databuff1= "xxx";
emit this->signalforsomething(databuff1);
});
another is same :
QtConcurrent::run(this{
QByteArray databuff2= "yyyy";
emit this->signalforsomething(databuff2);
});
btw ,the two connects is Qt::QueuedConnection;; for easy to manage. i use a global object to manage all the singnal of my app. and i wonder if the emit itself is thread-safe in diffrent worker thread ..and my result of test looks no problem as VRonin saying. but i still worry about it that it may have a small chance of working wrong -
yes i am mistake, char databuff[255]={0}; this should be QByteArray ..fix:
one is :
QtConcurrent::run(this{
QByteArray databuff1= "xxx";
emit this->signalforsomething(databuff1);
});
another is same :
QtConcurrent::run(this{
QByteArray databuff2= "yyyy";
emit this->signalforsomething(databuff2);
});
btw ,the two connects is Qt::QueuedConnection;; for easy to manage. i use a global object to manage all the singnal of my app. and i wonder if the emit itself is thread-safe in diffrent worker thread ..and my result of test looks no problem as VRonin saying. but i still worry about it that it may have a small chance of working wrong@jimfar said in dose the emit self is thread safe?:
i use a global object to manage all the singnal of my app
Bad idea.
which doc of qt i can found to described this detail?
I don't think it's documented, you just have to trust QMetaObject::activate to remain thread safe
-
which doc of qt i can found to described this detail?
I suggest to take another look at an article like “How Qt Signals and Slots Work” by Olivier Goffart from 2012-12-02.
-
ok thanks .did u have a better idea for manage that upto 100+ signals or more, VRonin. i use global object cos i can easy to look them up and sort them .There seems to be a better way.
