From d07a085757dc121ec0ca8c58d17dde8ff6f046e4 Mon Sep 17 00:00:00 2001
From: Andrew Murray <radarhere@users.noreply.github.com>
Date: Tue, 29 Jun 2021 20:20:34 +1000
Subject: [PATCH] Simplified use of struct

---
 src/PIL/IcnsImagePlugin.py | 18 ++++++++----------
 1 file changed, 8 insertions(+), 10 deletions(-)

diff --git a/src/PIL/IcnsImagePlugin.py b/src/PIL/IcnsImagePlugin.py
index 2fec04683..cc5b8affa 100644
--- a/src/PIL/IcnsImagePlugin.py
+++ b/src/PIL/IcnsImagePlugin.py
@@ -331,24 +331,22 @@ def _save(im, fp, filename):
         entries.append(
             {"type": _to_int(size_str[index]), "size": len(stream), "stream": stream}
         )
-    file_size = sum(entry["size"] for entry in entries)
 
     # Header
-    fp.write(struct.pack("<i", _to_int(MAGIC))[::-1])
-    fp.write(struct.pack("<i", file_size)[::-1])
+    fp.write(struct.pack(">i", _to_int(MAGIC)))
+    fp.write(struct.pack(">i", sum(entry["size"] for entry in entries)))
 
     # TOC
-    toc_size = HEADERSIZE + (len(entries) * HEADERSIZE)
-    fp.write(struct.pack("<i", _to_int(TOC))[::-1])
-    fp.write(struct.pack("<i", toc_size)[::-1])
+    fp.write(struct.pack(">i", _to_int(TOC)))
+    fp.write(struct.pack(">i", HEADERSIZE + len(entries) * HEADERSIZE))
     for entry in entries:
-        fp.write(struct.pack("<i", entry["type"])[::-1])
-        fp.write(struct.pack("<i", HEADERSIZE + entry["size"])[::-1])
+        fp.write(struct.pack(">i", entry["type"]))
+        fp.write(struct.pack(">i", HEADERSIZE + entry["size"]))
 
     # Data
     for entry in entries:
-        fp.write(struct.pack("<i", entry["type"])[::-1])
-        fp.write(struct.pack("<i", HEADERSIZE + entry["size"])[::-1])
+        fp.write(struct.pack(">i", entry["type"]))
+        fp.write(struct.pack(">i", HEADERSIZE + entry["size"]))
         fp.write(entry["stream"])
 
     if hasattr(fp, "flush"):