maki-rxrz / Caption2Ass_PCR

[DL] https://1drv.ms/f/s!AtWZll0n7FiGjGj45fo564dwG5AW?e=FlPLXG
61 stars 13 forks source link

Caption2Ass_PCR: Two line caption issue #5

Closed maki-rxrz closed 5 years ago

maki-rxrz commented 6 years ago

【概要】 2行に渡って表示される字幕データの内、2行目の字幕の表示位置を間違った値で出力するケースが存在する。

【発生バージョン】 Caption2Ass_PCR 0,2,0,1 以降。

【状況】 Caption.dllから得るデータで、字幕2行目の表示位置の値が0で返される場合があり、 Caption2Ass_PCR側の表示位置算出に間違った計算処理を適応し、動画の範囲外となる値を出力してしまう。

【対応】 Caption.dll の処理を確認中。 一時的な処置としてCaption2Ass_PCR側に2行目の表示位置が無効な値(0)だった場合に補完する処理を追加。 b6b1e2c8e839d29e14b3df6c369354414c7f9a7b

-- (追記) 上記コミットではNG。

maki-rxrz commented 5 years ago

以下のコミットにて修正。 https://github.com/maki-rxrz/Caption2Ass_PCR/commit/a34e493ae1da80a88ffa38363dba210138576d79

-- (追記) 本修正を含め、Caption2Ass_PCR/Caption.dll ともにコード全般でのリファクタリングが必要。 別途検討事項とする。

maki-rxrz commented 5 years ago

(追記) a34e493ae1da80a88ffa38363dba210138576d79 にて、2行目の表示位置に前行の情報を参照してしまうデグレを発生させていた。

以下のコミットで修正。 e8b0804289ac15b38d0ace8e2e6346fa7e902d56