From 4b18f3119def906dad2940c36b0cdf2fb8e7f188 Mon Sep 17 00:00:00 2001 From: filip mertens Date: Thu, 22 Sep 2022 11:25:06 +0200 Subject: [PATCH] smarter disabling --- .../services/code-parser-antlr-service.ts | 20 +++++++++++-------- 1 file changed, 12 insertions(+), 8 deletions(-) diff --git a/apps/remix-ide/src/app/plugins/parser/services/code-parser-antlr-service.ts b/apps/remix-ide/src/app/plugins/parser/services/code-parser-antlr-service.ts index f8f8ba0de5..705ff6c2eb 100644 --- a/apps/remix-ide/src/app/plugins/parser/services/code-parser-antlr-service.ts +++ b/apps/remix-ide/src/app/plugins/parser/services/code-parser-antlr-service.ts @@ -30,9 +30,9 @@ export default class CodeParserAntlrService { text: string, ast: antlr.ParseResult | null, duration?: number, - blockDuration?: number, parsingEnabled?: boolean, - blocks?: BlockDefinition[] + blocks?: BlockDefinition[], + blockDurations?: number[] } } = {}; constructor(plugin: CodeParser) { @@ -52,15 +52,15 @@ export default class CodeParserAntlrService { switch (ev.data.cmd) { case 'parsed': if (ev.data.ast && self.parserStartTime === ev.data.timestamp) { - self.setFileParsingState(ev.data.file, ev.data.blockDuration) self.cache[ev.data.file] = { ...self.cache[ev.data.file], text: ev.data.text, ast: ev.data.ast, duration: ev.data.duration, - blockDuration: ev.data.blockDuration, blocks: ev.data.blocks, + blockDurations: self.cache[ev.data.file].blockDurations? [...self.cache[ev.data.file].blockDurations.slice(-3), ev.data.blockDuration]: [ev.data.blockDuration] } + self.setFileParsingState(ev.data.file) } break; } @@ -68,16 +68,20 @@ export default class CodeParserAntlrService { }); } - setFileParsingState(file: string, duration: number) { + setFileParsingState(file: string) { if (this.cache[file]) { - if (this.cache[file].blockDuration) { - if (this.cache[file].blockDuration > this.parserTreshHold && duration > this.parserTreshHold) { + if (this.cache[file].blockDurations && this.cache[file].blockDurations.length > 3) { + // calculate average of durations to determine if the parsing should be disabled + const values = [...this.cache[file].blockDurations] + const average = values.reduce((a, b) => a + b, 0) / values.length + if (average > this.parserTreshHold) { this.cache[file].parsingEnabled = false - this.plugin.call('notification', 'toast', `This file is big so some autocomplete features will be disabled.`) + this.plugin.call('notification', 'toast','Some autocomplete features will be disabled temporarily because the file takes too long to process.') } else { this.cache[file].parsingEnabled = true } + } } }