EricBerendsen / dvbinspector

DVB Inspector is an open-source DVB analyzer, written in java
http://www.digitalekabeltelevisie.nl/dvb_inspector/
GNU General Public License v3.0
184 stars 31 forks source link

Recent files, problem with file separator in path. #58

Closed avrecko closed 3 months ago

avrecko commented 1 year ago

I have some files that are like this: {IP}:{PORT}.ts e.g. 239.12.12.12:5000.ts

this is a valid filename on Linux and MacOS. Unsure about Windows. On MacOS you need to use command line (cp, mv, etc) to use : in filename. It shows in finder with / instead of : char.

in the recent files 239.12.12.12:5000.ts splits to 239.12.12.12 and 5000.ts.

I've changed the separator to something more unique and very doubtful that could be part of a valid name. Not that original. Wanted to spell DVB. \u001D\u0011\u001B.

So now the recent files work for me with files using : in name.

The only downside is that this change would break the existing recent files list to a one line entry.

EricBerendsen commented 3 months ago

As you say, it would break existing recent files lists. And using a : in a filename is (although allowed on Linux) asking for trouble.

avrecko commented 3 months ago

As you say, it would break existing recent files lists. And using a : in a filename is (although allowed on Linux) asking for trouble.

Yeah, I ended up renaming the files anyway. It does look weird.