chore: support node.js >=6, remove safe-buffer

.github/workflows/ci.yml

@@ -37,6 +37,9 @@ jobs:
         run: npm run lint
 
   performance:
+    needs:
+      - lint
+      - coverage
     name: Run performance test
     strategy:
       fail-fast: false
@@ -64,12 +67,8 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        os: [ubuntu-latest]
+        os: [ubuntu-latest, windows-latest]
         node-version:
-          - "0.10"
-          - "0.12"
-          - "4"
-          - "5"
           - "6"
           - "7"
           - "8"
@@ -92,28 +91,6 @@ jobs:
         # Use supported versions of our testing tools under older versions of Node
         # Install npm in some specific cases where we need to
         include:
-          - node-version: "0.10"
-            npm-i: "mocha@3.5.3 nyc@10.3.2 supertest@2.0.0"
-            # Npm isn't being installed on windows w/ setup-node for
-            # 0.10 and 0.12, which will end up choking when npm uses es6
-            npm-version: "npm@2.15.1"
-            npm-rm: 'iconv'
-
-          - node-version: "0.12"
-            npm-i: "mocha@3.5.3 nyc@10.3.2"
-            npm-version: "npm@2.15.11"
-            npm-rm: 'iconv'
-
-          - node-version: "4"
-            npm-i: "mocha@5.2.0 nyc@11.9.0"
-            npm-rm: 'iconv'
-
-          - node-version: "5"
-            npm-i: "mocha@5.2.0 nyc@11.9.0"
-            # fixes https://github.com/npm/cli/issues/681
-            npm-version: npm@2.15.11
-            npm-rm: 'iconv'
-
           - node-version: "6"
             npm-i: "mocha@6.2.2 nyc@14.1.1"
             npm-version: npm@6.13.4

encodings/dbcs-codec.js

@@ -1,5 +1,5 @@
 "use strict"
-var Buffer = require("safer-buffer").Buffer
+var Buffer = require("buffer").Buffer
 
 // Multibyte codec. In this scheme, a character is represented by 1 or more bytes.
 // Our codec supports UTF-16 surrogates, extensions for GB18030 and unicode sequences.

encodings/internal.js

@@ -1,5 +1,5 @@
 "use strict"
-var Buffer = require("safer-buffer").Buffer
+var Buffer = require("buffer").Buffer
 
 // Export Node.js internal encodings.
 

encodings/sbcs-codec.js

@@ -1,5 +1,5 @@
 "use strict"
-var Buffer = require("safer-buffer").Buffer
+var Buffer = require("buffer").Buffer
 
 // Single-byte codec. Needs a 'chars' string parameter that contains 256 or 128 chars that
 // correspond to encoded bytes (if 128 - then lower half is ASCII).

encodings/utf16.js

@@ -1,5 +1,5 @@
 "use strict"
-var Buffer = require("safer-buffer").Buffer
+var Buffer = require("buffer").Buffer
 
 // Note: UTF16-LE (or UCS2) codec is Node.js native. See encodings/internal.js
 

encodings/utf32.js

@@ -1,6 +1,6 @@
 "use strict"
 
-var Buffer = require("safer-buffer").Buffer
+var Buffer = require("buffer").Buffer
 
 // == UTF32-LE/BE codec. ==========================================================
 

encodings/utf7.js

@@ -1,5 +1,5 @@
 "use strict"
-var Buffer = require("safer-buffer").Buffer
+var Buffer = require("buffer").Buffer
 
 // UTF-7 codec, according to https://tools.ietf.org/html/rfc2152
 // See also below a UTF-7-IMAP codec, according to http://tools.ietf.org/html/rfc3501#section-5.1.3

eslint.config.js

