Stream work.

TODO.md

@@ -17,11 +17,13 @@ x Pickup sprites for every angle
 n  - Spool of wire, a tool, and some electical tape
 x Set bulk
 
-- 1st-person sprites
-  - Wire cutters and electrical tape
+x 1st-person sprites
+  n Wire cutters and electrical tape
 
 - Pickup sound
 
-
+- Fault detection sound
+
+- ready animation
 
 

source_data/Makefile

@@ -236,10 +236,12 @@ brightmaps : povsprite.aseprite
 		--layer "brightmap_leds" \
 		--frame-range 0,2 --save-as brightmap_leds.png
 
-	aseprite -b $^ \
-		--layer "brightmap_digits" \
-		--frame-range 0,0 --save-as brightmap_digits.png
-
 	composite brightmap_display.png brightmap_leds1.png -compose screen ../sprites/jmpr_brightmap_1.png
 	composite brightmap_display.png brightmap_leds2.png -compose screen ../sprites/jmpr_brightmap_2.png
 	composite brightmap_display.png brightmap_leds3.png -compose screen ../sprites/jmpr_brightmap_3.png
+
+	rm brightmap_display.png
+	rm brightmap_leds?.png
+
+	rm tmp?.png
+

zscript.zs

@@ -28,16 +28,6 @@ class JumperCablesMapLoader : EventHandler
         worldScanCompleted = false;
     }
 
-    override void WorldLoaded(WorldEvent e)
-    {
-        // JumperCablesMapLoader loader = Get();
-
-        // // Clear out anything from the last level.
-        // deployedCables.Clear();
-        // loader.originalLinedefSpecials2.Clear();
-        // worldScanCompleted = false;
-    }
-
     override void WorldTick()
     {
         JumperCablesMapLoader loader = Get();
@@ -54,21 +44,16 @@ class JumperCablesMapLoader : EventHandler
             }
 
             loader.worldScanCompleted = true;
-
-            Console.printf("JMP: World scan completed.");
         }
     }
 
     override void WorldLineActivated(WorldEvent e)
     {
-        // TODO: Deplete batteries.
-        //Console.printf("Line activated: %d %d\n", e.ActivatedLine.index(), e.ActivatedLine.special);
-
+        // Is this line affected by a battery?
         for(int cableIndex = 0; cableIndex < deployedCables.size(); cableIndex++) {
             if(deployedCables[cableIndex].lineDefIndex == e.ActivatedLine.index()) {
-
+                // Yes. Deplete the battery.
                 deployedCables[cableIndex].DecrementBattery();
-
                 break;
             }
         }
@@ -76,24 +61,16 @@ class JumperCablesMapLoader : EventHandler
 
     void RegisterCables(JumperCablesDeployed cableActor)
     {
-        // Console.printf("REGISTERING %d\n", deployedCables.Find(cableActor));
         if(deployedCables.Find(cableActor) == deployedCables.size()) {
             deployedCables.Push(cableActor);
-
-            // FIXME: Remove this.
-            // Console.printf("Cables registered %d\n", deployedCables.size());
         }
     }
 
     void UnregisterCables(JumperCablesDeployed cableActor)
     {
-        // Console.printf("UNREGISTERING %d\n", deployedCables.Find(cableActor));
         int index = deployedCables.find(cableActor);
         if(index != deployedCables.size()) {
             deployedCables.delete(index);
-
-            // FIXME: Remove this.
-            // Console.printf("Cables unregistered %d\n", deployedCables.size());
         }
     }
 }
@@ -120,7 +97,6 @@ class JumperCablesDeployed : HDUPK
 
     override void PostBeginPlay()
     {
-        // Console.printf("IS THIS EVEN GETTING CALLED!?\n");
         bNoGravity = true;
 
         // Register us with the global event handler so we can
@@ -235,10 +211,13 @@ class JumperCablesDeployed : HDUPK
         if(frandom(0.0, 1.0) < 0.02) {
             SpawnSpark();
         }
-    }
 
-    // TODO: Tick function (or whatever) where we set the sprite to indicate the charge level.
-    // TODO: Despawn if the sector gets broken *again*?
+        // Despawn to battery if the sector gets re-destroyed.
+        if(level.lines[lineDefIndex].special == 0)
+        {
+            DespawnToBattery();
+        }
+    }
 }
 
 class JumperCablesUsable : HDWeapon
@@ -290,18 +269,13 @@ class JumperCablesUsable : HDWeapon
         Actor hpc,
         String whichdot)
     {
-        // int cx,cy,cw,ch;
-		// [cx,cy,cw,ch]=screen.GetClipRect();
-		// statusBar.SetClipRect(cx,cy,cw,ch);
-
+        // Switch reticle colors depending on the LED status.
         String filename = String.Format("graphics/jmpr_reticle%d.png", lastIndicatorStatus + 1);
-        // console.printf("blep, %s %d\n", filename, frame);
         statusBar.drawImage(
             filename,
             (0, 0) + bob,
             statusBar.DI_SCREEN_CENTER | statusBar.DI_ITEM_TOP,
 			scale : (0.5, 0.5));
-
     }
 
     states
@@ -313,9 +287,21 @@ class JumperCablesUsable : HDWeapon
 		stop;
 
 	select:
-		JMPP A 0;
+		JMPP A 1;
+
+        // Raise it back up...
+        JMPP A 1 offset(0, 128);
+        JMPP A 1 offset(0, 96);
+        JMPP A 1 offset(0, 72);
+        JMPP A 1 offset(0, 64);
+        JMPP A 1 offset(0, 48);
+        JMPP A 1 offset(0, 32);
+        JMPP A 1 offset(0, 16);
+        JMPP A 1 offset(0, 0);
+
 		goto super::select;
 
+    // LCD overlay states.
     ledd_left_indicator:
         JMIL A 1 offset(0, -32); wait;
     ledd_right_indicator:
@@ -329,27 +315,18 @@ class JumperCablesUsable : HDWeapon
             }
             A_WeaponReady(WRF_NOFIRE | WRF_ALLOWUSER3); // USER3 = MagManager.
 
+            // Line trace to see what we might be inspecting.
             FLineTraceData linetraceData;
             DoLineTrace(HDPlayerPawn(invoker.owner), linetraceData);
 
-            Vector3 hitPos;
-            if(linetraceData.hitType != TRACE_HitNone) {
-                hitPos = linetraceData.hitLocation - Vec3Offset(0, 0, 0);
-            }
-
+            // Overlay setup.
 			A_Overlay(355, "ledd_left_indicator");
 			A_Overlay(356, "ledd_right_indicator");
 
-            SpawnParticleForLineTrace(
+            // Update displays.
+            UpdateVisualsForLineTrace(
                 HDPlayerPawn(invoker.owner),
                 linetraceData);
-
-            // console.printf("blep, %d\n", frame);
-
-            //player.getpsprite(PSP_WEAPON).frame = randompick(0, 1, 2);
-
-            // invoker.lastIndicatorStatus += 1;
-            // invoker.lastIndicatorStatus %= 3;
         }
         goto readyend;
 
@@ -358,6 +335,7 @@ class JumperCablesUsable : HDWeapon
         #### # 0 A_Jump(256,"ready");
 
     deploy:
+        // Lower it down...
         JMPP A 1 offset(0, 16);
         JMPP A 1 offset(0, 32);
         JMPP A 1 offset(0, 48);
@@ -366,16 +344,17 @@ class JumperCablesUsable : HDWeapon
         JMPP A 1 offset(0, 96);
         JMPP A 1 offset(0, 128);
 
+        // Play some sounds.
         JMPP A 0 A_JumpIf(!pressingfire(),"ready");
 		JMPP A 4 A_StartSound("weapons/pismagclick",CHAN_WEAPON);
 		JMPP A 2 A_StartSound("derp/crawl",CHAN_WEAPON,CHANF_OVERLAP);
 
+        // Actual cable attachment happens here.
 		JMPP A 1 {
-            // invoker.owner.A_Log("Blep");
-            A_WeaponReady(WRF_NOFIRE | WRF_ALLOWRELOAD | WRF_ALLOWUSER4);
             AttachCables(HDPlayerPawn(invoker.owner));
         }
 
+        // Raise it back up...
         JMPP A 1 offset(0, 128);
         JMPP A 1 offset(0, 96);
         JMPP A 1 offset(0, 72);
@@ -397,28 +376,18 @@ class JumperCablesUsable : HDWeapon
 
     }
 
-    action void SpawnParticleForLineTrace(HDPlayerPawn pawn, FLineTraceData linetraceData)
+    action void UpdateVisualsForLineTrace(HDPlayerPawn pawn, FLineTraceData linetraceData)
     {
 
         Line hitLine = linetraceData.hitLine;
         String particleColor = "darkred";
-
-        // if(linetraceData.hitType == TRACE_HitWall) {
-        //     JumperCablesMapLoader loader = JumperCablesMapLoader.Get();
-        //     int originalSpecial = loader.originalLinedefSpecials2.Get(hitLine.index());
-        //     int thisLineSpecial = hitLine.special;
-
-        //     if(originalSpecial != thisLineSpecial) {
-        //         particleColor = "green";
-        //     }
-        // }
-
         Line brokenLine = LineTraceFindbrokenLine(linetraceData);
 
+        // Set main frame (LEDs) based on raytrace status.
         if(brokenLine) {
-            particleColor = "green";
 
             // Found somethine we can fix!
+            particleColor = "green";
             player.getpsprite(PSP_WEAPON).frame = 1;
 
         } else {
@@ -437,6 +406,9 @@ class JumperCablesUsable : HDWeapon
 
         }
 
+        // Set display overlay frames.
+
+        // Default to random numbers.
         int frame_left = random(0, 15);
         int frame_right = random(0, 15);
 
@@ -447,19 +419,19 @@ class JumperCablesUsable : HDWeapon
         if(lineForDisplay) {
             JumperCablesMapLoader loader = JumperCablesMapLoader.Get();
             int originalSpecial = loader.originalLinedefSpecials2.Get(lineForDisplay.index());
-            console.printf("Original special: %d", originalSpecial);
             frame_right = originalSpecial % 16;
             frame_left = originalSpecial / 16;
         }
-
         player.getpsprite(355).frame = frame_left;
         player.getpsprite(356).frame = frame_right;
 
+        // Set indicator status fo the HUD.
         invoker.lastIndicatorStatus = player.getpsprite(PSP_WEAPON).frame;
 
+        // Debug particle display for the linetrace.
+        if(hd_debug > 1) {
             if(linetraceData.hitType != TRACE_HitNone) {
                 Vector3 hitPos = linetraceData.hitLocation - Vec3Offset(0, 0, 0);
-            // console.printf("hit pos: %f %f %f\n", hitPos.x, hitPos.y, hitPos.z);
                 A_SpawnParticle(
                     particleColor,
                     0, 10, 2, 0,
@@ -468,6 +440,7 @@ class JumperCablesUsable : HDWeapon
                     hitPos.z);
             }
         }
+    }
 
     action float Vec2Mag(Vector2 v)
     {
@@ -517,7 +490,7 @@ class JumperCablesUsable : HDWeapon
             48,
             pawn.pitch,
             flags : TRF_THRUACTORS | TRF_ABSOFFSET,
-            offsetz : pawn.gunPos.z - 1, // * 0.8,
+            offsetz : pawn.gunPos.z - 1,
             offsetforward : pawn.gunPos.x,
             offsetside : pawn.gunPos.y,
             data : linetraceData);
