Remove the need for BltPlayerTop

[Robin]

Code:

For y = 0 To MAX_MAPY
    ' Blit out the npcs
    For i = 1 To MAX_MAP_NPCS
        If MapNpc(i).y = y Then
            Call BltNpc(i)
        End If
    Next i
                    
    ' Blit out players
    For i = 1 To MAX_PLAYERS
        If IsPlaying(i) Then
            If GetPlayerMap(i) = GetPlayerMap(MyIndex) Then
                If GetPlayerY(i) = y Then
                    Call BltPlayer(i)
                End If
            End If
        End If
    Next i
Next y

[JokeofWeek]

Aha, such a quick fix, good optimization too, since it saves you the extra MAX_PLAYERS loop You can do this with BltNPCtop also, but you can't have both fixes, coz what if, for example, the NPC's top had to go over the player's bottom. All in all, a good optimization though Saves at least one big loop Nice one Robin :wink:

[Robin]

It already handles the NPC as well. Look at the code xD

I've been meaning to do something like this for ages. I originally had it as:

For y = max_mapy to 0 step - 1

Thinking that it should render from bottom to top. Silly mistake tbh xD

[JokeofWeek]

It already handles the NPC as well. Look at the code xD

I've been meaning to do something like this for ages. I originally had it as:

For y = max_mapy to 0 step - 1

Thinking that it should render from bottom to top. Silly mistake tbh xD

:O True xD Hadn't realized that Although, it's probably the easiest to do in DX8, just set the Z value I'd beleive xD Nice optimization for DX7 though

[Robin]

I use the same method in Dx8.