QList<const QBluetoothDeviceInfo* declaration / definiton problem

[J.Hilk]

I think the case is pretty clear,

QList<const QBluetoothDeviceInfo* > RemoteDeviceInfo; expects a heap allocated QBluetoothDeviceInfo instance

RemoteDeviceInfo.append(&info); info is a local function scoped stack allocated instance of QBluetoothDeviceInfo instance.

this is a different scoped function

void SettingsDialog::on_applyButton_11_clicked()
{
text = " DEBUG TEST Direct access ";
text += RemoteDeviceInfo.at(0)->name();
qDebug()<< text;
return;

trying to access the now destructed info instance.

Very likely, but we will never know for sure.

[Anonymous_Banned275]

@JonB info is a result of SIGNAL - remote Bluetooth device found.

[Anonymous_Banned275]

@J-Hilk Thanks for confirming the problem.

[Anonymous_Banned275]

@AnneRanch The crash happens in the attempt to access a deleted object. Probably it just goes out of scope after the adding function has returned. This very helpful to know.

[Anonymous_Banned275]

@J-Hilk Many thanks.
Finally something which makes sense.

My mistake - I was just trying to make a copy of "info" for more processing AFTER the device scan is finished , not during the actual scan , which can take almost a full minute to finish.

This is all I know about "info" :

public slots:
void addDevice(const QBluetoothDeviceInfo&);

the slot processes SIGNAL deviceDiscovered

connect(discoveryAgent, SIGNAL(deviceDiscovered(QBluetoothDeviceInfo)),
this, SLOT(addDevice(QBluetoothDeviceInfo)));

If the solution is to put "info" on heap - I do not know how...

however if I can manage to do the desired work on received "info" , while the scan is still not finished,
I may have to put it all into the slot "addDevice". Prefer not to do that.

Cheers

[Anonymous_Banned275]

@AnneRanch An idea - would this work in my application ?
Little advanced for me , but if it would work I would try it.

https://embeddeduse.com/2013/06/29/copied-or-not-copied-arguments-signals-slots/#:~:text=When QMetaObject%3A%3Aactivate creates the command object%2C it stores,order to make the routing of QMetaObject%3A%3Aactivate work.

PS Do I understand it correctly - QT does not have "copy constructor " ?

[Axel Spoerl]

PS Do I understand it correctly - QT does not have "copy constructor " ?

Goodness, no. Of course we have.

[Pl45m4]

@AnneRanch said in QList<const QBluetoothDeviceInfo* declaration / definiton problem:

PS Do I understand it correctly - QT does not have "copy constructor " ?

QObjects don't have. By design, because they have to stay unique.

Think about QObject properties, parent/child, unique identifier (like objectName). If you clone a QObject "one to one", it would cause a big mess and would violate the Qt objecttree / parent-child paradigm, since nothing is unique anymore.
Where should parent of some widget point to, when the parent widget up the object tree was cloned and there are two identical objects now?!

• https://doc.qt.io/qt-6/qobject.html#no-copy-constructor-or-assignment-operator

As mentioned here, a copy c'tor might be declared / implemented in the private section of a class, but is disabled through Q_DISABLE_COPY macro.

• https://doc.qt.io/qt-6/qtclasshelpermacros-proxy.html#Q_DISABLE_COPY

[Anonymous_Banned275]

@J-Hilk Would it be too offensive to ask you to discuss this ?
PS I do not get notified about new posts so it may take a few days before I ;ll read your reply.

[Pl45m4]

@AnneRanch said in QList<const QBluetoothDeviceInfo* declaration / definiton problem:

Would it be too offensive to ask you to discuss this ?

What do you want to discuss? Whether the documentation is lying?
You should not copy QObject.

Everything else, you can copy/use copy c'tors, as it's just standard C++ language and not "Qt".

[Anonymous_Banned275]

@Pl45m4 I was looking for alternative way to make a copy of "info" and I got it solved. CLOSED

[J.Hilk]

@AnneRanch said in QList<const QBluetoothDeviceInfo* declaration / definiton problem:

@J-Hilk Would it be too offensive to ask you to discuss this ?

No, of course not, open discourse is one of the fundamentals of society.

@AnneRanch said in QList<const QBluetoothDeviceInfo* declaration / definiton problem:

@Pl45m4 I was looking for alternative way to make a copy of "info" and I got it solved. CLOSED

I'm curious on what you now ended up using! But for your information I wanted to point out, that you can click on the cog symbol on your opening post and actually select "Mark topic as solved". Alternatively, if you think that one answer in particular solved your issue, you can mark that answer as the correct one.

[Anonymous_Banned275]

@J-Hilk

Currently I just add "info" to struct - class variable and pass "info" to another function for( partial) processing.
I am still not sure if I can process multiple SIGNAL.

Here is a code snippet. PLEASE keep in mind it is a code under construction and I like to keep old code so I do not duplicate and of course I use QDebug extensively.
I am posting the cope "as is " PLEASE do not tell me it does not compile.

       void SettingsDialog::addDevice(const QBluetoothDeviceInfo &info)
       {
           // //*BT_Database_Array[16];
           //HERE
           BT_Database_Array[Array_Index].BT_INFO_Address = info.address();
           BT_Database_Array[Array_Index].INFO_Name = info.name();

           //Array_Index++;
           //#endif
           //temporary copy to QString
           INFO_Name = info.name();
           BT_Database.INFO_Name = info.name();
           BT_Database.INFO_Address = info.address().toString();
           // verify name
           text = " TASK  verify name";
           text += BT_Database_Array[Array_Index].INFO_Name;
#ifdef DEBUG_TASK_SCAN
           qDebug()<< text;
#endif
           text= "Read info ";
           text += " name ";
           text += info.name();
           text += " address ";
           text += info.address().toString();

#ifdef DEBUG_TASK_SCAN
           qDebug()<< text;
#endif

           // verify address
           text = " TASK  verify address";
           text += BT_Database_Array[Array_Index].BT_INFO_Address.toString();
#ifdef DEBUG_TASK_PAIRING
           qDebug()<< text;
#endif

           //Pairing_verify(0);
           Pairing_verify(0, info);

[Anonymous_Banned275]

@Axel-Spoerl Hello sir, I wonder if I could ask you a favor which
would be immensely helpful in using QT.
I am looking forward to your reply.
Thanks
Cheers

[Axel Spoerl]

@AnneRanch
Feeling honored to be addressed aristocraticly :-)
Just ask!

[Anonymous_Banned275]

@Axel-Spoerl OK - here it comes

is there a written document (QT ) describing how to setup QT to
compile ONCE ?

There are options but I must have them set incorrectly - "typical" user error...

I have a growing "subdirs" project and when I do "rebuild" and then "run" or "debug" it will compile again - without me doing any editing.

Same when I do edit , then "run " and "run" again, it compiles again , without me doing any more editing.

This redundant recompile is beginning to take more and more of my precious time ... I could spent with my grand kids instead ...

I had posted a bug report but got no solution.

[Axel Spoerl]

@AnneRanch
I guess this is worth a thread of its own.
Generally, it is not the intention for Qt Creator to always recompile.
If the project is configured correctly, what you describe should not happen.
Troubleshooting depends on whether you use qmake or cmake.
Both build systems check time stamps to determine if a build tree is dirty and needs compilation.

Typical constellations that trigger rebuilding are

• git pull or git rebase in the source tree
• a build step in Qt Creator that triggers a cleanup before building
• broken subdir configuration

Suggest you post your subdir structure and cmake / .pro files in a separate thread.

[Anonymous_Banned275]

@Axel-Spoerl Thanks , I did post "sarcastic" ...." it compiles" in "the lounge".

Maybe I could ask for it to be moved .....

and do as you suggested after