| Commit message (Collapse) | Author | Age | Files | Lines | 
| |  | 
 | 
| | 
| 
| 
| 
| 
| 
|  | 
it no longer cares whether the edit is a comment or not.
it now sets the original to whatever the edit was.
better for doing stuff with comments etc
 | 
| |  | 
 | 
| |  | 
 | 
| |  | 
 | 
| |  | 
 | 
| | 
| 
| 
| 
|  | 
fix inconsistent indentation with the rest of the script, and itself
made it a local function
 | 
| | 
| 
| 
| 
| 
|  | 
though apparently it's a bit faster
Co-Authored-By: Akatsumekusa <Akatsumekusa@protonmail.com>
 | 
| |\  
| | 
| | 
| |  | 
Closes #5
 | 
| | |  | 
 | 
| | | 
| | 
| | 
| | 
| | 
| |  | 
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.
 | 
| | | 
| | 
| | 
| | 
| | 
| | 
| | 
| | 
| | 
| | 
| | 
| | 
| | 
| | 
| | 
| | 
| | 
| | 
| | 
| | 
| | 
| |  | 
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.
 | 
| |/  
|   
|   
|   
|   
|   
|    | 
selected lines and active line will
now be properly set after do and undo
removed aegisub.util from requiredModules
 | 
| | 
| 
| 
|  | 
oops
 | 
| |  | 
 | 
| |  | 
 | 
| | 
| 
| 
| 
| 
|  | 
i don't have html any more
nor do i use that directory
i do have ass files at the moment though
 | 
| |  | 
 | 
| | 
| 
| 
|  | 
to get the script back to how it was, but with any changes you've made
 | 
| | 
| 
| 
| 
| 
| 
| 
| 
| 
|  | 
local functions
split a thing off into a function
sane function name
hmm lot of functions this time
also local variables and addressing luacheck warnings
 | 
| |  | 
 | 
| | 
| 
| 
| 
| 
| 
| 
|  | 
>write function specifically to help do a thing
>time passes
>have to do the thing again
>dont use function i wrote
genius
 | 
| |  | 
 | 
| |  | 
 | 
| | 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
|  | 
better than finding a file that's probably the config, parsing the json, and writing it back in,
for obvious reasons
turns out no i wasnt thinning out the soup too much
twas just right
Reason for adding values one by one in key-pair loop instead of just config = new_config:
> [01:31] arch1t3cht: oh, I think you can't assign entire tables to DependencyControl.config.c
> [01:31] arch1t3cht: like, you can't do DependencyControl.config.c = <my new config>,
>                     you should do DependencyControl.config.myfield = <my new value> and whatnot
> [01:32] arch1t3cht: because behind the scenes it uses metatable magic to track what field changes
>                     so it can merge its own changes with ones that other scripts made in the meantime
Co-Authored-By: arch1t3cht <arch1t3cht@gmail.com>
 | 
| |  | 
 | 
| |  | 
 | 
| |  | 
 | 
| |  | 
 | 
| |  | 
 | 
| |  | 
 | 
| |  | 
 | 
| | 
| 
| 
| 
| 
|  | 
no longer rewrites the entire config below the existing one(!)
added forgotten local variables, removed unnecessary ones
fixed a typo
 | 
| |  | 
 | 
| | 
| 
| 
|  | 
hm perhaps i should not have this running on every script load lol
 | 
| |  | 
 | 
| | 
| 
| 
| 
|  | 
holy recursion
also, "cleaned" up a bit
 | 
| |  | 
 | 
| |  | 
 | 
| |  | 
 | 
| |  | 
 | 
| | 
| 
| 
| 
|  | 
apparently people actually use it
This reverts commit 7876e626d73b8010ed63a2423c354f2412f318be.
 | 
| |  | 
 | 
| | 
| 
| 
| 
| 
|  | 
newlines start new comments
"type it yourself" message has been removed
local variables have been remembered this time
 | 
| | 
| 
| 
| 
| 
| 
|  | 
I'm sure there are better layer orderers out there.
I'm not a typesetter, so I'll never actually use mine enough that I fix the glaring issues.
I don't even remember what inverse glow was for any more
 | 
| |  | 
 | 
| | 
| 
| 
|  | 
i have discovered that i actually do have an em-dash key
 | 
| | 
| 
| 
|  | 
oops
 | 
| | 
| 
| 
|  | 
note to self: dont forget this or depctrl will see it as itself
 | 
| |  | 
 |