Skip to content

Made grpc_byte_buffer_reader able to decompress msgs.#1963

Merged
ctiller merged 4 commits intogrpc:masterfrom
dgquintas:compression
Jun 9, 2015
Merged

Made grpc_byte_buffer_reader able to decompress msgs.#1963
ctiller merged 4 commits intogrpc:masterfrom
dgquintas:compression

Conversation

@dgquintas
Copy link
Copy Markdown
Contributor

Part of #1795.

@dgquintas
Copy link
Copy Markdown
Contributor Author

I've also moved a number of function grpc_byte_buffer_* declarations from grpc.h to a more fitting byte_buffer.h.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'm not sure I understand what this means.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The meaning behind the comment was that a channel compressed by default would, when instructed, not compress certain messages, resulting in said messages having this BB_COMPRESSED_NONE type. This would signal that the message indeed came from a compressing source but that in this instance, the data isn't compressed. I wonder if this distinction is useful though.

Given your proposed change below, the redundancy between BB_SLICE_BUFFER and COMPRESSED_NONE is removed anyway.

@dgquintas
Copy link
Copy Markdown
Contributor Author

Comments addressed. It evolved into a bit of a refactoring. Plenty of changed files, but most are just "dumb" renamings. PTAL.

ctiller added a commit that referenced this pull request Jun 9, 2015
Made grpc_byte_buffer_reader able to decompress msgs.
@ctiller ctiller merged commit 887774d into grpc:master Jun 9, 2015
@lock lock bot locked as resolved and limited conversation to collaborators Jan 31, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants