From a3f88aa164fe6e4778466b99db5b6bd7a476be98 Mon Sep 17 00:00:00 2001 From: Stephen Gold Date: Sun, 6 Jan 2019 12:22:26 -0800 Subject: [PATCH] fix issue #997 (cloning UpdateControl throws IllegalArgumentException) --- .../src/main/java/com/jme3/scene/control/UpdateControl.java | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/jme3-core/src/main/java/com/jme3/scene/control/UpdateControl.java b/jme3-core/src/main/java/com/jme3/scene/control/UpdateControl.java index 9c101c73e..084179b85 100644 --- a/jme3-core/src/main/java/com/jme3/scene/control/UpdateControl.java +++ b/jme3-core/src/main/java/com/jme3/scene/control/UpdateControl.java @@ -1,5 +1,5 @@ /* - * Copyright (c) 2009-2012 jMonkeyEngine + * Copyright (c) 2009-2019 jMonkeyEngine * All rights reserved. * * Redistribution and use in source and binary forms, with or without @@ -57,7 +57,7 @@ import java.util.concurrent.Future; */ public class UpdateControl extends AbstractControl { - private final ConcurrentLinkedQueue> taskQueue = new ConcurrentLinkedQueue>(); + private ConcurrentLinkedQueue> taskQueue = new ConcurrentLinkedQueue<>(); /** * Enqueues a task/callable object to execute in the jME3 @@ -99,6 +99,7 @@ public class UpdateControl extends AbstractControl { @Override public Object jmeClone() { UpdateControl clone = (UpdateControl)super.jmeClone(); + clone.taskQueue = new ConcurrentLinkedQueue<>(); // This is kind of questionable since the tasks aren't cloned and have // no reference to the new spatial or anything. They'll get run again