From 3ce89e3c80907cce01d26c71ee9925fedf1f4d1d Mon Sep 17 00:00:00 2001 From: Dan Winship Date: Mon, 15 Mar 2010 09:50:05 -0400 Subject: [PATCH] Add missing semicolons noted by jslint Taken from a patch by Lex Hider on the mailing list --- js/ui/altTab.js | 4 ++-- js/ui/appDisplay.js | 14 +++++++------- js/ui/calendar.js | 2 +- js/ui/dash.js | 22 +++++++++++----------- js/ui/genericDisplay.js | 4 ++-- js/ui/lookingGlass.js | 16 ++++++++-------- js/ui/notificationDaemon.js | 6 +++--- js/ui/panel.js | 6 +++--- js/ui/placeDisplay.js | 6 +++--- js/ui/runDialog.js | 2 +- js/ui/search.js | 4 ++-- js/ui/widget.js | 2 +- js/ui/windowAttentionHandler.js | 10 +++++----- js/ui/workspace.js | 2 +- js/ui/workspaceSwitcherPopup.js | 2 +- js/ui/workspacesView.js | 2 +- 16 files changed, 52 insertions(+), 52 deletions(-) diff --git a/js/ui/altTab.js b/js/ui/altTab.js index 865e1f0eb..7c8eb042f 100644 --- a/js/ui/altTab.js +++ b/js/ui/altTab.js @@ -455,7 +455,7 @@ SwitcherList.prototype = { }, addSeparator: function () { - let box = new St.Bin({ style_class: 'separator' }) + let box = new St.Bin({ style_class: 'separator' }); this._separator = box; this._list.add_actor(box); }, @@ -599,7 +599,7 @@ AppIcon.prototype = { this._label = new St.Label({ text: this.app.get_name() }); this.actor.add(this._label, { x_fill: false }); } -} +}; function AppSwitcher(apps) { this._init(apps); diff --git a/js/ui/appDisplay.js b/js/ui/appDisplay.js index fb41bb4ad..b980460b7 100644 --- a/js/ui/appDisplay.js +++ b/js/ui/appDisplay.js @@ -259,9 +259,9 @@ AppSearchResultDisplay.prototype = { return; let children = this._container.get_children(); let targetActor = children[this.selectionIndex]; - this.provider.activateResult(targetActor._delegate.app.get_id()) + this.provider.activateResult(targetActor._delegate.app.get_id()); } -} +}; function BaseAppSearchProvider() { this._init(); @@ -330,7 +330,7 @@ AppSearchProvider.prototype = { expandSearch: function(terms) { log("TODO expand search"); } -} +}; function PrefsSearchProvider() { this._init(); @@ -356,7 +356,7 @@ PrefsSearchProvider.prototype = { controlCenter.launch(); Main.overview.hide(); } -} +}; function AppIcon(app) { this._init(app); @@ -382,7 +382,7 @@ AppIcon.prototype = { this._name.clutter_text.line_alignment = Pango.Alignment.CENTER; box.add_actor(this._name); } -} +}; function AppWellIcon(app) { this._init(app); @@ -588,7 +588,7 @@ AppWellIcon.prototype = { getDragActorSource: function() { return this._icon.icon; } -} +}; Signals.addSignalMethods(AppWellIcon.prototype); function AppIconMenu(source) { @@ -1013,7 +1013,7 @@ WellGrid.prototype = { addItem: function(actor) { this._grid.add_actor(actor); } -} +}; function AppWell() { this._init(); diff --git a/js/ui/calendar.js b/js/ui/calendar.js index 68ec6a9d8..ec4689c2f 100644 --- a/js/ui/calendar.js +++ b/js/ui/calendar.js @@ -16,7 +16,7 @@ function _sameDay(dateA, dateB) { function Calendar() { this._init(); -}; +} Calendar.prototype = { _init: function() { diff --git a/js/ui/dash.js b/js/ui/dash.js index d0e32c668..cb80d4a36 100644 --- a/js/ui/dash.js +++ b/js/ui/dash.js @@ -112,7 +112,7 @@ Pane.prototype = { else this.open(); } -} +}; Signals.addSignalMethods(Pane.prototype); function ResultArea(displayType, flags) { @@ -131,7 +131,7 @@ ResultArea.prototype = { this.resultsContainer.append(this.display.actor, Big.BoxPackFlags.EXPAND); this.display.load(); } -} +}; // Utility function shared between ResultPane and the DocDisplay in the main dash. // Connects to the detail signal of the display, and on-demand creates a new @@ -187,7 +187,7 @@ ResultPane.prototype = { })); return resultArea.display; } -} +}; function SearchEntry() { this._init(); @@ -390,7 +390,7 @@ SearchResult.prototype = { else this.provider.activateResult(this.metaInfo.id); } -} +}; function OverflowSearchResults(provider) { this._init(provider); @@ -440,7 +440,7 @@ OverflowSearchResults.prototype = { let targetActor = children[this.selectionIndex]; targetActor._delegate.activate(); } -} +}; function SearchResults(searchSystem) { this._init(searchSystem); @@ -607,7 +607,7 @@ SearchResults.prototype = { resultDisplay.activateSelected(); Main.overview.hide(); } -} +}; function MoreLink() { this._init(); @@ -641,7 +641,7 @@ MoreLink.prototype = { this._expander.style_class = 'more-link-expander open'; })); } -} +}; Signals.addSignalMethods(MoreLink.prototype); @@ -655,7 +655,7 @@ BackLink.prototype = { reactive: true }); this.actor.set_child(new St.Bin({ style_class: "section-header-back-image" })); } -} +}; function SectionHeader(title, suppressBrowse) { this._init(title, suppressBrowse); @@ -726,7 +726,7 @@ SectionHeader.prototype = { this.countText.text = countText; } } -} +}; Signals.addSignalMethods(SectionHeader.prototype); @@ -750,7 +750,7 @@ SearchSectionHeader.prototype = { this.actor.connect('clicked', onClick); } -} +}; function Section(titleString, suppressBrowse) { this._init(titleString, suppressBrowse); @@ -766,7 +766,7 @@ Section.prototype = { vertical: true }); this.actor.add(this.content); } -} +}; function Dash() { this._init(); diff --git a/js/ui/genericDisplay.js b/js/ui/genericDisplay.js index b2207408a..f275bf253 100644 --- a/js/ui/genericDisplay.js +++ b/js/ui/genericDisplay.js @@ -240,7 +240,7 @@ Signals.addSignalMethods(GenericDisplayItem.prototype); const GenericDisplayFlags = { DISABLE_VSCROLLING: 1 << 0 -} +}; /* This is a virtual class that represents a display containing a collection of items * that can be filtered with a search string. @@ -698,7 +698,7 @@ GenericDisplay.prototype = { this._selectedIndex = index; if (index < 0) - return + return; // Mark the new item as selected and create its details pane let item = this._findDisplayedByIndex(index); diff --git a/js/ui/lookingGlass.js b/js/ui/lookingGlass.js index 08d191bfd..c5d5051c5 100644 --- a/js/ui/lookingGlass.js +++ b/js/ui/lookingGlass.js @@ -136,7 +136,7 @@ Notebook.prototype = { let vAdjust = tabData.scrollView.vscroll.adjustment; vAdjust.value = vAdjust.upper - vAdjust.page_size; } -} +}; Signals.addSignalMethods(Notebook.prototype); function Result(command, o, index) { @@ -164,7 +164,7 @@ Result.prototype = { padBin.add_actor(line); this.actor.append(padBin, Big.BoxPackFlags.NONE); } -} +}; function ActorHierarchy() { this._init(); @@ -215,7 +215,7 @@ ActorHierarchy.prototype = { let link = children[idx]; this.emit('selection', actor); } -} +}; Signals.addSignalMethods(ActorHierarchy.prototype); function PropertyInspector() { @@ -249,7 +249,7 @@ PropertyInspector.prototype = { this.actor.add_actor(propDisplay); } } -} +}; function Inspector() { this._init(); @@ -311,7 +311,7 @@ Inspector.prototype = { })); Clutter.grab_pointer(eventHandler); } -} +}; Signals.addSignalMethods(Inspector.prototype); @@ -329,13 +329,13 @@ ErrorLog.prototype = { }, _formatTime: function(d){ - function pad(n) { return n < 10 ? '0' + n : n }; + function pad(n) { return n < 10 ? '0' + n : n; } return d.getUTCFullYear()+'-' + pad(d.getUTCMonth()+1)+'-' + pad(d.getUTCDate())+'T' + pad(d.getUTCHours())+':' + pad(d.getUTCMinutes())+':' - + pad(d.getUTCSeconds())+'Z' + + pad(d.getUTCSeconds())+'Z'; }, _renderText: function() { @@ -349,7 +349,7 @@ ErrorLog.prototype = { } this.text.text = text; } -} +}; function Extensions() { this._init(); diff --git a/js/ui/notificationDaemon.js b/js/ui/notificationDaemon.js index c3cb27077..367152ee1 100644 --- a/js/ui/notificationDaemon.js +++ b/js/ui/notificationDaemon.js @@ -21,17 +21,17 @@ const BusIface = { methods: [{ name: 'GetConnectionUnixProcessID', inSignature: 's', outSignature: 'i' }] -} +}; const Bus = function () { this._init(); -} +}; Bus.prototype = { _init: function() { DBus.session.proxifyObject(this, 'org.freedesktop.DBus', '/org/freedesktop/DBus'); } -} +}; DBus.proxifyPrototype(Bus.prototype, BusIface); diff --git a/js/ui/panel.js b/js/ui/panel.js index 7046bec84..6915bc5bd 100644 --- a/js/ui/panel.js +++ b/js/ui/panel.js @@ -39,7 +39,7 @@ const TRAY_BORDER_WIDTH = 0; const HOT_CORNER_ACTIVATION_TIMEOUT = 0.5; -const STANDARD_TRAY_ICON_ORDER = ['keyboard', 'volume', 'bluetooth', 'network', 'battery'] +const STANDARD_TRAY_ICON_ORDER = ['keyboard', 'volume', 'bluetooth', 'network', 'battery']; const STANDARD_TRAY_ICON_IMPLEMENTATIONS = { 'bluetooth-applet': 'bluetooth', 'gnome-volume-control-applet': 'volume', @@ -130,7 +130,7 @@ TextShadower.prototype = { child.allocate(childBox, flags); } } -} +}; /** * AppPanelMenu: @@ -287,7 +287,7 @@ AppPanelMenu.prototype = { this.emit('changed'); } -} +}; Signals.addSignalMethods(AppPanelMenu.prototype); diff --git a/js/ui/placeDisplay.js b/js/ui/placeDisplay.js index da7fe0356..9e3ed87b0 100644 --- a/js/ui/placeDisplay.js +++ b/js/ui/placeDisplay.js @@ -59,7 +59,7 @@ PlaceInfo.prototype = { isRemovable: function() { return false; } -} +}; function PlaceDeviceInfo(mount) { this._init(mount); @@ -99,7 +99,7 @@ PlaceDeviceInfo.prototype = { _removeFinish: function(o, res, data) { this._mount.unmount_finish(res); } -} +}; function PlacesManager() { @@ -581,4 +581,4 @@ PlaceSearchProvider.prototype = { let places = previousResults.map(function (id) { return Main.placesManager.lookupPlaceById(id); }); return this._searchPlaces(places, terms); } -} +}; diff --git a/js/ui/runDialog.js b/js/ui/runDialog.js index e1c26eb8c..7808780be 100644 --- a/js/ui/runDialog.js +++ b/js/ui/runDialog.js @@ -168,7 +168,7 @@ CommandCompleter.prototype = { function RunDialog() { this._init(); -}; +} RunDialog.prototype = { _init : function() { diff --git a/js/ui/search.js b/js/ui/search.js index c25e02ded..bacf33612 100644 --- a/js/ui/search.js +++ b/js/ui/search.js @@ -207,7 +207,7 @@ SearchProvider.prototype = { expandSearch: function(terms) { throw new Error("not implemented"); } -} +}; Signals.addSignalMethods(SearchProvider.prototype); function SearchSystem() { @@ -275,5 +275,5 @@ SearchSystem.prototype = { return results; } -} +}; Signals.addSignalMethods(SearchSystem.prototype); diff --git a/js/ui/widget.js b/js/ui/widget.js index d6fe8fe6d..1cb740f72 100644 --- a/js/ui/widget.js +++ b/js/ui/widget.js @@ -302,7 +302,7 @@ AppsWidgetInfo.prototype = { launch : function() { this._info.launch(); } -} +}; function AppsWidget() { this._init.apply(this, arguments); diff --git a/js/ui/windowAttentionHandler.js b/js/ui/windowAttentionHandler.js index 8e0228b3d..c4aba46ff 100644 --- a/js/ui/windowAttentionHandler.js +++ b/js/ui/windowAttentionHandler.js @@ -77,12 +77,12 @@ WindowAttentionHandler.prototype = { window.connect('notify::title', Lang.bind(this, function(win) { notification.update(this._getTitle(app, win), this._getBanner(app, win), false); })); - window.connect('notify::demands-attention', Lang.bind(this, function() { source.destroy() })); - window.connect('focus', Lang.bind(this, function() { source.destroy() })); - window.connect('unmanaged', Lang.bind(this, function() { source.destroy() })); + window.connect('notify::demands-attention', Lang.bind(this, function() { source.destroy(); })); + window.connect('focus', Lang.bind(this, function() { source.destroy(); })); + window.connect('unmanaged', Lang.bind(this, function() { source.destroy(); })); } -} +}; function Source(sourceId, app, window) { this._init(sourceId, app, window); @@ -106,4 +106,4 @@ Source.prototype = { MessageTray.Source.prototype.clicked.call(this); } -} +}; diff --git a/js/ui/workspace.js b/js/ui/workspace.js index 4d9bf30c4..f1e73699b 100644 --- a/js/ui/workspace.js +++ b/js/ui/workspace.js @@ -1429,7 +1429,7 @@ Workspace.prototype = { // Tests if @win should be shown in the Overview _isOverviewWindow : function (win) { - let tracker = Shell.WindowTracker.get_default() + let tracker = Shell.WindowTracker.get_default(); return tracker.is_window_interesting(win.get_meta_window()); }, diff --git a/js/ui/workspaceSwitcherPopup.js b/js/ui/workspaceSwitcherPopup.js index 11ceebbf0..274c7dd1c 100644 --- a/js/ui/workspaceSwitcherPopup.js +++ b/js/ui/workspaceSwitcherPopup.js @@ -92,7 +92,7 @@ WorkspaceSwitcherPopup.prototype = { Tweener.addTween(this._container, { opacity: 0.0, time: ANIMATION_TIME, transition: "easeOutQuad", - onComplete: function() { this.actor.hide() }, + onComplete: function() { this.actor.hide(); }, onCompleteScope: this }); } diff --git a/js/ui/workspacesView.js b/js/ui/workspacesView.js index f87211ced..64d444d1a 100644 --- a/js/ui/workspacesView.js +++ b/js/ui/workspacesView.js @@ -262,7 +262,7 @@ GenericWorkspacesView.prototype = { _addWorkspaceActor: function() { throw new Error("Not implemented"); } -} +}; function MosaicView(width, height, x, y, animate) { this._init(width, height, x, y, animate);