Skip to content

WARNING: Use recvLen defined in "Network.Socket.ByteString" #246

Closed
Closed
WARNING: Use recvLen defined in "Network.Socket.ByteString"#246
Labels
documentation

Description

@p-alik
p-alik
opened on May 15, 2017

The warning refers to Network.Socket.ByteString but it doesn't provide recvLen

Activity

  1. kazu-yamamoto commented on May 22, 2017

    @kazu-yamamoto
    kazu-yamamoto
    on May 22, 2017
    Collaborator

    I think that we should use Network.Socket.ByteString.recv and Network.ByteString.length instead.

    @eborden What do you think?

  2. eborden commented on May 30, 2017

    @eborden
    eborden
    on May 30, 2017
    Collaborator

    Agreed, we should update this documentation. recvLen is a hack to get around the performance problems of length on String.

  3. eborden
    added
    documentation
    on May 30, 2017

  4. kazu-yamamoto commented on May 30, 2017

    @kazu-yamamoto
    kazu-yamamoto
    on May 30, 2017
    Collaborator

    This is fixed by #252. Let's close.

  5. kazu-yamamoto
    closed this as completedon May 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    documentation

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions