From 3703f96ad852c49182dacda5ef42b9a8a7caa6da Mon Sep 17 00:00:00 2001
From: teejusb <5017202+teejusb@users.noreply.github.com>
Date: Wed, 31 May 2023 22:51:01 -0700
Subject: [PATCH] Let earlyHits on hold heads start the hold immediately.

---
 src/Player.cpp | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/src/Player.cpp b/src/Player.cpp
index d01bf3a5fd..eecab3c791 100644
--- a/src/Player.cpp
+++ b/src/Player.cpp
@@ -1291,8 +1291,11 @@ void Player::UpdateHoldNotes( int iSongRow, float fDeltaTime, std::vector<TrackR
 		// taps was hit before activating this group of holds.
 		/* Something about the logic in this section is causing 192nd steps to
 		 * fail for some odd reason. -aj */
-		bSteppedOnHead &= (tns != TNS_Miss && tns != TNS_None);	// did they step on the start of this hold?
-		bHeadJudged &= (tns != TNS_None);	// has this hold really even started yet?
+		// NOTE(teejusb): We want early hits to count as a hit on the hold head
+		// otherwise it visually looks weird if you never have a second hit for the
+		// arrow.
+		bSteppedOnHead &= (tns != TNS_Miss && tns != TNS_None || tn.result.earlyTns != TNS_None);	// did they step on the start of this hold?
+		bHeadJudged &= (tns != TNS_None || tn.result.earlyTns != TNS_None);	// has this hold really even started yet?
 
 		/*
 		if(bSteppedOnHead)