diff options
author | Kartik K. Agaram <vc@akkartik.com> | 2022-02-02 23:44:25 -0800 |
---|---|---|
committer | Kartik K. Agaram <vc@akkartik.com> | 2022-02-02 23:44:25 -0800 |
commit | a8dfea1d3c686437a29938a29b4b0543d8190267 (patch) | |
tree | f7c8019ba86b35f68af8ec8726e4f860f5acaa31 /src/lmem.h | |
parent | 8f8a0e5a181de799ce95c5d91c0266e0a06671d8 (diff) | |
download | teliva-a8dfea1d3c686437a29938a29b4b0543d8190267.tar.gz |
drop io.lines()
I'd already dropped the variant without a filename. But even the variant with a filename is very easy to use in a way that is confusing to use in the presence of sandboxing: * call io.lines() * Sandboxing saves an error, io.lines() returns nil * Caller (usually a loop) raises an error. * We show the error and not the sandboxing failure. * Worse, there's no way to adjust permissions from within Teliva, because we can't ever get to that menu while there's an error. Best solution I can come up with: encourage a separate step for translating filename to file handle. That way it's more obvious that we need to check for errors.
Diffstat (limited to 'src/lmem.h')
0 files changed, 0 insertions, 0 deletions