Feature #1442
RPC should support argumentless methods properly
Status: | Resolved | Start date: | 02/27/2013 | |
---|---|---|---|---|
Priority: | Normal | Due 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
Associated revisions
fixes #1442: Add argument-less and void methods to RPC
History
#1 Updated by S. Wrede over 11 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 11 years ago
Sorry, I cannot find the new signatures. Did your commit include all changes?
#3 Updated by S. Wrede over 11 years ago
It should be part of the feature-branch, cf.:
#4 Updated by J. Moringen about 11 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 11 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 11 years ago
- Status changed from In Progress to Resolved
- % Done changed from 60 to 100
Applied in changeset rsb-java|commit:990a47dbf4885ecb61343665bf27c8044fc5a523.