Lucio Franco

Results 524 comments of Lucio Franco

Yeah, I think this would be a good addition.

While I agree that API is ideal I think its quite hard now with current futures infrastructure. Is there a reason you can't get to something like `Arc`?

So I think doing it by ref will be quite hard like you said. For now having an owned buffer is really the right way to go imo. Its likely...

I will try and give this a review in the coming weeks I have a decent amount of backlog right now.

At this point I don't have much time for this and in general tonic will be moving to be lighter (will eventually post the roadmap when I finish it lol)....

@jdahlq the `into_http` I think we can merge but the boxed one I believe we shouldn't expose that method maybe but we can expose stuff on `Status` to let you...

@seanmonstar you may be interested

Sounds about right. Tonic doesn't do any sort of checking for that and delegates to h2.

Agreed we can make this change. I believe we should do this in conjunction with https://github.com/hyperium/tonic/pull/830