Fix typo in parsing HDR10+ files #147

Merged

mplewis merged 1 commit from patch-1 into master

2023-09-11 18:23:34 +00:00

mplewis commented

2023-09-10 22:22:18 +00:00

(Migrated from github.com)

Hello! I noticed that HDR10+ is misspelled as HRD10+. I suspect this is leading to some files being misclassified, so I wanted to suggest this fix.

I also noticed that some torrents have HDR10Plus in their names rather than the + sign, for example: Star.Trek.Strange.New.Worlds.S02E05.Sciarada.2160p.DVHDR10Plus.HEVC.WEBDL.AC3.ITA.ENG.SUB.G66.mkv. It might be a good idea to add HDR10Plus to this parser.

Thanks for all your hard work!

Hello! I noticed that `HDR10+` is misspelled as `HRD10+`. I suspect this is leading to some files being misclassified, so I wanted to suggest this fix. I also noticed that some torrents have `HDR10Plus` in their names rather than the `+` sign, for example: `Star.Trek.Strange.New.Worlds.S02E05.Sciarada.2160p.DVHDR10Plus.HEVC.WEBDL.AC3.ITA.ENG.SUB.G66.mkv`. It might be a good idea to add `HDR10Plus` to this parser. Thanks for all your hard work!

TheBeastLT (Migrated from github.com) approved these changes 2023-09-11 18:23:29 +00:00

TheBeastLT (Migrated from github.com) left a comment

Thanks!

Regarding Star.Trek.Strange.New.Worlds.S02E05.Sciarada.2160p.DVHDR10Plus.HEVC.WEBDL.AC3.ITA.ENG.SUB.G66.mkv - HDR10Plus is supported as a modifier, but this title doesn't have it at word boundary as probably it's a typo. It probably should've been DV.HDR10Plus to imply it supports both DV and HDR10+, but tbh this is the first time I'm seeing such naming and unless it's a common case I probably won't try to correct it and detect such typos.

Thanks! Regarding `Star.Trek.Strange.New.Worlds.S02E05.Sciarada.2160p.DVHDR10Plus.HEVC.WEBDL.AC3.ITA.ENG.SUB.G66.mkv` - `HDR10Plus` is supported as a modifier, but this title doesn't have it at word boundary as probably it's a typo. It probably should've been `DV.HDR10Plus` to imply it supports both `DV` and `HDR10+`, but tbh this is the first time I'm seeing such naming and unless it's a common case I probably won't try to correct it and detect such typos.

mplewis commented

2023-09-12 02:23:01 +00:00

(Migrated from github.com)

Thanks for the clarification – I didn't even see the DV typo prefix there :)

Thanks for the clarification – I didn't even see the `DV` typo prefix there :)