Skip to content

Can't extend encoding by hack Buffer.isEncoding anymore #1547

Description

@hefangshi

a widely used iconv library iconv-lite can extend encoding by hack Buffer.isEncoding using extendNodeEncodings

but a newly commit 8a94581 cached the Buffer.isEncoding method https://github.com/iojs/io.js/blob/master/lib/string_decoder.js#L3-L9 which made extendNodeEncodings won't work.

Activity

  1. added
    string_decoderIssues and PRs related to the string_decoder subsystem.
    on Apr 28, 2015
  2. Fishrock123 commented on Apr 28, 2015

    @Fishrock123
    Contributor

    @mscdex Looks like that could be undone easily enough without any real downsides. You on it?

  3. mscdex commented on Apr 28, 2015

    @mscdex
    Contributor

    Fixed in 0fa6c4a.

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

    string_decoderIssues and PRs related to the string_decoder subsystem.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions