For me, the remaining question is: should we merge:
https://github.com/mixxxdj/mixxx/pull/771
What are the real world use cases?
Without the merge, the user has full control, but may hit pitfalls if he tries to use unescaped strings.
according to https://gist.github.com/ePirat/adc3b8ba00d85b7e3870#specifying-mountpoint-information
samplerate=44100;quality=10.0;channels=2
should become
samplerate=44100;quality=10%2e0;channels=2
but with the patch, it becomes
samplerate%3D44100%3Bquality%3D10.0%3Bchannels%3D2
Is this an issue?
If yes, the user is not able to fix it, after the applying the patch.
For me, the remaining question is: should we merge:
https:/ /github. com/mixxxdj/ mixxx/pull/ 771
What are the real world use cases?
Without the merge, the user has full control, but may hit pitfalls if he tries to use unescaped strings.
according to /gist.github. com/ePirat/ adc3b8ba00d85b7 e3870#specifyin g-mountpoint- information
https:/
samplerate= 44100;quality= 10.0;channels= 2
should become
samplerate= 44100;quality= 10%2e0; channels= 2
but with the patch, it becomes
samplerate% 3D44100% 3Bquality% 3D10.0% 3Bchannels% 3D2
Is this an issue?
If yes, the user is not able to fix it, after the applying the patch.