TypesettingTools / Myaamori-Aegisub-Scripts

Overengineering is my middle name.
MIT License
37 stars 7 forks source link

Output file has an additional blank line #22

Open Ninelpienel opened 1 year ago

Ninelpienel commented 1 year ago

Hey, when I use subdigest my output file has an additional blank line at the end. Various other programs that work with ass cannot handle this. Would be great if you could fix this.

Input:

image

Output:

image

FichteFoll commented 1 year ago

Subdigest doesn't do anything special in this regard, so this may be a result of the upstream ass library. Haven't tried to reproduce it yet, however.

https://github.com/TypesettingTools/Myaamori-Aegisub-Scripts/blob/f2a52ee38eeb60934175722fa9d7f2c2aae015c6/scripts/sub-digest/subdigest.py#L565-L566

Funami580 commented 1 year ago

Just a guess, but perhaps this line?

https://github.com/chireiden/python-ass/blob/8243bd515531fa8e7180941bfc0fbac7fb24702f/ass/document.py#L152

FichteFoll commented 1 year ago

Yeah, that should be it. Called from dubdigest via

https://github.com/TypesettingTools/Myaamori-Aegisub-Scripts/blob/f2a52ee38eeb60934175722fa9d7f2c2aae015c6/scripts/sub-digest/subdigest.py#L512-L517

Subdigest would also do better in using the dump_file method with the file it wants to write to directly instead of always writing to a StringIO first and then writing that. Catching the warning may be necessary for writing to stdout, though iirc we fixed overreporting with that warning a while ago. Could also be that it only occurs on Windows, though.

FichteFoll commented 1 year ago

Fixed in https://github.com/chireiden/python-ass/releases/tag/0.5.3.