Skip to content

Commit 5ce5f12

Browse files
committed
Don't dump additional empty line at the end
Fixes TypesettingTools/Myaamori-Aegisub-Scripts#22.
1 parent 8243bd5 commit 5ce5f12

File tree

2 files changed

+8
-3
lines changed

2 files changed

+8
-3
lines changed

ass/document.py

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -147,6 +147,11 @@ def dump_file(self, f):
147147
warnings.warn("It is recommended to write UTF-8 with BOM"
148148
" using the '%s' encoding" % self.PREFERRED_ENCODING.name)
149149

150+
first = True
150151
for section in self.sections.values():
151-
f.write("\n".join(section.dump()))
152-
f.write("\n\n")
152+
if not first:
153+
f.write("\n")
154+
for line in section.dump():
155+
f.write(line)
156+
f.write("\n")
157+
first = False

tests/test_ass.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ def test_parse_dump(self):
2323
out = StringIO()
2424
doc.dump_file(out)
2525

26-
assert out.getvalue().strip() == contents.strip()
26+
assert out.getvalue() == contents
2727

2828
def test_parse_encoding(self):
2929
with self.test_ass.open("r", encoding='utf_8') as f:

0 commit comments

Comments
 (0)