about summary refs log tree commit diff stats
path: root/src/lstate.h
diff options
context:
space:
mode:
authorKartik K. Agaram <vc@akkartik.com>2022-02-02 23:44:25 -0800
committerKartik K. Agaram <vc@akkartik.com>2022-02-02 23:44:25 -0800
commita8dfea1d3c686437a29938a29b4b0543d8190267 (patch)
treef7c8019ba86b35f68af8ec8726e4f860f5acaa31 /src/lstate.h
parent8f8a0e5a181de799ce95c5d91c0266e0a06671d8 (diff)
downloadteliva-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/lstate.h')
0 files changed, 0 insertions, 0 deletions