Ignoring numbers in HI removal?

SubtitleEdit / subtitleedit

the subtitle editor :)

http://www.nikse.dk/SubtitleEdit/Help

GNU General Public License v3.0

8.95k stars 918 forks source link

Ignoring numbers in HI removal? #9052

Open Boulder08 opened 2 days ago

Boulder08 commented 2 days ago

Is it possible to make SE ignore two corner cases like the ones highlighted in this example?

ivandrofly commented 1 day ago

This is not related with remove HI, the time is not well formatted! There is white space between "11" and ":".

Added ignore logic will allow more buggy text.

Maybe a rule need to be added where (maybe in FCE) to handle this case.

Boulder08 commented 1 day ago

I was thinking along the lines of ignoring anything after the first ":" (line 214) and also ignoring cases where there are only numbers before the character like on line 218.

ivandrofly commented 1 day ago

Just opened a pull-request with the fixes. If merge happen you will still need to run the Fix Common Errors (FCE) with option Remove unneeded spaces ticked to handle invalid time and then run Remove text from hearing impaired