Skip to content
This repository was archived by the owner on Aug 11, 2023. It is now read-only.

Commit 9a4116c

Browse files
author
Juan Ignacio Ubeira
authored
Merge pull request #296 from rjcausarano/test-fix-proposal
Fix problems shutting down nodes
2 parents 1f2ec3e + 1824187 commit 9a4116c

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,3 +9,4 @@ bin
99
*.iml
1010
.idea
1111
bazel-*
12+
local.properties

rosjava/src/main/java/org/ros/internal/node/DefaultNode.java

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -416,8 +416,6 @@ public void shutdown() {
416416
// NOTE(damonkohler): We don't want to raise potentially spurious
417417
// exceptions during shutdown that would interrupt the process. This is
418418
// simply best effort cleanup.
419-
slaveServer.shutdown();
420-
topicParticipantManager.shutdown();
421419
for (ServiceServer<?, ?> serviceServer : serviceManager.getServers()) {
422420
try {
423421
Response<Integer> response =
@@ -436,8 +434,9 @@ public void shutdown() {
436434
for (ServiceClient<?, ?> serviceClient : serviceManager.getClients()) {
437435
serviceClient.shutdown();
438436
}
439-
registrar.shutdown();
440437
slaveServer.shutdown();
438+
topicParticipantManager.shutdown();
439+
registrar.shutdown();
441440
signalOnShutdownComplete();
442441
}
443442

0 commit comments

Comments
 (0)