[gnome-shell] cleanup: Remove empty leading/trailing lines in blocks
- From: Georges Basile Stavracas Neto <gbsneto src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-shell] cleanup: Remove empty leading/trailing lines in blocks
- Date: Mon, 16 Nov 2020 18:08:41 +0000 (UTC)
commit 8a47f1c66720516a1229ce1361b05d5b4b5475c8
Author: Florian Müllner <fmuellner gnome org>
Date: Fri Nov 13 23:52:26 2020 +0100
cleanup: Remove empty leading/trailing lines in blocks
gjs added a new rule to its eslint ruleset that forbids "block padding",
so make sure we conform to that rule before syncing up the configuration.
https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/1498
js/dbusServices/screencast/screencastService.js | 2 --
js/misc/history.js | 2 --
js/misc/introspect.js | 1 -
js/misc/jsParse.js | 1 -
js/misc/smartcardManager.js | 1 -
js/ui/altTab.js | 1 -
js/ui/appDisplay.js | 1 -
js/ui/components/automountManager.js | 1 -
js/ui/components/polkitAgent.js | 1 -
js/ui/dialog.js | 1 -
js/ui/kbdA11yDialog.js | 3 +--
js/ui/lookingGlass.js | 1 -
js/ui/magnifier.js | 1 -
js/ui/messageList.js | 1 -
js/ui/padOsd.js | 1 -
js/ui/status/keyboard.js | 1 -
js/ui/status/network.js | 1 -
js/ui/status/thunderbolt.js | 1 -
js/ui/switcherPopup.js | 1 -
js/ui/userWidget.js | 1 -
js/ui/windowManager.js | 3 ---
js/ui/workspaceSwitcherPopup.js | 1 -
22 files changed, 1 insertion(+), 27 deletions(-)
---
diff --git a/js/dbusServices/screencast/screencastService.js b/js/dbusServices/screencast/screencastService.js
index e98089663e..9ede9b541c 100644
--- a/js/dbusServices/screencast/screencastService.js
+++ b/js/dbusServices/screencast/screencastService.js
@@ -383,7 +383,6 @@ var ScreencastService = class extends ServiceImplementation {
this._removeRecorder(sender);
invocation.return_value(GLib.Variant.new('(bs)', returnValue));
}
-
});
} catch (error) {
log(`Failed to start recorder: ${error.message}`);
@@ -441,7 +440,6 @@ var ScreencastService = class extends ServiceImplementation {
this._removeRecorder(sender);
invocation.return_value(GLib.Variant.new('(bs)', returnValue));
}
-
});
} catch (error) {
log(`Failed to start recorder: ${error.message}`);
diff --git a/js/misc/history.js b/js/misc/history.js
index c82722c5a9..1fba770090 100644
--- a/js/misc/history.js
+++ b/js/misc/history.js
@@ -20,7 +20,6 @@ var HistoryManager = class {
this._history = global.settings.get_strv(this._key);
global.settings.connect(`changed::${this._key}`,
this._historyChanged.bind(this));
-
} else {
this._history = [];
}
@@ -72,7 +71,6 @@ var HistoryManager = class {
addItem(input) {
if (this._history.length == 0 ||
this._history[this._history.length - 1] != input) {
-
this._history = this._history.filter(entry => entry != input);
this._history.push(input);
this._save();
diff --git a/js/misc/introspect.js b/js/misc/introspect.js
index bae7f63736..e46a7e8c59 100644
--- a/js/misc/introspect.js
+++ b/js/misc/introspect.js
@@ -170,7 +170,6 @@ var IntrospectService = class {
for (let app of apps) {
let windows = app.get_windows();
for (let window of windows) {
-
if (!this._isEligibleWindow(window))
continue;
diff --git a/js/misc/jsParse.js b/js/misc/jsParse.js
index c92465fc20..c4e077f626 100644
--- a/js/misc/jsParse.js
+++ b/js/misc/jsParse.js
@@ -201,7 +201,6 @@ function removeLiterals(str) {
// Returns true if there is reason to think that eval(str)
// will modify the global scope
function isUnsafeExpression(str) {
-
// Check for any sort of assignment
// The strategy used is dumb: remove any quotes
// or regexs and comparison operators and see if there is an '=' character.
diff --git a/js/misc/smartcardManager.js b/js/misc/smartcardManager.js
index e1d0f1bce8..d9b6ff4747 100644
--- a/js/misc/smartcardManager.js
+++ b/js/misc/smartcardManager.js
@@ -111,6 +111,5 @@ var SmartcardManager = class {
return true;
}
-
};
Signals.addSignalMethods(SmartcardManager.prototype);
diff --git a/js/ui/altTab.js b/js/ui/altTab.js
index 25683b55da..5332d3b7c4 100644
--- a/js/ui/altTab.js
+++ b/js/ui/altTab.js
@@ -907,7 +907,6 @@ class ThumbnailSwitcher extends SwitcherPopup.SwitcherList {
} else {
this.addItem(box, null);
}
-
}
this.connect('destroy', this._onDestroy.bind(this));
diff --git a/js/ui/appDisplay.js b/js/ui/appDisplay.js
index ab4bb6c863..322aa3cd7a 100644
--- a/js/ui/appDisplay.js
+++ b/js/ui/appDisplay.js
@@ -867,7 +867,6 @@ var PageManager = GObject.registerClass({
global.settings.connect('changed::app-picker-layout',
this._loadPages.bind(this));
-
}
_loadPages() {
diff --git a/js/ui/components/automountManager.js b/js/ui/components/automountManager.js
index 1cb84a7ec5..b83bd22c6a 100644
--- a/js/ui/components/automountManager.js
+++ b/js/ui/components/automountManager.js
@@ -201,7 +201,6 @@ var AutomountManager = class {
e.message.includes('Failed to activate device: Incorrect passphrase') ||
// cryptsetup returns EINVAL in many cases, including wrong TCRYPT password/parameters
e.message.includes('Failed to load device\'s parameters: Invalid argument')) {
-
this._reaskPassword(volume);
} else {
if (e.message.includes('Compiled against a version of libcryptsetup that does not support
the VeraCrypt PIM setting')) {
diff --git a/js/ui/components/polkitAgent.js b/js/ui/components/polkitAgent.js
index ae75f3c00f..e8f3b30f43 100644
--- a/js/ui/components/polkitAgent.js
+++ b/js/ui/components/polkitAgent.js
@@ -238,7 +238,6 @@ var AuthenticationDialog = GObject.registerClass({
/* Yay, all done */
if (gainedAuthorization) {
this._emitDone(false);
-
} else {
/* Unless we are showing an existing error message from the PAM
* module (the PAM module could be reporting the authentication
diff --git a/js/ui/dialog.js b/js/ui/dialog.js
index bb54ac0291..4af154d6f7 100644
--- a/js/ui/dialog.js
+++ b/js/ui/dialog.js
@@ -224,7 +224,6 @@ var MessageDialogContent = GObject.registerClass({
return GLib.SOURCE_REMOVE;
});
}
-
}
set title(title) {
diff --git a/js/ui/kbdA11yDialog.js b/js/ui/kbdA11yDialog.js
index 3cf56b1362..a45e02443d 100644
--- a/js/ui/kbdA11yDialog.js
+++ b/js/ui/kbdA11yDialog.js
@@ -33,8 +33,7 @@ class KbdA11yDialog extends GObject.Object {
: _("Slow Keys Turned Off");
description = _('You just held down the Shift key for 8 seconds. This is the shortcut ' +
'for the Slow Keys feature, which affects the way your keyboard works.');
-
- } else if (whatChanged & Clutter.KeyboardA11yFlags.STICKY_KEYS_ENABLED) {
+ } else if (whatChanged & Clutter.KeyboardA11yFlags.STICKY_KEYS_ENABLED) {
key = KEY_STICKY_KEYS_ENABLED;
enabled = (newFlags & Clutter.KeyboardA11yFlags.STICKY_KEYS_ENABLED) > 0;
title = enabled
diff --git a/js/ui/lookingGlass.js b/js/ui/lookingGlass.js
index 5abee0f752..147e46b2eb 100644
--- a/js/ui/lookingGlass.js
+++ b/js/ui/lookingGlass.js
@@ -205,7 +205,6 @@ var Notebook = GObject.registerClass({
scrollToBottom(index) {
let tabData = this._tabs[index];
tabData._scrollToBottom = true;
-
}
_onAdjustValueChanged(tabData) {
diff --git a/js/ui/magnifier.js b/js/ui/magnifier.js
index 036de7f759..7cfe06fd4d 100644
--- a/js/ui/magnifier.js
+++ b/js/ui/magnifier.js
@@ -1656,7 +1656,6 @@ var ZoomRegion = class ZoomRegion {
var Crosshairs = GObject.registerClass(
class Crosshairs extends Clutter.Actor {
_init() {
-
// Set the group containing the crosshairs to three times the desktop
// size in case the crosshairs need to appear to be infinite in
// length (i.e., extend beyond the edges of the view they appear in).
diff --git a/js/ui/messageList.js b/js/ui/messageList.js
index a79f87bb4e..a5b6249c96 100644
--- a/js/ui/messageList.js
+++ b/js/ui/messageList.js
@@ -291,7 +291,6 @@ var LabelExpanderLayout = GObject.registerClass({
if (child.visible)
child.allocate(box);
}
-
}
});
diff --git a/js/ui/padOsd.js b/js/ui/padOsd.js
index 9ab1cbb9df..110962259d 100644
--- a/js/ui/padOsd.js
+++ b/js/ui/padOsd.js
@@ -673,7 +673,6 @@ var PadOsd = GObject.registerClass({
// the same group.
this._groupPads.splice(this._groupPads.indexOf(device), 1);
this._updatePadChooser();
-
}
});
diff --git a/js/ui/status/keyboard.js b/js/ui/status/keyboard.js
index 43fd89db2e..74c02056fe 100644
--- a/js/ui/status/keyboard.js
+++ b/js/ui/status/keyboard.js
@@ -785,7 +785,6 @@ function getInputSourceManager() {
var InputSourceIndicatorContainer = GObject.registerClass(
class InputSourceIndicatorContainer extends St.Widget {
-
vfunc_get_preferred_width(forHeight) {
// Here, and in vfunc_get_preferred_height, we need to query
// for the height of all children, but we ignore the results
diff --git a/js/ui/status/network.js b/js/ui/status/network.js
index 3d4acd5b08..6b19fad0ec 100644
--- a/js/ui/status/network.js
+++ b/js/ui/status/network.js
@@ -1588,7 +1588,6 @@ var DeviceCategory = class extends PopupMenu.PopupMenuSection {
this._summaryItem.menu.addSettingsAction(_('Network Settings'),
'gnome-network-panel.desktop');
this._summaryItem.hide();
-
}
_sync() {
diff --git a/js/ui/status/thunderbolt.js b/js/ui/status/thunderbolt.js
index d98355deeb..e30a4222fa 100644
--- a/js/ui/status/thunderbolt.js
+++ b/js/ui/status/thunderbolt.js
@@ -77,7 +77,6 @@ var Client = class {
this.probing = this._proxy.Probing;
if (this.probing)
this.emit('probing-changed', this.probing);
-
}
_onPropertiesChanged(proxy, properties) {
diff --git a/js/ui/switcherPopup.js b/js/ui/switcherPopup.js
index 5cc4ab2f2c..b1e2730068 100644
--- a/js/ui/switcherPopup.js
+++ b/js/ui/switcherPopup.js
@@ -487,7 +487,6 @@ var SwitcherList = GObject.registerClass({
this._scrollToRight(index);
else if (this._items[index].allocation.x1 - value < 0)
this._scrollToLeft(index);
-
}
_scrollToLeft(index) {
diff --git a/js/ui/userWidget.js b/js/ui/userWidget.js
index 0f4ed08912..4e72779082 100644
--- a/js/ui/userWidget.js
+++ b/js/ui/userWidget.js
@@ -222,7 +222,6 @@ class UserWidget extends St.BoxLayout {
opacity: 0,
});
this.add_child(this._label);
-
}
this._updateUser();
diff --git a/js/ui/windowManager.js b/js/ui/windowManager.js
index 91b764ea8d..8ea178153b 100644
--- a/js/ui/windowManager.js
+++ b/js/ui/windowManager.js
@@ -357,7 +357,6 @@ var WorkspaceTracker = class {
workspace._windowAddedId = workspace.connect('window-added',
this._queueCheckWorkspaces.bind(this));
workspace._windowRemovedId = workspace.connect('window-removed',
this._windowRemoved.bind(this));
}
-
} else {
// Assume workspaces are only removed sequentially
// (e.g. 2,3,4 - not 2,4,7)
@@ -516,7 +515,6 @@ var AppSwitchAction = GObject.registerClass({
}
vfunc_gesture_progress(_actor) {
-
if (this.get_n_current_points() == 3) {
for (let i = 0; i < this.get_n_current_points(); i++) {
let [startX, startY] = this.get_press_coords(i);
@@ -526,7 +524,6 @@ var AppSwitchAction = GObject.registerClass({
Math.abs(y - startY) > APP_MOTION_THRESHOLD)
return false;
}
-
}
return true;
diff --git a/js/ui/workspaceSwitcherPopup.js b/js/ui/workspaceSwitcherPopup.js
index 4613e96ecf..5b65bec8ad 100644
--- a/js/ui/workspaceSwitcherPopup.js
+++ b/js/ui/workspaceSwitcherPopup.js
@@ -172,7 +172,6 @@ class WorkspaceSwitcherPopup extends St.Widget {
indicator = new St.Bin({ style_class: 'ws-switcher-box' });
this._list.add_actor(indicator);
-
}
let workArea = Main.layoutManager.getWorkAreaForMonitor(Main.layoutManager.primaryIndex);
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]
