[gnome-shell/wip/fmuellner/more-minor-bug-fixes: 7/7] cleanup: Remove erroneous vfunc parameters
- From: Florian Müllner <fmuellner src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-shell/wip/fmuellner/more-minor-bug-fixes: 7/7] cleanup: Remove erroneous vfunc parameters
- Date: Tue, 5 Feb 2019 02:22:14 +0000 (UTC)
commit 15e7625c80738906ec0770c337e5a86564987173
Author: Florian Müllner <fmuellner gnome org>
Date: Thu Jan 31 22:07:38 2019 +0100
cleanup: Remove erroneous vfunc parameters
Unlike in C or signal handlers, vfuncs don't include the this-object
in their arguments.
https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/388
HACKING.md | 6 +++---
js/ui/dateMenu.js | 4 ++--
js/ui/edgeDragAction.js | 6 +++---
js/ui/magnifier.js | 4 ++--
js/ui/panel.js | 2 +-
js/ui/viewSelector.js | 6 +++---
js/ui/windowManager.js | 6 +++---
7 files changed, 17 insertions(+), 17 deletions(-)
---
diff --git a/HACKING.md b/HACKING.md
index d8d940d04..f1b3d3765 100644
--- a/HACKING.md
+++ b/HACKING.md
@@ -158,15 +158,15 @@ you to inherit from a type to use it, you can do so:
var MyClutterActor = GObject.registerClass(
class MyClutterActor extends Clutter.Actor {
- vfunc_get_preferred_width(actor, forHeight) {
+ vfunc_get_preferred_width(forHeight) {
return [100, 100];
}
- vfunc_get_preferred_height(actor, forWidth) {
+ vfunc_get_preferred_height(forWidth) {
return [100, 100];
}
- vfunc_paint(actor) {
+ vfunc_paint() {
let alloc = this.get_allocation_box();
Cogl.set_source_color4ub(255, 0, 0, 255);
Cogl.rectangle(alloc.x1, alloc.y1,
diff --git a/js/ui/dateMenu.js b/js/ui/dateMenu.js
index 5d6938960..a04efb2f3 100644
--- a/js/ui/dateMenu.js
+++ b/js/ui/dateMenu.js
@@ -377,13 +377,13 @@ class IndicatorPad extends St.Widget {
super._init();
}
- vfunc_get_preferred_width(container, forHeight) {
+ vfunc_get_preferred_width(forHeight) {
if (this._source.visible)
return this._source.get_preferred_width(forHeight);
return [0, 0];
}
- vfunc_get_preferred_height(container, forWidth) {
+ vfunc_get_preferred_height(forWidth) {
if (this._source.visible)
return this._source.get_preferred_height(forWidth);
return [0, 0];
diff --git a/js/ui/edgeDragAction.js b/js/ui/edgeDragAction.js
index ec94c1b42..0cad5da34 100644
--- a/js/ui/edgeDragAction.js
+++ b/js/ui/edgeDragAction.js
@@ -30,7 +30,7 @@ var EdgeDragAction = GObject.registerClass({
return global.display.get_monitor_geometry(monitorIndex);
}
- vfunc_gesture_prepare(action, actor) {
+ vfunc_gesture_prepare(actor) {
if (this.get_n_current_points() == 0)
return false;
@@ -46,7 +46,7 @@ var EdgeDragAction = GObject.registerClass({
(this._side == St.Side.BOTTOM && y > monitorRect.y + monitorRect.height - EDGE_THRESHOLD));
}
- vfunc_gesture_progress(action, actor) {
+ vfunc_gesture_progress(actor) {
let [startX, startY] = this.get_press_coords(0);
let [x, y] = this.get_motion_coords(0);
let offsetX = Math.abs (x - startX);
@@ -66,7 +66,7 @@ var EdgeDragAction = GObject.registerClass({
return true;
}
- vfunc_gesture_end(action, actor) {
+ vfunc_gesture_end(actor) {
let [startX, startY] = this.get_press_coords(0);
let [x, y] = this.get_motion_coords(0);
let monitorRect = this._getMonitorRect(startX, startY);
diff --git a/js/ui/magnifier.js b/js/ui/magnifier.js
index 0b480f117..b33c300c6 100644
--- a/js/ui/magnifier.js
+++ b/js/ui/magnifier.js
@@ -62,14 +62,14 @@ var MouseSpriteContent = GObject.registerClass({
this._texture = null;
}
- vfunc_get_preferred_size(content) {
+ vfunc_get_preferred_size() {
if (!this._texture)
return [0, 0];
return [this._texture.get_width(), this._texture.get_height()];
}
- vfunc_paint_content(content, actor, node) {
+ vfunc_paint_content(actor, node) {
if (!this._texture)
return;
diff --git a/js/ui/panel.js b/js/ui/panel.js
index 9b264d9b0..9c1954841 100644
--- a/js/ui/panel.js
+++ b/js/ui/panel.js
@@ -835,7 +835,7 @@ class Panel extends St.Widget {
this._updateSolidStyle();
}
- vfunc_get_preferred_width(actor, forHeight) {
+ vfunc_get_preferred_width(forHeight) {
let primaryMonitor = Main.layoutManager.primaryMonitor;
if (primaryMonitor)
diff --git a/js/ui/viewSelector.js b/js/ui/viewSelector.js
index f19d67b12..3f690982f 100644
--- a/js/ui/viewSelector.js
+++ b/js/ui/viewSelector.js
@@ -81,7 +81,7 @@ var ShowOverviewAction = GObject.registerClass({
});
}
- vfunc_gesture_prepare(action, actor) {
+ vfunc_gesture_prepare(actor) {
return Main.actionMode == Shell.ActionMode.NORMAL &&
this.get_n_current_points() == this.get_n_touch_points();
}
@@ -115,12 +115,12 @@ var ShowOverviewAction = GObject.registerClass({
height: maxY - minY });
}
- vfunc_gesture_begin(action, actor) {
+ vfunc_gesture_begin(actor) {
this._initialRect = this._getBoundingRect(false);
return true;
}
- vfunc_gesture_end(action, actor) {
+ vfunc_gesture_end(actor) {
let rect = this._getBoundingRect(true);
let oldArea = this._initialRect.width * this._initialRect.height;
let newArea = rect.width * rect.height;
diff --git a/js/ui/windowManager.js b/js/ui/windowManager.js
index 8c4cb36d8..2cb8dfb31 100644
--- a/js/ui/windowManager.js
+++ b/js/ui/windowManager.js
@@ -599,7 +599,7 @@ var AppSwitchAction = GObject.registerClass({
});
}
- vfunc_gesture_prepare(action, actor) {
+ vfunc_gesture_prepare(actor) {
if (Main.actionMode != Shell.ActionMode.NORMAL) {
this.cancel();
return false;
@@ -608,7 +608,7 @@ var AppSwitchAction = GObject.registerClass({
return this.get_n_current_points() <= 4;
}
- vfunc_gesture_begin(action, actor) {
+ vfunc_gesture_begin(actor) {
// in milliseconds
const LONG_PRESS_TIMEOUT = 250;
@@ -632,7 +632,7 @@ var AppSwitchAction = GObject.registerClass({
return this.get_n_current_points() <= 4;
}
- vfunc_gesture_progress(action, actor) {
+ vfunc_gesture_progress(actor) {
const MOTION_THRESHOLD = 30;
if (this.get_n_current_points() == 3) {
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]