kannagi0303 / yt-dlp-gui

Windows GUI for yt-dlp

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Fixing decimal.TryParse issue

amaillo-coop opened this issue · comments

GUI crash almost at finish download because Util.Tools.SecToStr() throw exception due to a Main.GetStatus() wrong decimal.TryParse() interpretation of a decimal number because my OS culture (es-ES) uses comma as decimal separator instead of a dot and yt-dlp std output is always showing decimal numbers with en-US number fomatting.

Fixing it's really simple, just force culture to "en-US" in every decimal.TryParse() on Main.GetStatus():
image

I can send you a PR with this fix, just text me.

Hope this could help.
Best regards!

I had the same problem, made a pull request #147, but I only fixed the last 2 TryParse statements, I forgot the other ones in my sleep deprived state when I fixed it.... Will do so and amend the pull request tonight or tomorrow.

And done..... :)