respond to review comment

Author: microlith57

Celeste.Mod.mm/Patches/Level.cs

@@ -595,24 +595,25 @@ public override void End() {
 
             Scene nextScene = patch_Engine.NextScene;
 
+            // whether to reload the vanilla Portraits.xml when exiting.
+            // if a map overrides the Portraits.xml and doesn't have portrait_madeline defined, the game would
+            // crash when trying to load a save file (since it shows madeline's portrait)
+            //
+            // however we need to pay attention to the new scene, else we'll reload vanilla portraits too soon
+            // and make custom portraits stop working. therefore, we ignore these scenes as they don't actually
+            // exit the map
             bool shouldReloadPortraits = nextScene is not (LevelLoader or Pico8.Emulator or OverworldReflectionsFall);
+
+            // whether to break all links between this level and its entities.
+            // similarly, if the scene we're entering shouldn't count as exiting the map, then ignore this.
             bool shouldDissociateEntities = nextScene is not (Pico8.Emulator or OverworldReflectionsFall);
 
             Everest.Events.Level.End(this, nextScene, ref shouldReloadPortraits, ref shouldDissociateEntities);
 
-            // reload the vanilla Portraits.xml when exiting; if a map overrides the Portraits.xml
-            // and doesn't have portrait_madeline defined, the game would crash when trying to load a save file
-            // (since it shows madeline's portrait)
-            //
-            // however we need to pay attention to the new scene, else we'll reload vanilla portraits too soon
-            // and make custom portraits stop working. therefore, we ignore these scenes as they don't actually
-            // exit the map
             if (shouldReloadPortraits)
                 GFX.PortraitsSpriteBank = new SpriteBank(GFX.Portraits, Path.Combine("Graphics", "Portraits.xml"));
 
-            // if we are not entering PICO-8, the Reflection Fall cutscene, or some similar modded scene...
             if (shouldDissociateEntities) {
-                // break all links between this level and its entities.
                 foreach (Entity entity in Entities)
                     ((patch_Entity) entity).DissociateFromScene();