8000 [csharp] add a callback-based ServiceClient API and a corresponding C# sample (2) by rex-schilasky · Pull Request #2199 · eclipse-ecal/ecal · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

[csharp] add a callback-based ServiceClient API and a corresponding C# sample (2) #2199

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Jun 11, 2025

Conversation

rex-schilasky
Copy link
Contributor

Description

This PR introduces a new, callback-based variant of the service client call (CallWithCallback) alongside the existing CallWithResponse. It also adds or updates C# sample code to demonstrate the new API.

API Changes

  • Added ServiceClient::CallWithCallback(String^ methodName, array<Byte>^ request, ResponseCallback^ callback, int timeoutMs) to the C# wrapper.

Sample Added

  • mirror_client_callback_csharp.cs: Demonstrates registering a response-callback delegate instead of using CallWithResponse.

@@ -112,7 +116,20 @@ Iterate through all responses and print them.
}
else
{
Console.WriteLine("Method blocking call failed.");
Console.WriteLine("Method call with reponse failed.");
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Little Typo - response

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will fix this 😀

@hannemn hannemn merged commit 925b4cd into master Jun 11, 2025
25 of 27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherry-pick-to-support/v6.0 Cherry pick these changes to support/v6.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0