[plasmashell] [Bug 451124] Switching back from simple menu to Kickoff crashes Plasma in PlaceholderModel::reset()

2023-07-24 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=451124

--- Comment #6 from Nate Graham  ---
David, did that change ever get submitted?

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 451124] Switching back from simple menu to Kickoff crashes Plasma in PlaceholderModel::reset()

2022-08-02 Thread medin
https://bugs.kde.org/show_bug.cgi?id=451124

--- Comment #5 from medin  ---
(In reply to David Edmundson from comment #4)
> Otherwise I'll submit it anyway and we can see what happens.

I did full upgrade today using kde-unstable repo on Manjaro but the problem is
still repro.

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 451124] Switching back from simple menu to Kickoff crashes Plasma in PlaceholderModel::reset()

2022-07-17 Thread David Edmundson
https://bugs.kde.org/show_bug.cgi?id=451124

--- Comment #4 from David Edmundson  ---
If you can build KDE and can reliably reproduce can you change:

placeholdermodel.cpp line 246 from

connect(sourceModelPtr, SIGNAL(destroyed()), this, SLOT(reset()));

to
connect(sourceModelPtr, ::destroyed, this,
::reset);

Otherwise I'll submit it anyway and we can see what happens.

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 451124] Switching back from simple menu to Kickoff crashes Plasma in PlaceholderModel::reset()

2022-07-17 Thread David Edmundson
https://bugs.kde.org/show_bug.cgi?id=451124

David Edmundson  changed:

   What|Removed |Added

 CC||k...@davidedmundson.co.uk

--- Comment #3 from David Edmundson  ---
KAStatsModel is setting the PlaceholderSourceModel to itself (or rather the
d-pointer)

PlaceholderSourceModel calls, so when "d" gets destroyed we're still alive
enough to invoke the slot.
connect(sourceModelPtr, SIGNAL(destroyed()), this, SLOT(reset()));

This means we handle the reset during KAStatsModel's own destructor. 
It's weird, but because we're in the leaf subclass it shouldn't be an issue?

My only theory is that going through the old style syntax means we go via
metaObject which means calling a virtual method. Calling a virtual from a
destructor is awkward.

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 451124] Switching back from simple menu to Kickoff crashes Plasma in PlaceholderModel::reset()

2022-07-17 Thread medin
https://bugs.kde.org/show_bug.cgi?id=451124

--- Comment #2 from medin  ---
This also happens with:

Operating System: Manjaro Linux
KDE Plasma Version: 5.25.80
KDE Frameworks Version: 5.97.0
Qt Version: 5.15.5
Kernel Version: 5.18.12-2-MANJARO (64-bit)
Graphics Platform: X11

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 451124] Switching back from simple menu to Kickoff crashes Plasma in PlaceholderModel::reset()

2022-03-21 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=451124

Nate Graham  changed:

   What|Removed |Added

Summary|Switching back from simple  |Switching back from simple
   |menu to Kickoff kills   |menu to Kickoff crashes
   |Plasma  |Plasma in
   ||PlaceholderModel::reset()
 CC||n...@kde.org

--- Comment #1 from Nate Graham  ---
Oh, those debug symbols are super helpful.

> #6  0x7f480bf4a870 in PlaceholderModel::reset() (this=0x5641c7a11740) at 
> /usr/src/debug/plasma-workspace-5.24.2/applets/kicker/plugin/placeholdermodel.cpp:231
> #7  PlaceholderModel::qt_static_metacall(QObject*, QMetaObject::Call, int, 
> void**) (_o=0x5641c7a11740, _c=, _id=, 
> _a=0x7ffc7ce11650)
> at 
> /usr/src/debug/build/applets/kicker/kickerplugin_autogen/7RBZBFH7CH/moc_placeholdermodel.cpp:116
> #8  0x7f4839968a1f in  () at /usr/lib/libQt5Core.so.5
> #9  0x7f4839968c24 in QObject::destroyed(QObject*) () at 
> /usr/lib/libQt5Core.so.5
> #10 0x7f483995f1c4 in QObject::~QObject() () at /usr/lib/libQt5Core.so.5
> #11 0x7f480bf67f08 in KAStatsFavoritesModel::Private::~Private() 
> (this=0x5641bc8168f0, this=)
> at 
> /usr/src/debug/plasma-workspace-5.24.2/applets/kicker/plugin/kastatsfavoritesmodel.cpp:56
> #12 KAStatsFavoritesModel::Private::~Private() (this=0x5641bc8168f0, 
> this=)
> at 
> /usr/src/debug/plasma-workspace-5.24.2/applets/kicker/plugin/kastatsfavoritesmodel.cpp:56
> #13 0x7f480bf67c01 in KAStatsFavoritesModel::~KAStatsFavoritesModel() 
> (this=0x5641c7a11740, this=)
> at 
> /usr/src/debug/plasma-workspace-5.24.2/applets/kicker/plugin/kastatsfavoritesmodel.cpp:451
> #14 KAStatsFavoritesModel::~KAStatsFavoritesModel() (this=0x5641c7a11740, 
> this=)
> at 
> /usr/src/debug/plasma-workspace-5.24.2/applets/kicker/plugin/kastatsfavoritesmodel.cpp:452

-- 
You are receiving this mail because:
You are watching all bug changes.