Opened 19 years ago
Closed 19 years ago
#420 closed patch (fixed)
UK DVB EIT enhancements
| Reported by: | Owned by: | danielk | |
|---|---|---|---|
| Priority: | minor | Milestone: | 0.19 |
| Component: | mythtv | Version: | head |
| Severity: | low | Keywords: | DVB EIT |
| Cc: | Ticket locked: | no |
Description
More updates to the UK EIT parsing code
The attached patch does the following :
Strips out "...." to join Title - Subtitles up Enables the use of "?" or "!" as a subtitle limiter Removes a hack I'd put in for trying to work out if a subtitle was valid. (now uses a percentage ratio to check if a subtitle is too long) Also fixes a typo in a comment Also changes "Movies" to "Movie" in the category table
Attachments (5)
Change History (11)
Changed 19 years ago by
| Attachment: | uk.eit.patch added |
|---|
comment:1 Changed 19 years ago by
| Type: | defect → patch |
|---|
Changed 19 years ago by
| Attachment: | uk.eit.2.patch added |
|---|
comment:2 Changed 19 years ago by
comment:3 Changed 19 years ago by
Having tested this, the percentage based approach to trying to work out where the subtitles begin and end doesn't work very well. Quite a lot of shows have a short title and a long subtitle. Friends is a prime example. With your patch episodes of friends lose there subtitle in the guide data.
Additionally I added support for dropping T4: from the front of channel 4's shows. I'll attach that patch. Applies on top of uk.eit.3.patch.
Changed 19 years ago by
| Attachment: | uk.eit-no-t4.over-eit-3-patch.diff added |
|---|
Strips leading T4: from titles. Applies over uk.eit.3.patch
Changed 19 years ago by
| Attachment: | uk.eit-no-t4.over-eit-3-patch.2.diff added |
|---|
fixup patch to remove T4 from front of program names
comment:4 Changed 19 years ago by
Ignore what I said before. My bad. I broke it. I fixed it :-(
But at least now my patch works properly...
comment:5 Changed 19 years ago by
| Milestone: | → 0.19 |
|---|---|
| Owner: | changed from Isaac Richards to danielk |
| Severity: | medium → low |
| Status: | new → assigned |
I'll look at this today or next week.
please use uk.eit.2.patch instead (found a minor problem)