diff options
author | garret <garret@airmail.cc> | 2023-01-15 00:05:27 +0000 |
---|---|---|
committer | garret <garret@airmail.cc> | 2023-01-15 13:26:14 +0000 |
commit | 96bd8086c75576cfd77d83e6122428490869740b (patch) | |
tree | 242d62e6f1cee88683edeb1a6d47b0ecedef9190 /README.md | |
parent | b2f27bd080dd52f1a4e93019701ad4254f5d6176 (diff) | |
download | aegisub-scripts-96bd8086c75576cfd77d83e6122428490869740b.tar.gz aegisub-scripts-96bd8086c75576cfd77d83e6122428490869740b.tar.bz2 aegisub-scripts-96bd8086c75576cfd77d83e6122428490869740b.zip |
dupe and comment: format and comment new "do" implementation
i find it easier to read when the different parts are separated,
and while i have done one-line `if this then do that end`s before,
i've only ever done it for short operations where it's immediately
obvious what's going on and you don't need to do any thinking whatsoever.
e.g. this function from depctrl config:
local function get_log_level(num)
if num == 0 then return "0: Fatal"
elseif num == 1 then return "1: Error"
elseif num == 2 then return "2: Warning"
elseif num == 3 then return "3: Hint"
elseif num == 4 then return "4: Debug"
elseif num == 5 then return "5: Trace" end
return nil
end
I don't think the bits that fix sel and act count, at least not without comments.
speaking of comments, i've added some.
i hope i've understood what's going on correctly, please feel free to correct me if not.
Diffstat (limited to 'README.md')
0 files changed, 0 insertions, 0 deletions