search

xiph / awcy

http://arewecompressedyet.com/
MIT License
70 stars 46 forks source link

awcy: replace all spaces in run name with _ #346

Closed tmatth closed 1 year ago

tmatth commented 1 year ago

Before this change, only the first was being replaced. Fixes #27 and #197.

tmatth commented 1 year ago

Marked as draft as I haven't had the chance to test yet, if anyone can easily build+run+validate the fix it'd be appreciated.

tmatth commented 1 year ago

Thanks for the patch,

Tested locally in VLC12,

image image image

It works :)

Edit: http://vlc12.xiph.osuosl.org:3000/?job=testing_locally_a_broken_run_without_commit Can test here,

Thanks for testing, I pushed a minor tweak so that any whitespace will get replaced (not just space characters).