Skip to content

Commit 8fe4cd0

Browse files
committed
Removed unnessissary listener removal and re-add
THis made the set to free functuion in the datagram to take tons of CPU. It is more processor light now.
1 parent 32d94ab commit 8fe4cd0

File tree

2 files changed

+16
-8
lines changed

2 files changed

+16
-8
lines changed

javasdk/NRSDK/src/com/neuronrobotics/sdk/common/BowlerDatagram.java

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -437,12 +437,19 @@ public void setFree(boolean isFree, BowlerDatagramFactory factory) {
437437
setFree(isFree);
438438
}
439439

440+
private void setNotFree(){
441+
clear();
442+
timeout.stop();
443+
}
444+
445+
private void setToFree(){
446+
timeout.initialize(BowlerDatagramFactory.getPacketTimeout(), this);
447+
}
440448
void setFree(boolean isFree) {
441449
if(isFree== true){
442-
clear();
443-
timeout.stop();
450+
setNotFree();
444451
}else{
445-
timeout.initialize(BowlerDatagramFactory.getPacketTimeout(), this);
452+
setToFree();
446453
}
447454
this.isPackedAvailibleForLoading = isFree;
448455
}

javasdk/NRSDK/src/com/neuronrobotics/sdk/common/ThreadedTimeout.java

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -57,11 +57,11 @@ public boolean isTimedOut() {
5757
}
5858

5959
public void initialize(long sleepTime,IthreadedTimoutListener listener) {
60-
stop();
61-
this.time = (sleepTime);
60+
//stop();
6261
setStartTime(System.currentTimeMillis());
62+
this.time = (sleepTime);
6363
setTimeoutListener(listener);
64-
timerThread.addTimer(this);
64+
//timerThread.addTimer(this);
6565
}
6666

6767
public long getAmountOfTimeForTimerToRun() {
@@ -83,6 +83,7 @@ public void run(){
8383
while(true){
8484
if(getTimers().size()>0){
8585
toRemove.clear();
86+
8687
for(int i=0;i<getTimers().size();i++){
8788
try{
8889
ThreadedTimeout t = getTimers().get(i);
@@ -112,15 +113,15 @@ public void addTimer(ThreadedTimeout time){
112113
if(!getTimers().contains(time))
113114
getTimers().add(time);
114115
}catch (Exception e){
115-
116+
e.printStackTrace();
116117
}
117118
}
118119
public void removeTimer(ThreadedTimeout time){
119120
try{
120121
if(getTimers().contains(time))
121122
getTimers().remove(time);
122123
}catch (Exception e){
123-
124+
e.printStackTrace();
124125
}
125126

126127
}

0 commit comments

Comments
 (0)