search

eclipse-uprotocol / up-rust

uProtocol Language Specific Library for Rust
Apache License 2.0
11 stars 9 forks source link

Throw out map_response() and map_response_to_result() from RpcMapper? #106

Closed AnotherDaniel closed 3 months ago

AnotherDaniel commented 4 months ago

Hi especially @PLeVasseur and @evshary - question: doing my usubscription thing, I've come to the point where I found out what kind of functionality I need from RpcMapper around using invoke_method(), and the current map_response() and map_response_to_result() are essentially useless from my PoV.

I have more pragmatic alternatives in the pipeline, but would like to remove the current map_response code along with that.

So my question is, are you using RpcMapper::map_response() and RpcMapper::map_response_to_result() anywhere in your code?

PLeVasseur commented 4 months ago

I am not, nope!

May be good to see if @stevenhartley has any thoughts around this.

evshary commented 4 months ago

Nope on my side, either.

AnotherDaniel commented 3 months ago

Done