Improve safety in code

[tmartsum]

Delete areas later so that that they can be accessed by (inherited) dock widgets in dtor. Add some QPointer to prevent crashes.

Hence allow users to do more while dock widgets etc are being destroyed.

[githubuser0xFFFF]

@tmartsum Thank you for this pull request. I just reviewed it and it looks good for me. Just a wish regarding coding style:

Instead of:

if (!DockArea) continue;

or

if (!DockArea) {
   continue;
}

I would prefer the following style: always brackets and the opening bracket on a new line

if (!DockArea)
{
   continue;
}

Just in case you would like to provide more pull requests 😏. It would be great, if you could polish this pull request to match this style.

[tmartsum]

Hi Uwe
I have updated the MR 😄 and I wrote your mail (at the time I pushed the original review).
Interesting that you use KeyShot 😄 - and I am glad you like it 😉. I like to comment your thoughts in a mail - I did write you one when creating this MR.

I did try to fit style (e.g by using tabs though most code-bases in both Luxion and Qt are using spaces), but I may not have cought all aspects and habbits can be difficult to change 😉 - just poke me in the future MRs if I do any mistakes include styling. I can however see, that others are contributing with spaces, but I suspect you prefer tabs in the project?

Best Regards
Thorbjørn

[githubuser0xFFFF]

Please have a look at my review comments - thank you.

[githubuser0xFFFF]

Maybe it would be better to return a const QList<QPointer<CFloatingDockDontainer>> here because the function floatingWidgets() is used in CDockManager::eventFilter on linux. So this change may have a negative performance impact because it is called very often.

[githubuser0xFFFF]

Thank you. I merged your pull request.