Skip to content

Commit

Permalink
Don't show error messages when disabling
Browse files Browse the repository at this point in the history
Since the destroy() function can be called several times, it is
paramount to don't call objects that have been freed in
previous calls.
  • Loading branch information
sergio-costas authored and vanvugt committed Jun 28, 2024
1 parent 28e64a9 commit 88a1160
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion docking.js
Original file line number Diff line number Diff line change
Expand Up @@ -2551,7 +2551,7 @@ export class DockManager {
this._appSwitcherSettings = null;
this._oldDash = null;

this._desktopIconsUsableArea.destroy();
this._desktopIconsUsableArea?.destroy();
this._desktopIconsUsableArea = null;
this._extension = null;
DockManager._singleton = null;
Expand Down
2 changes: 1 addition & 1 deletion notificationsMonitor.js
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ export class NotificationsMonitor {

destroy() {
this.emit('destroy');
this._signalsHandler.destroy();
this._signalsHandler?.destroy();
this._signalsHandler = null;
this._appNotifications = null;
this._settings = null;
Expand Down

0 comments on commit 88a1160

Please sign in to comment.