Copilot commented on code in PR #802:
URL: https://github.com/apache/celix/pull/802#discussion_r2389635731
##########
bundles/remote_services/rsa_rpc_json/src/rsa_json_rpc_proxy_impl.c:
##########
@@ -46,8 +46,9 @@ struct rsa_json_rpc_proxy_factory {
endpoint_description_t *endpointDesc;
celix_long_hash_map_t *proxies;//Key:requestingBundle, Value:
rsa_json_rpc_proxy_t *. Work on the celix_event thread , so locks are not
required
remote_interceptors_handler_t *interceptorsHandler;
- rsa_request_sender_tracker_t *reqSenderTracker;
- long reqSenderSvcId;
+ celix_thread_rwlock_t sendRequestLock; //protects sendRequest
+ celix_rsa_send_request_fp sendRequest;
+ void* sendRequestHandle;
};
Review Comment:
Unnecessary semicolon after closing brace of function definition.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
