diff --git a/js/extensionPrefs/main.js b/js/extensionPrefs/main.js index 552360080..9c5e3ecc0 100644 --- a/js/extensionPrefs/main.js +++ b/js/extensionPrefs/main.js @@ -285,8 +285,9 @@ var Application = GObject.registerClass({ log(`Failed to connect to shell proxy: ${e}`); this._mainStack.add_named(new NoShellPlaceholder(), 'noshell'); this._mainStack.visible_child_name = 'noshell'; - } else + } else { throw e; + } return; } diff --git a/js/ui/layout.js b/js/ui/layout.js index 17bda4778..4708710f4 100644 --- a/js/ui/layout.js +++ b/js/ui/layout.js @@ -1075,16 +1075,17 @@ var LayoutManager = GObject.registerClass({ side = Meta.Side.RIGHT; else continue; - } else if (x1 <= monitor.x) + } else if (x1 <= monitor.x) { side = Meta.Side.LEFT; - else if (y1 <= monitor.y) + } else if (y1 <= monitor.y) { side = Meta.Side.TOP; - else if (x2 >= monitor.x + monitor.width) + } else if (x2 >= monitor.x + monitor.width) { side = Meta.Side.RIGHT; - else if (y2 >= monitor.y + monitor.height) + } else if (y2 >= monitor.y + monitor.height) { side = Meta.Side.BOTTOM; - else + } else { continue; + } let strutRect = new Meta.Rectangle({ x: x1, y: y1, width: x2 - x1, height: y2 - y1 }); let strut = new Meta.Strut({ rect: strutRect, side: side }); diff --git a/js/ui/popupMenu.js b/js/ui/popupMenu.js index b66f3c5d1..eb0616706 100644 --- a/js/ui/popupMenu.js +++ b/js/ui/popupMenu.js @@ -718,10 +718,11 @@ var PopupMenuBase = class { this.disconnect(openStateChangeId); menuItem.disconnect(destroyId); }); - } else if (menuItem instanceof PopupBaseMenuItem) + } else if (menuItem instanceof PopupBaseMenuItem) { this._connectItemSignals(menuItem); - else + } else { throw TypeError("Invalid argument to PopupMenuBase.addMenuItem()"); + } menuItem._setParent(this); diff --git a/js/ui/windowManager.js b/js/ui/windowManager.js index 8d5db083b..297278849 100644 --- a/js/ui/windowManager.js +++ b/js/ui/windowManager.js @@ -630,9 +630,9 @@ var AppSwitchAction = GObject.registerClass({ let nPoints = this.get_n_current_points(); let event = this.get_last_event (nPoints - 1); - if (nPoints == 3) + if (nPoints == 3) { this._longPressStartTime = event.get_time(); - else if (nPoints == 4) { + } else if (nPoints == 4) { // Check whether the 4th finger press happens after a 3-finger long press, // this only needs to be checked on the first 4th finger press if (this._longPressStartTime != null &&