search

catalinii / minisatip

minisatip is an SATIP server for linux using local DVB-S2, DVB-C, DVB-T or ATSC cards
https://minisatip.org
328 stars 81 forks source link

Fix bugs in "-o" parameter parsing, factor out and add tests #1105

Closed Jalle19 closed 1 year ago

Jalle19 commented 1 year ago

This is against next but should be backported to master as well.

@uwe5 are you able to test this?

@catalinii I only factored out parsing of this particular parameter here, but IMO all but the most trivial parameters should get the same treatment so we can write tests and verify the parsing actually works. Especially options with multiple optional prefixes/suffixes etc. are really hard to verify just by looking at the code.

@lars18th feel free to test as well, you were the last one who touched this

Jalle19 commented 1 year ago

@uwe5 have you had a chance to test this?