Fixes #1261 - Clone the Terrain Picker, so that loading a terrain from file still works (after cloning, the picker would have the wrong terrain quad instance)
This commit is contained in:
parent
17fbedd5fe
commit
55a36abe9d
@ -1793,9 +1793,8 @@ public class TerrainQuad extends Node implements Terrain {
|
||||
// This was not cloned before... I think that's a mistake.
|
||||
this.affectedAreaBBox = cloner.clone(affectedAreaBBox);
|
||||
|
||||
// picker is not cloneable and not cloned. This also seems like
|
||||
// a mistake if you ever load the same terrain twice.
|
||||
// this.picker = cloner.clone(picker);
|
||||
// Otherwise, picker would be cloned by reference and thus "this" would be wrong
|
||||
this.picker = new BresenhamTerrainPicker(this);
|
||||
|
||||
// neighbourFinder is also not cloned. Maybe that's ok.
|
||||
}
|
||||
|
Loading…
x
Reference in New Issue
Block a user