@@ -590,10 +563,6 @@ class JumperCablesUsable : HDWeapon
             // If we found something broken, then return the closest
             // thing we found.
             if(closestLineInRange) {
-                // Console.printf(
-                //     "Dist to line: %f\n",
-                //     GetDistanceToLine(
-                //         closestLineInRange, linetraceData.hitLocation));
                 return closestLineInRange;
             }
 
@@ -604,8 +573,8 @@ class JumperCablesUsable : HDWeapon
 
     action void AttachCables(HDPlayerPawn pawn)
     {
+        // Do we have enough ammo?
         HDMagAmmo batteryAmmo = HDMagAmmo(FindInventory("HDBattery"));
-
         if(!batteryAmmo || batteryAmmo.amount < 1) {
             invoker.owner.A_Log("You need a battery to use this.");
             return;
@@ -615,32 +584,36 @@ class JumperCablesUsable : HDWeapon
         FLineTraceData linetraceData;
         DoLineTrace(pawn, linetraceData);
 
+        // Are we even close enough to a line?
         Line brokenLine = LineTraceFindbrokenLine(linetraceData);
+        if(!brokenLine) {
+            invoker.owner.A_Log("Not close enough to repair.");
+            return;
+        }
 
-        if(brokenLine) {
-
-            if(loader.originalLinedefSpecials2.CheckKey(brokenLine.index())) {
-
-                // pawn.A_Log(string.Format("Blep3 %d",
-                //         loader.originalLinedefSpecials2.Get(hitLine.index())));
+        if(!loader.originalLinedefSpecials2.CheckKey(brokenLine.index())) {
+            invoker.owner.A_Log("No mechanism present to repair.");
+            return;
+        }
 
+        // Check to see if it's something that actually needs fixing.
         int originalSpecial = loader.originalLinedefSpecials2.Get(brokenLine.index());
         int currentSpecial = brokenLine.special;
+        if(originalSpecial == currentSpecial) {
+            invoker.owner.A_Log("This is already functioning as normal.");
+            return;
+        }
 
-                if(originalSpecial != currentSpecial) {
-
-                    // Remove battery from inventory.
+        // Remove battery from inventory, but make sure we're actually
+        // getting a charged one.
         int batteryCharges = batteryAmmo.TakeMag(true);
-                    // Console.printf("batteryCharges: %d", batteryCharges);
-
         if(batteryCharges < 1) {
-
-                        // Handle getting an empty battery. Just give it back.
             invoker.owner.A_Log("No charged batteries available to use.");
-                        batteryAmmo.AddAMag(batteryCharges);
-
-                    } else {
+            batteryAmmo.AddAMag(batteryCharges); // Just give it back.
+            return;
+        }
 
+        // Actually spawn the deployed item.
         JumperCablesDeployed deployed = JumperCablesDeployed(
             Spawn(
                 "JumperCablesDeployed",
@@ -650,6 +623,8 @@ class JumperCablesUsable : HDWeapon
         deployed.bNoGravity = true;
         deployed.charges = batteryCharges;
 
+        // Use a different kind of flat sprite and angle depending on
+        // if we're attaching to wall or floor/ceiling.
         if(linetraceData.hitType == TRACE_HitFloor || linetraceData.hitType == TRACE_HitCeiling) {
             deployed.SetFlatSprite(invoker.owner.angle);
         } else {
@@ -657,12 +632,9 @@ class JumperCablesUsable : HDWeapon
             deployed.SetWallSprite(lineDir);
         }
 
+        // Restore the damn line special already.
         brokenLine.special = originalSpecial;
     }
-                }
-            }
-        }
-    }
 
     override void DrawHUDStuff(HDStatusBar statusBar, HDWeapon weapon, HDPlayerPawn pawn)
     {