beautify d+c

todo: pre-commit hook

macros/garret.dupe-and-comment.lua

@@ -1,5 +1,6 @@
-script_name="Dupe and Comment"
-script_description="Copies a line and comments out the original.\nbecause i like seeing the original while editing, and being able to go back to it easily"
+script_name = "Dupe and Comment"
+script_description =
+    "Copies a line and comments out the original.\nbecause i like seeing the original while editing, and being able to go back to it easily"
 script_author = "garret"
 script_version = "2.2.0"
 script_namespace = "garret.dupe-and-comment"
@@ -8,52 +9,52 @@ local haveDepCtrl, DependencyControl, depctrl = pcall(require, "l0.DependencyCon
 local util, json
 
 if haveDepCtrl then
-    depctrl = DependencyControl {
+    depctrl = DependencyControl({
         --feed="TODO",
         {
-            {"aegisub.util"},
-            {"json"}
-        }
-    }
+            { "aegisub.util" },
+            { "json" },
+        },
+    })
     util, json = depctrl:requireModules()
 else
-    util = require 'aegisub.util'
+    util = require("aegisub.util")
     local _
-    _, json = pcall(require, 'json') -- if you have depctrl, you have json, but even if you don't, you might have it anyway so worth checking
+    _, json = pcall(require, "json") -- if you have depctrl, you have json, but even if you don't, you might have it anyway so worth checking
     _ = nil
 end
 
 --inspect = require 'inspect'
 
 function comment(subs, sel)
-    for i=#sel,1,-1 do
-        local line=subs[sel[i]]
+    for i = #sel, 1, -1 do
+        local line = subs[sel[i]]
         local dupe = util.copy(line)
         line.comment = false -- going to edit it, so we probably want to see it on the video
         dupe.comment = true -- this is the actual original one
-        subs.insert(sel[i]+1,dupe) -- putting it on the next line so i don't have to change line
+        subs.insert(sel[i] + 1, dupe) -- putting it on the next line so i don't have to change line
     end
     aegisub.set_undo_point(script_name)
 end
 
 function undo(subs, sel)
-    for i=#sel,1,-1 do
-        local edit=subs[sel[i]]
-        local original=subs[sel[i]+1]
+    for i = #sel, 1, -1 do
+        local edit = subs[sel[i]]
+        local original = subs[sel[i] + 1]
         if edit.comment == false and original.comment == true then
             original.comment = false
-            subs[sel[i]+1] = original
+            subs[sel[i] + 1] = original
             subs.delete(sel[i])
         end
     end
-    aegisub.set_undo_point("Undo "..script_name)
+    aegisub.set_undo_point("Undo " .. script_name)
 end
 
 local function hide(subs, sel)
-    for i=#sel,1,-1 do
-        local orig=subs[sel[i]]
+    for i = #sel, 1, -1 do
+        local orig = subs[sel[i]]
         if orig.comment then
-            edit = subs[sel[i]-1]
+            edit = subs[sel[i] - 1]
             edit.extra.dupencom = json.encode(orig)
             subs[sel[i] - 1] = edit
             subs.delete(sel[i])
@@ -65,16 +66,16 @@ local function hide(subs, sel)
 end
 
 local function unhide(subs, sel)
-    for i=#sel,1,-1 do
+    for i = #sel, 1, -1 do
         local j = 0
         while true do
-            local edit=subs[sel[i] + j]
+            local edit = subs[sel[i] + j]
             if edit.extra.dupencom ~= nil then
                 original = json.decode(edit.extra.dupencom)
                 edit.extra.dupencom = nil
-                subs[sel[i]+j] = edit
+                subs[sel[i] + j] = edit
                 j = j + 1
-                subs.insert(sel[i]+j, original)
+                subs.insert(sel[i] + j, original)
             else
                 break
             end
@@ -84,21 +85,21 @@ local function unhide(subs, sel)
 end
 
 local macros = {
-    {"Do", script_description, comment},
-    {"Undo","Deletes selected line and restores the original", undo},
+    { "Do", script_description, comment },
+    { "Undo", "Deletes selected line and restores the original", undo },
 }
 
-if json ~= nil then  -- dont lock out core functionality because you dont have this one thing
-        table.insert(macros, {"Cleanup/Do","Hides the original in extradata", hide})
-        table.insert(macros, {"Cleanup/Undo","Restores the original from extradata", unhide})
+if json ~= nil then -- dont lock out core functionality because you dont have this one thing
+    table.insert(macros, { "Cleanup/Do", "Hides the original in extradata", hide })
+    table.insert(macros, { "Cleanup/Undo", "Restores the original from extradata", unhide })
 end
 
 if haveDepCtrl then
     depctrl:registerMacros(macros)
 else
-    for _,macro in ipairs(macros) do
+    for _, macro in ipairs(macros) do
         local name, desc, fun = unpack(macro)
-        aegisub.register_macro(script_name .. '/' .. name, desc, fun)
+        aegisub.register_macro(script_name .. "/" .. name, desc, fun)
     end
 end
 -- i trust what petzku from the cartel has to say on adding multiple macros with depctrl