From 7d1cd576d1d1d0c11dcad5fc75bf549bb81243de Mon Sep 17 00:00:00 2001
From: Thomas Goyne <plorkyeran@aegisub.org>
Date: Wed, 5 Mar 2014 15:57:31 -0800
Subject: [PATCH] Match the parser's section order when constructing files for
 undo

---
 aegisub/src/subs_controller.cpp | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/aegisub/src/subs_controller.cpp b/aegisub/src/subs_controller.cpp
index b398aefb4..c176c8f66 100644
--- a/aegisub/src/subs_controller.cpp
+++ b/aegisub/src/subs_controller.cpp
@@ -128,6 +128,10 @@ struct SubsController::UndoInfo {
 			c->ass->Line.push_back(*new AssInfo(info.first, info.second));
 		for (auto const& style : styles)
 			c->ass->Line.push_back(*new AssStyle(style));
+		for (auto const& attachment : fonts)
+			c->ass->Line.push_back(*new AssAttachment(attachment));
+		for (auto const& attachment : graphics)
+			c->ass->Line.push_back(*new AssAttachment(attachment));
 		for (auto const& event : events) {
 			auto copy = new AssDialogue(event);
 			c->ass->Line.push_back(*copy);
@@ -136,10 +140,6 @@ struct SubsController::UndoInfo {
 			if (binary_search(begin(selection), end(selection), copy->Id))
 				new_sel.insert(copy);
 		}
-		for (auto const& attachment : graphics)
-			c->ass->Line.push_back(*new AssAttachment(attachment));
-		for (auto const& attachment : fonts)
-			c->ass->Line.push_back(*new AssAttachment(attachment));
 
 		c->subsGrid->BeginBatch();
 		c->selectionController->SetSelectedSet({ });
-- 
GitLab