@@ -12,7 +12,6 @@ module.exports = [
   }),
   {
     rules: {
-      "object-shorthand": ["off"], // Compatibility with older code
       "@stylistic/quotes": [2, "double"], // Prevent many change of code
       "new-cap": ["off"], // We need improve this
       "no-labels": ["off"], // Can remove the labels?

generation/gen-sbcs.js

@@ -1,7 +1,7 @@
 var fs  = require("fs");
 var path = require("path");
 var Iconv  = require("iconv").Iconv;
-var Buffer = require("safer-buffer").Buffer;
+var Buffer = require("buffer").Buffer;
 
 // Generate encoding families using original iconv.
 var destFileName = "encodings/sbcs-data-generated.js";

generation/research/get-iconv-encodings.js

@@ -5,7 +5,7 @@
 
 var iconv = require('iconv'),
     crypto = require('crypto');
-var Buffer = require("safer-buffer").Buffer;
+var Buffer = require("buffer").Buffer;
 
 
 var skipEncodings = {};

lib/index.js

@@ -1,7 +1,6 @@
 "use strict"
 
-var Buffer = require("safer-buffer").Buffer
-var objectAssign = require("object-assign")
+var Buffer = require("buffer").Buffer
 
 var bomHandling = require("./bom-handling")
 var iconv = module.exports
@@ -64,7 +63,7 @@ iconv._codecDataCache = Object.create(null)
 iconv.getCodec = function getCodec (encoding) {
   if (!iconv.encodings) {
     var raw = require("../encodings")
-    iconv.encodings = objectAssign(Object.create(null), raw) // Lazy load all encoding definitions.
+    iconv.encodings = Object.assign(Object.create(null), raw) // Lazy load all encoding definitions.
   }
 
   // Canonicalize encoding name: strip all non-alphanumeric chars and appended year.

lib/streams.js

@@ -1,6 +1,6 @@
 "use strict"
 
-var Buffer = require("safer-buffer").Buffer
+var Buffer = require("buffer").Buffer
 
 // NOTE: Due to 'stream' module being pretty large (~100Kb, significant in browser environments),
 // we opt to dependency-inject it instead of creating a hard dependency.
@@ -99,7 +99,7 @@ module.exports = function (streamModule) {
   }
 
   return {
-    IconvLiteEncoderStream: IconvLiteEncoderStream,
-    IconvLiteDecoderStream: IconvLiteDecoderStream
+    IconvLiteEncoderStream,
+    IconvLiteDecoderStream
   }
 }

package.json

@@ -19,7 +19,7 @@
         "url": "git://github.com/ashtuchkin/iconv-lite.git"
     },
     "engines": {
-        "node": ">=0.10.0"
+        "node": ">=6"
     },
     "scripts": {
         "lint": "eslint",
@@ -48,9 +48,5 @@
         "request": "^2.88.2",
         "semver": "^6.3.0",
         "unorm": "^1.6.0"
-    },
-    "dependencies": {
-        "object-assign": ">= 4.1.1 < 5.0.0",
-        "safer-buffer": ">= 2.1.2 < 3.0.0"
     }
 }

test/big5-test.js

@@ -1,5 +1,5 @@
 var assert = require("assert")
-var Buffer = require("safer-buffer").Buffer
+var Buffer = require("buffer").Buffer
 var join = require("path").join
 var iconv = require(join(__dirname, "/../"))
 

test/bom-test.js

@@ -1,5 +1,5 @@
 var assert = require("assert")
-var Buffer = require("safer-buffer").Buffer
+var Buffer = require("buffer").Buffer
 var join = require("path").join
 var iconv = require(join(__dirname, "/../"))
 
@@ -71,13 +71,13 @@ describe("BOM Handling", function () {
     var stripBOM = function () { bomStripped = true }
 
     var body = Buffer.concat([utf8BOM, Buffer.from(sampleStr)])
-    assert.equal(iconv.decode(body, "utf8", { stripBOM: stripBOM }), sampleStr)
+    assert.equal(iconv.decode(body, "utf8", { stripBOM }), sampleStr)
     assert(bomStripped)
 
     bomStripped = false
 
     body = Buffer.from(sampleStr)
-    assert.equal(iconv.decode(body, "utf8", { stripBOM: stripBOM }), sampleStr)
+    assert.equal(iconv.decode(body, "utf8", { stripBOM }), sampleStr)
     assert(!bomStripped)
   })
 })

test/cesu8-test.js

@@ -1,5 +1,5 @@
 var assert = require("assert")
-var Buffer = require("safer-buffer").Buffer
+var Buffer = require("buffer").Buffer
 var join = require("path").join
 var iconv = require(join(__dirname, "/../"))
 

test/cyrillic-test.js

@@ -1,5 +1,5 @@
 var assert = require("assert")
-var Buffer = require("safer-buffer").Buffer
+var Buffer = require("buffer").Buffer
 var join = require("path").join
 var iconv = require(join(__dirname, "/../"))
 

