hiro/tridactyl
1
0
Fork 0
mirror of https://github.com/vale981/tridactyl synced 2025-03-06 01:51:40 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

commandline_frame.ts: Fix compile-time error

Browse source 
10cb692d72 broke the build bot by
introducing type incompatibilites. This is fixed by making sure .catch()
returns an array.
This enables removing the try/catch in the input event handler since
the await won't throw anymore.
This commit is contained in:
glacambre 2019-02-12 12:43:53 +01:00
parent ec43f7bb1c
commit f119a89ac5
No known key found for this signature in database
GPG key ID: B9625DB1767553AC
1 changed files with 6 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
src/commandline_frame.ts
View file

@ -250,7 +250,10 @@ export function refresh_completions(exstr) {
if (!activeCompletions) enableCompletions()
return Promise.all(
activeCompletions.map(comp => comp.filter(exstr).then(resizeArea)),
).catch(err => console.error(err)) // We can't use the regular logging mechanism because the user is using the command line.
).catch(err => {
console.error(err)
return []
}) // We can't use the regular logging mechanism because the user is using the command line.
}
/** @hidden **/
@ -260,14 +263,8 @@ clInput.addEventListener("input", () => {
const exstr = clInput.value
// Schedule completion computation. We do not start computing immediately because this would incur a slow down on quickly repeated input events (e.g. maintaining <Backspace> pressed)
setTimeout(async () => {
try {
// Make sure the previous computation has ended
await onInputPromise
} catch (e) {
// we don't actually care because this is the previous computation, which we will throw away
logger.warning(e)
}
// Make sure the previous computation has ended
await onInputPromise
// If we're not the current completion computation anymore, stop
if (exstr != clInput.value) return