Add event.surface to on_player_selected_area

[darkfrei]

Can you please add the surface to this event?
https://lua-api.factorio.com/latest/eve ... ected_area

on_player_selected_area
Called after a player selects an area with a selection-tool item.

Contains
player_index :: uint: The player doing the selection.
area :: BoundingBox: The area selected.
item :: string: The item used to select the area.
entities :: array of LuaEntity: The entities selected.
tiles :: array of LuaTile: The tiles selected.

The information about area without surface is not full.

[Rseding91]

The surface is the player.surface. But I also don't mind adding it to the event.

[darkfrei]

The surface is the player.surface. But I also don't mind adding it to the event.

There is actually no player, but
local surface = game.players[event.player_index].surface
instead of just
local surface = event.surface

[Rseding91]

The surface is the player.surface. But I also don't mind adding it to the event.

There is actually no player, but
local surface = game.players[event.player_index].surface
instead of just
local surface = event.surface

I know. It's that way on purpose because making the player object for every event when in a lot of cases event handlers don't care about the player would use a lot of extra CPU time.

[darkfrei]

The surface is the player.surface. But I also don't mind adding it to the event.

There is actually no player, but
local surface = game.players[event.player_index].surface
instead of just
local surface = event.surface

I know. It's that way on purpose because making the player object for every event when in a lot of cases event handlers don't care about the player would use a lot of extra CPU time.

So the game.players[event.player_index].surface needs at least the same amount CPU time as event.player?

[Rseding91]

No, but most event handlers won't use the player index value.