Gather my thoughts

1 files changed, 9 insertions, 11 deletions

diff --git a/npc/commands/event.txt b/npc/commands/event.txt
index 709f26363..080f9315a 100644
--- a/npc/commands/event.txt
+++ b/npc/commands/event.txt
@@ -464,18 +464,16 @@ OnEffect:
     } else {
         setarray .@choices$, l("Cancel"), "-1";
 
-        // Not sure which IDs to use here, so I just encode map ID.
-        // First digit is 1 to avoid octal interpretation
-        // First part is encoded with 3 digits,
-        // Second+ part of id gets encoded with 2 digits.
-        // Note: maybe better to add some more stuff to TP_* constants
-        // and reuse those?
-        // Best would be to pass warp X & Y coordinates as well, to avoid
-        // more conditionals down below and potential desyncs in them.
-        // One idea is to printf and use scanf to extra the stuff
-        // but then tower would need special handling as well.
+        // I'm not sure which IDs to use for the switch after the menu,
+        // so I just encode map ID:
+        //   First digit is 1 to avoid octal interpretation
+        //   First part is encoded with 3 digits,
+        //   Second+ part of id gets encoded with 2 digits.
+        // Passing map, x, y via printf & scanf would have worked, but
+        // would probably not have saved much space. And some events
+        // have warpto functions as they need map setup first.
         // Note: cannot use menuint & rif as I want for the char to go
-        // without menu if there is only one option.
+        //       without menu if there is only one option.
         // Use of labels would also help, but nobody likes label spaghetti?
 
         if ($@MK_SCENE == MK_SIEGE_TULIM || BaseLevel > 20) {