test/dbcs-test.js

@@ -1,5 +1,5 @@
 var assert = require("assert")
-var Buffer = require("safer-buffer").Buffer
+var Buffer = require("buffer").Buffer
 var join = require("path").join
 var iconv = require(join(__dirname, "/../"))
 
@@ -560,7 +560,7 @@ describe("Full DBCS encoding tests", function () {
               { return }
             }
 
-            errors.push({ input: inp.toString("hex"), strExpected: strExpected, strActual: strActual })
+            errors.push({ input: inp.toString("hex"), strExpected, strActual })
           })
 
           if (errors.length > 0) {
@@ -606,7 +606,7 @@ describe("Full DBCS encoding tests", function () {
 
             if (iconvChgs[str] == str1) { continue } // Skip iconv changes.
 
-            errors.push({ input: strToHex(str), inputChar: str, strExpected: strExpected, strActual: strActual })
+            errors.push({ input: strToHex(str), inputChar: str, strExpected, strActual })
           }
 
           if (errors.length > 0) {

test/gbk-test.js

@@ -1,6 +1,6 @@
 var fs = require("fs")
 var assert = require("assert")
-var Buffer = require("safer-buffer").Buffer
+var Buffer = require("buffer").Buffer
 var join = require("path").join
 var iconv = require(join(__dirname, "/../"))
 

test/greek-test.js

@@ -1,5 +1,5 @@
 var assert = require("assert")
-var Buffer = require("safer-buffer").Buffer
+var Buffer = require("buffer").Buffer
 var join = require("path").join
 var iconv = require(join(__dirname, "/../"))
 

test/main-test.js

@@ -1,5 +1,5 @@
 var assert = require("assert")
-var Buffer = require("safer-buffer").Buffer
+var Buffer = require("buffer").Buffer
 var join = require("path").join
 var iconv = require(join(__dirname, "/../"))
 

test/sbcs-test.js

@@ -1,6 +1,6 @@
 var assert = require("assert")
 var unorm = require("unorm")
-var Buffer = require("safer-buffer").Buffer
+var Buffer = require("buffer").Buffer
 var join = require("path").join
 var iconv = require(join(__dirname, "/../"))
 
@@ -75,7 +75,7 @@ describe("Full SBCS encoding tests", function () {
             var strActual = iconv.decode(buf, enc)
             var strExpected = convertWithDefault(conv, buf, iconv.defaultCharUnicode).toString()
 
-            if (strActual != strExpected) { errors.push({ input: buf.toString("hex"), strExpected: strExpected, strActual: strActual }) }
+            if (strActual != strExpected) { errors.push({ input: buf.toString("hex"), strExpected, strActual }) }
           }
           if (errors.length > 0) {
             assert.fail(null, null, "Decoding mismatch: <input> | <expected> | <actual> | <expected char> | <actual char>\n" +
@@ -133,7 +133,7 @@ describe("Full SBCS encoding tests", function () {
             if (iconvEquivChars[enc] && iconvEquivChars[enc][str] &&
                         strExpected == iconv.encode(iconvEquivChars[enc][str], enc).toString("hex")) { continue }
 
-            errors.push({ input: strToHex(str), inputChar: str, strExpected: strExpected, strActual: strActual })
+            errors.push({ input: strToHex(str), inputChar: str, strExpected, strActual })
           }
 
           if (errors.length > 0) {

test/shiftjis-test.js

@@ -1,5 +1,5 @@
 var assert = require("assert")
-var Buffer = require("safer-buffer").Buffer
+var Buffer = require("buffer").Buffer
 var join = require("path").join
 var iconv = require(join(__dirname, "/../"))
 

test/streams-test.js

@@ -1,7 +1,7 @@
 var mocha = require("mocha")
 var describeMocha = mocha.describe
 var assert = require("assert")
-var Buffer = require("safer-buffer").Buffer
+var Buffer = require("buffer").Buffer
 var semver = require("semver")
 var join = require("path").join
 var iconv = require(join(__dirname, "/../"))
@@ -327,7 +327,7 @@ describe("Encoding using internal modules with surrogates in separate chunks:",
   function checkUtf8EncodeStream (input) {
     return checkEncodeStream({
       encoding: "utf8",
-      input: input,
+      input,
       output: Buffer.from(input.join(""), "utf8").toString("hex")
     })
   }