pw_rpc: Move server/client calls to separate files

- Move the server/client calls to improve code organization and make
  upcoming lock annotations simpler, since the Endpoint class is fully
  defined in the server/client call headers.
- No longer have the server respond to a cancel for an unknown call.
  There is no point in sending an error for that RPC to the client
  because the client cancelled it and won't know about that call by the
  time it gets the error. In general, error packets should only be sent
  when an RPC fails to start or needs to be terminated.
- Use PW_TRY_ASSIGN in the server and client.

Change-Id: I467d06ef7139a5698be0b30e9ed1bac3a625e02e
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/66201
Commit-Queue: Wyatt Hepler <hepler@google.com>
Pigweed-Auto-Submit: Wyatt Hepler <hepler@google.com>
Reviewed-by: Alexei Frolov <frolv@google.com>

1 files changed, 1 insertions, 5 deletions

diff --git a/pw_rpc/endpoint.cc b/pw_rpc/endpoint.cc
index fa765e30c..25cec031a 100644
--- a/pw_rpc/endpoint.cc
+++ b/pw_rpc/endpoint.cc
@@ -27,8 +27,7 @@ Endpoint::~Endpoint() {
 }
 
 Result<Packet> Endpoint::ProcessPacket(std::span<const std::byte> data,
-                                       Packet::Destination destination,
-                                       Call*& ongoing_call) {
+                                       Packet::Destination destination) {
   Result<Packet> result = Packet::FromBuffer(data);
 
   if (!result.ok()) {
@@ -48,9 +47,6 @@ Result<Packet> Endpoint::ProcessPacket(std::span<const std::byte> data,
     return Status::InvalidArgument();
   }
 
-  // Find an existing reader/writer for this RPC, if any.
-  ongoing_call = FindCall(packet);
-
   return result;
 }