Grabbing bug

[BannZay]

Grab something from picker(for example) switch chosed layer then brush. This way you could do a lot of bugs.
For example grab tune tile - swithc to tele - draw = not working but marked tele tiles
Grabbing game freeze - switch to tele - paint = game crash.

[timakro]

Can't replicate, for me no unused index is placeable in the tele layer.

[BannZay]

Try to use exactly paint feature (pick entitie -> press shift and paint some area)
P.s: here is a lot of ways how to crash client this way

[timakro]

Ah right, I would call that filling, thanks for the report.

[timakro]

In my opinion the logic for checking if a tile is allowed to be placed should be shifted to the new GetTile method I introduced for game and front layer already. It doesn't support extra data though like for example the number of the tune zone for tune layer.

[4:41:08 PM] Tim: yeah id like to push stuff like BrushDraw only to CLayerTiles
[4:41:15 PM] Tim: because its basically the same for all layers
[4:41:22 PM] Tim: just that some tiles are removed is different
[4:41:31 PM] Tim: and that logic can go to SetTile
[4:41:37 PM] Tim: but I'm too lazy to do that
[4:41:48 PM] Tim: also we would need a more general type that represents tiles
[4:42:17 PM] Tim: maybe we could inherit from CTile
[4:42:20 PM] Tim: to CTileTele
[4:42:34 PM] Tim: which additionally stores the tele number
[4:42:56 PM] Tim: SetTile and GetTile work with CTile
[4:43:07 PM] Tim: and CLayerTele::SetTile could cast the CTile it takes to CTileTele

[4:52:14 PM] Tim: yes the correct thing to do is to make CTileLayers::BrushDraw call CLayerTele::GetTile which returns a CTileTele casted to a CTile and then call CLayerTele::SetTile a lot of times for every tile we want to fill

[C0D3D3V]

I can not reproduce this in latest master. Maybe it is fixed.
If I try to do the steps, then only the tiles of the other layers are removed at the marked location, but nothing else happens.