Re: [PATCH crosvm 2/2] sys_util: expand on UnixSeqpacket documentation

Alyssa Ross
2020-07-06T23:07:26Z

[-- Attachment #1: Type: text/plain, Size: 562 bytes --]

Cole Helbling <cole.e.helbling@outlook.com> writes:

>> - /// Receive data from the socket fd to a given buffer.
>> + /// Receive a message from the socket fd to a given buffer. The message will then
>> + /// be removed from the socket, whether `buf` was large enough to hold all of it or
>
> "whether or not `buf` was large enough..." sounds better, IMO.

I agree.

>> + ///
>> /// # Errors
>> /// Returns error when `libc::recv` failed.
>
> Returns *an* error.

Ah, yes, I can change this while I'm here. Good catch. :)

v2 on its way!

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 832 bytes --]