fix(fishing handler): minor logic fix

optional/handlers/rsfishinghandler.simba

@@ -161,7 +161,6 @@ end;
 ```pascal
 procedure TRSFishingHandler.Reset();
 ```
-
 Resets TRSFishingHandler to the non fishing state.
 This will:
   - Pause TRSFishingHandler.Timer;
@@ -180,7 +179,6 @@ end;
 ```pascal
 function TRSFishingHandler.GetMMShoreLine(): TPointArray;
 ```
-
 Get the shore lines on the minimap.
 
 Example:
@@ -212,7 +210,6 @@ end;
 ```pascal
 function TRSFishingHandler.GetMSShoreLine(): TPointArray;
 ```
-
 Get the shore lines on the mainscreen.
 
 Example:
@@ -244,7 +241,6 @@ end;
 ```pascal
 function TRSFishingHandler.FindWaterDirection(): TRectangle;
 ```
-
 Finds the adjacent tile that is directly close to the player that has water.
 
 Example:
@@ -280,11 +276,8 @@ end;
 ## TRSFishingHandler.FindSpot
 ```pascal
 function TRSFishingHandler.FindSpot(out atpa: T2DPointArray; waitTime: Int32 = 350): Boolean;
-```
-```pascal
 function TRSFishingHandler.FindSpot(): Boolean;  overload;
 ```
-
 Find fishing spots on the mainscreen. Optionally return at ATPA of the found spots.
 
 Example:
@@ -363,11 +356,8 @@ end;
 ## TRSFishingHandler.WaitSpot
 ```pascal
 function TRSFishingHandler.WaitSpot(out atpa: T2DPointArray; waitTime: Int32 = 350): Boolean;
-```
-```pascal
 function TRSFishingHandler.WaitSpot(): Boolean;  overload;
 ```
-
 Wait for fishing spots on the mainscreen. Optionally return at ATPA of the found spots.
 
 Example:
@@ -393,6 +383,8 @@ function TRSFishingHandler.WaitSpot(out atpa: T2DPointArray; waitTime: Int32 = 3
     t := GetTickCount() + waitTime;
 
     repeat
+      if Inventory.IsFull() then Exit;
+
       for c in Self.FishSpotColors do
         if SRL.FindColors(tpa, c, bounds) then
           spotTPA += tpa;
@@ -450,11 +442,8 @@ end;
 ## TRSFishingHandler.SpotMoved
 ```pascal
 function TRSFishingHandler.SpotMoved(out atpa: T2DPointArray): Boolean;
-```
-```pascal
 function TRSFishingHandler.SpotMoved(): Boolean;  overload;
 ```
-
 Check if TRSFishingHandler.CurrentSpot moved.
 
 Example:
@@ -525,7 +514,6 @@ end;
 ```pascal
 function TRSFishingHandler._StoppedFishing(): Boolean;
 ```
-
 Internal helper method to check for quick things that guarantee us we are not fishing anymore.
 *)
 function TRSFishingHandler._StoppedFishing(): Boolean;
@@ -539,11 +527,8 @@ end;
 ## TRSFishingHandler.CheckFishing
 ```pascal
 function TRSFishingHandler.CheckFishing(out atpa: T2DPointArray): Boolean;
-```
-```pascal
 function TRSFishingHandler.CheckFishing(): Boolean;  overload;
 ```
-
 Check if we are still fishing. This function could give false positives but it's extremely unlikely.
 To determine if we are still fishing it does the following:
 - Check if TRSFishingHandler.CurrentSpot (which is the last spot we clicked) has moved.
@@ -598,7 +583,6 @@ end;
 ```pascal
 function TRSFishingHandler.FacingWater(): Boolean;
 ```
-
 Check if we have water directly north, west, south or east of us.
 
 Example:
@@ -624,7 +608,6 @@ end;
 function TRSFishingHandler.ClickSpot(out atpa: T2DPointArray): Boolean;
 function TRSFishingHandler.ClickSpot(): Boolean;  overload;
 ```
-
 Click the closest fishing spot to the player and switches TRSFishingHandler to "IsFishing".
 We can check if we are still fishing with TRSFishingHandler.CheckFishing().