Feature #1442

RPC should support argumentless methods properly

Added by J. Moringen over 8 years ago. Updated about 8 years ago.

Status:ResolvedStart date:02/27/2013
Priority:NormalDue date:
Assignee:J. Wienke% Done:

100%

Category:Java
Target version:rsb-0.9

Description

There should be method signatures (if Java allows this) and unit tests for:

RemoteServer.call("METHOD")      // no argument
RemoteServer.callAsync("METHOD")

Also, RemoteServer.call("METHOD", null) should not trigger the event-based code path, treating null as an Event. Such a call should either signal an error or behave like the above signatures.


Related issues

Related to Robotics Service Bus - Tasks #1465: Provide better interface for parameterless RPC methods New 04/17/2013
Precedes Robotics Service Bus - Tasks #1553: Reformat whole source code Resolved 02/28/2013 02/28/2013

Associated revisions

Revision 990a47db
Added by J. Wienke about 8 years ago

fixes #1442: Add argument-less and void methods to RPC

History

#1 Updated by S. Wrede over 8 years ago

  • Status changed from New to Feedback
  • % Done changed from 0 to 60

Initial support is available in: 9a4cd505a9eba2939637623b55b7713cde7eda49

The client-side API now supports parameterless calls such as:

remote.call("methodId")

The server-side API needs a bit of discussion, e.g., whether an additional type of callback interface would be handy (or not) given the fact that event handlers already have the required semantics for this case (primarily: no return value). As an alternative, one may also just use the existing callback interfaces and just return null.

#2 Updated by J. Moringen over 8 years ago

Sorry, I cannot find the new signatures. Did your commit include all changes?

#4 Updated by J. Moringen about 8 years ago

Sebastian: I looked at this once and it seemed fine (if I remember correctly). Would you rather first finish everything or can you merge the @RemoteServer part?

#5 Updated by J. Wienke about 8 years ago

  • Status changed from Feedback to In Progress
  • Assignee changed from S. Wrede to J. Wienke

If it's ok, I would try to sort out the things remaining here.

#6 Updated by J. Wienke about 8 years ago

  • Status changed from In Progress to Resolved
  • % Done changed from 60 to 100

Applied in changeset rsb-java|commit:990a47dbf4885ecb61343665bf27c8044fc5a523.

Also available in: Atom PDF