hashirama/videoclip
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

🎨 Increased scope of error handling

Browse source 
The previous error handler used Windows-specific error codes
This commit is contained in:
TheColorman 2023-06-19 15:00:28 +02:00
parent d49c404816
commit c18bfe13f7
No known key found for this signature in database
GPG key ID: A9C799E05AC900AF
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
videoclip.lua
View file

@ -456,7 +456,7 @@ function main_menu:upload_litterbox()
notify("Uploading to " .. (config.litterbox and "litterbox.catbox.moe..." or "catbox.moe..."), "info", 9999) notify("Uploading to " .. (config.litterbox and "litterbox.catbox.moe..." or "catbox.moe..."), "info", 9999)
-- This uses cURL to send a request to the cat-/litterbox API. -- This uses cURL to send a request to the cat-/litterbox API.
-- (cURL is included with Windows 10 and up) -- cURL is included on Windows 10 and up, most Linux distributions and macOS.
local r = mp.command_native({ -- This is technically blocking, but I don't think it has any real consequences ..? local r = mp.command_native({ -- This is technically blocking, but I don't think it has any real consequences ..?
name = 'subprocess', name = 'subprocess',
@ -472,9 +472,9 @@ function main_menu:upload_litterbox()
} }
}) })
-- This really only happens for people that should have upgraded their system years ago. -- Exit codes in the range [0, 99] are returned by cURL itself.
-- Or people running a minimal installation i guess. -- Any other exit code means the shell failed to execute cURL.
if r.status == -3 then if r.status < 0 or r.status > 99 then
notify("Error: Failed to upload. Make sure cURL is installed and in your PATH.", "error", 3) notify("Error: Failed to upload. Make sure cURL is installed and in your PATH.", "error", 3)
return return
end end