First time here, first of all - this is an awesome project!! I'm trying to play ar

<a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="/us

`finally` block isn't executed if `try`/`catch` has control flow statements about espruino HOT 4 CLOSED

idobh2 commented on June 20, 2024

`finally` block isn't executed if `try`/`catch` has control flow statements

from espruino.

Comments (4)

gfwilliams commented on June 20, 2024

Thanks for tracking this down!

It seems that a finally block are only executed if the try/catch block had no control flow statements in them.

Is it any control flow statements, like if/whole/do/for/etc, or literally just return that's not working?

from espruino.

idobh2 commented on June 20, 2024

@gfwilliams It seems that return, break and continue are the problematic statements. I ran tests in the Web IDE using the Bangle.js emulator.
These are examples where finally is not executed:

// Not reaching finally block
(() => {
  try {
    console.log("testing return...");
    return;
  } finally {
    console.log("return success");
  }
})();

(() => {
  do {
    try {
      console.log("testing break...");
      break;
    } finally {
      console.log("break success");
    }
  } while (false);
})();

(() => {
  do {
    try {
      console.log("testing continue...");
      continue;
    } finally {
      console.log("continue success");
    }
  } while (false);
})();

And these are cases where it is executed correctly

// Reaching finally block
(() => {
  try {
    console.log("testing if...");
    let someVariable = 0;
    if (true) {
      someVariable = 1;
    }
    if (false) {
      someVariable = 0;
    }
  } finally {
    console.log("if success");
  }
})();

(() => {
  try {
    console.log("testing while...");
    let i = 0;
    while (i < 1) {
      i++;
    }
  } finally {
    console.log("while success");
  }
})();

(() => {
  try {
    console.log("testing do-while...");
    let i = 0;
    do {
      i++;
    } while (i < 2);
  } finally {
    console.log("do-while success");
  }
})();

(() => {
  try {
    console.log("testing for...");
    for (let i = 0; i < 1; i++) {
    }
  } finally {
    console.log("for success");
  }
})();

(() => {
  try {
    console.log("testing throw...");
    throw new Error();
  } catch (e) {
    throw new Error();
  } finally {
    console.log("throw success");
  }
})();