Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | dupe and comment: add notes about what's going on in undo function | garret | 2023-01-15 | 1 | -3/+11 |
| | | | | | | not as thorough as `do` because most of `undo` is pretty clear on its own, and all of the "magic" bits have been explained in `do`. The only difference is that here it's the other way round. | ||||
* | dupe and comment: format and comment new "do" implementation | garret | 2023-01-15 | 1 | -5/+14 |
| | | | | | | | | | | | | | | | | | | | | | | 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. | ||||
* | dupe and comment: fix sel and act | Akatsumekusa | 2023-01-15 | 1 | -11/+10 |
| | | | | | | | selected lines and active line will now be properly set after do and undo removed aegisub.util from requiredModules | ||||
* | dupe and comment: fix out of range error in undo | garret | 2022-12-24 | 1 | -6/+8 |
| | |||||
* | dupe and comment: variable names that make sense | garret | 2022-12-15 | 1 | -4/+4 |
| | |||||
* | dupe and comment: use local functions | garret | 2022-11-24 | 1 | -3/+3 |
| | |||||
* | add depctrl thing | garret | 2021-10-09 | 1 | -0/+55 |
no feed yet, will get round to it eventuallyâ„¢ |