protobuf icon indicating copy to clipboard operation
protobuf copied to clipboard

Implement WriteDelimitedTo(BufferWriter<byte>)

Open verdie-g opened this issue 9 months ago • 8 comments

Closes #21300

I could not find tests MessageExtensions so I added a new test file.

verdie-g avatar Apr 17 '25 03:04 verdie-g

@amanda-tarafa could you have a look at this PR please 🙏

verdie-g avatar Jun 16 '25 14:06 verdie-g

Can we get this pushed further somehow? Library leaves no public surface for workaround. Forcing to write our own length when proper api is just one internal call away is unnecessarily defensive library design.

aohotnik avatar Jun 23 '25 05:06 aohotnik

@jskeet I understand you don't have much time to dedicate to protobuf these days but I'm not able to get a response on this small PR so following the Contributing Process I'd appreciate your help here 🙏

verdie-g avatar Jul 03 '25 15:07 verdie-g

We triage inactive PRs and issues in order to make it easier to find active work. If this PR should remain active, please add a comment.

This PR is labeled inactive because the last activity was over 90 days ago. This PR will be closed and archived after 14 additional days without activity.

github-actions[bot] avatar Oct 02 '25 10:10 github-actions[bot]

Still waiting for a reviewer.

verdie-g avatar Oct 02 '25 11:10 verdie-g

C# team is still underwater.

googleberg avatar Oct 03 '25 20:10 googleberg

I'm sorry, but my focus is elsewhere at the moment.

amanda-tarafa avatar Oct 03 '25 22:10 amanda-tarafa

@esrauchg anything missing to get this merged?

verdie-g avatar Nov 05 '25 20:11 verdie-g