From cef81b8c23c10f0d347bc2f25b45f9491c76ff98 Mon Sep 17 00:00:00 2001 From: Luciano Iam Date: Mon, 14 Jun 2021 10:27:26 +0200 Subject: [PATCH] WebSockets: minor update in JS code style Use anonymous blocks { } instead of anonymous functions as the outermost scope in some files. Also fix incorrect wording for a comment --- share/web_surfaces/builtin/mixer/js/main.js | 6 +++--- share/web_surfaces/builtin/mixer/js/tkwidget.js | 2 +- share/web_surfaces/builtin/protocol/main.js | 4 ++-- share/web_surfaces/builtin/transport/main.js | 4 ++-- share/web_surfaces/index/main.js | 4 ++-- 5 files changed, 10 insertions(+), 10 deletions(-) diff --git a/share/web_surfaces/builtin/mixer/js/main.js b/share/web_surfaces/builtin/mixer/js/main.js index 9ba686e3c0..d2f0d47b41 100644 --- a/share/web_surfaces/builtin/mixer/js/main.js +++ b/share/web_surfaces/builtin/mixer/js/main.js @@ -22,8 +22,8 @@ import { createRootContainer, Container, Dialog, Label, Button, Toggle, AudioStripGainFader, MidiStripGainFader, AudioStripMeter, MidiStripMeter } from './tkwidget.js'; -(() => { - +{ + const ardour = new ArdourClient(); async function main () { @@ -223,4 +223,4 @@ import { createRootContainer, Container, Dialog, Label, Button, Toggle, main(); -})(); +} diff --git a/share/web_surfaces/builtin/mixer/js/tkwidget.js b/share/web_surfaces/builtin/mixer/js/tkwidget.js index ac1f494fcd..44aa43897a 100644 --- a/share/web_surfaces/builtin/mixer/js/tkwidget.js +++ b/share/web_surfaces/builtin/mixer/js/tkwidget.js @@ -167,7 +167,7 @@ export class Dialog extends BaseDialog { show () { // opening a TK.Dialog with auto_close=true from a TK.Button callback - // fails otherwise ev.stopPropagation() is called in the button event + // fails unless ev.stopPropagation() is called in the button event // handler or setTimeout() is used here setTimeout(() => { this.tk.set('display_state', 'show'); diff --git a/share/web_surfaces/builtin/protocol/main.js b/share/web_surfaces/builtin/protocol/main.js index f8db28ec6f..87341ffb7c 100644 --- a/share/web_surfaces/builtin/protocol/main.js +++ b/share/web_surfaces/builtin/protocol/main.js @@ -18,7 +18,7 @@ import ArdourClient from '/shared/ardour.js'; -(() => { +{ const MAX_LOG_LINES = 1000; @@ -65,4 +65,4 @@ import ArdourClient from '/shared/ardour.js'; main(); -})(); +} diff --git a/share/web_surfaces/builtin/transport/main.js b/share/web_surfaces/builtin/transport/main.js index cfadefa1d6..b9c2c40dc7 100644 --- a/share/web_surfaces/builtin/transport/main.js +++ b/share/web_surfaces/builtin/transport/main.js @@ -18,7 +18,7 @@ import ArdourClient from '/shared/ardour.js'; -(() => { +{ const dom = { main : document.getElementById('main'), @@ -126,4 +126,4 @@ import ArdourClient from '/shared/ardour.js'; main(); -})(); +} diff --git a/share/web_surfaces/index/main.js b/share/web_surfaces/index/main.js index 9569dea45a..a1b4038809 100644 --- a/share/web_surfaces/index/main.js +++ b/share/web_surfaces/index/main.js @@ -18,7 +18,7 @@ import ArdourClient from '/shared/ardour.js'; -(() => { +{ async function main () { try { @@ -81,4 +81,4 @@ import ArdourClient from '/shared/ardour.js'; main(); -})(); +}