Apply modernize suggestions

.github/workflows/run_tests.yml

@@ -0,0 +1,119 @@
+name: Tests
+
+on:
+  pull_request:
+    branches:
+      - master
+    types: [opened, synchronize]
+    paths-ignore:
+      - '*.md'
+  workflow_dispatch:
+
+env:
+  GO111MODULE: "on"
+
+jobs:
+  test_cover:
+    name: Coverage
+    runs-on: ubuntu-latest
+
+    env:
+      CGO_ENABLED: 0
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+
+      - name: Set up Go
+        uses: actions/setup-go@v5
+        with:
+          go-version: 1.26
+
+      - name: Write coverage profile
+        run: go test -v ./... -coverprofile=./coverage.txt -covermode=atomic
+
+      - name: Upload coverage results to Codecov
+        uses: codecov/codecov-action@v2
+        with:
+          fail_ci_if_error: false
+          path_to_write_report: ./coverage.txt
+          verbose: true
+
+  tests:
+    name: Go
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        go_versions: [ '1.25', '1.26' ]
+        os: [ubuntu-latest, windows-latest, macos-latest]
+        exclude:
+          # Only latest Go version for Windows and MacOS.
+          - os: windows-latest
+            go_versions: '1.25'
+          - os: macos-latest
+            go_versions: '1.25'
+          # Exclude latest Go version for Ubuntu as Coverage uses it.
+          - os: ubuntu-latest
+            go_versions: '1.26'
+      fail-fast: false
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+
+      - name: Set up Go
+        uses: actions/setup-go@v5
+        with:
+          go-version: '${{ matrix.go_versions }}'
+
+      - name: Run tests
+        run: go test -v -race ./...
+
+  lint:
+    name: Linter
+    uses: nspcc-dev/.github/.github/workflows/go-linter.yml@master
+
+  codeql:
+    name: CodeQL
+    runs-on: ubuntu-latest
+
+    strategy:
+      fail-fast: false
+      matrix:
+        language: [ 'go' ]
+        # CodeQL supports [ 'cpp', 'csharp', 'go', 'java', 'javascript', 'python' ]
+        # Learn more:
+        # https://docs.github.com/en/free-pro-team@latest/github/finding-security-vulnerabilities-and-errors-in-your-code/configuring-code-scanning#changing-the-languages-that-are-analyzed
+
+    steps:
+    - name: Checkout repository
+      uses: actions/checkout@v4
+
+    # Initializes the CodeQL tools for scanning.
+    - name: Initialize CodeQL
+      uses: github/codeql-action/init@v3
+      with:
+        languages: ${{ matrix.language }}
+        # If you wish to specify custom queries, you can do so here or in a config file.
+        # By default, queries listed here will override any specified in a config file.
+        # Prefix the list here with "+" to use these queries and those in the config file.
+        # queries: ./path/to/local/query, your-org/your-repo/queries@main
+
+    # Autobuild attempts to build any compiled languages  (C/C++, C#, or Java).
+    # If this step fails, then you should remove it and run the build manually (see below)
+    - name: Autobuild
+      uses: github/codeql-action/autobuild@v3
+
+    # ℹ️ Command-line programs to run using the OS shell.
+    # 📚 https://git.io/JvXDl
+
+    # ✏️ If the Autobuild fails above, remove it and uncomment the following three lines
+    #    and modify them (or add more) to build your code if your project
+    #    uses a compiled language
+
+    #- run: |
+    #   make bootstrap
+    #   make release
+
+    - name: Perform CodeQL Analysis
+      uses: github/codeql-action/analyze@v3

README.md

@@ -1,21 +1,18 @@
-go-ordered-json
-===============
+.NET-compatible JSON library
+============================
 
-There are some legacy/stupid applications[1] that you need to interoperate with,
-and they for whatever reason require that the JSON you're using is ordered in
-a particular way (contrary to the JSON specifications).
+It's made for 100% compatibility with the JSON variation used by
+[Neo blockchain](https://github.com/neo-project/). There are three problems
+there:
+ * it's ordered (that's why it's a fork of [go-ordered-json](https://github.com/virtuald/go-ordered-json))
+ * it has different conventions regarding control and "special" symbols
+ * it has different conventions wrt incorrect UTF-8
 
-Unfortunately, the golang authors are not willing to support such a broken use
-case, so on [their advice](https://groups.google.com/forum/#!topic/golang-dev/zBQwhm3VfvU)
-this is a fork of the golang encoding/json package, with the ordered JSON
-support originating with a patch from
-[Peter Waldschmidt](https://go-review.googlesource.com/c/7930/).
+The primary user of this library is [NeoGo](https://github.com/nspcc-dev/neo-go/),
+it has to be 100% compatible with C# implementation to correctly process
+transactions, that's why we're maintaining this library and solving any
+inconsistencies with .NET libraries if found.
 
 **If you can, you should avoid using this package**. However, if you can't
 avoid it, then you are welcome to. Provided under the MIT license, just like
 golang.
-
-Known broken applications 
--------------------------
-
-* [1][Windows Communication Foundation Json __type ordering](https://docs.microsoft.com/en-us/dotnet/framework/wcf/feature-details/stand-alone-json-serialization#type-hint-position-in-json-objects)

bench_test.go

@@ -13,7 +13,7 @@ package json
 import (
 	"bytes"
 	"compress/gzip"
-	"io/ioutil"
+	"io"
 	"os"
 	"strings"
 	"testing"
@@ -47,7 +47,7 @@ func codeInit() {
 	if err != nil {
 		panic(err)
 	}
-	data, err := ioutil.ReadAll(gz)
+	data, err := io.ReadAll(gz)
 	if err != nil {
 		panic(err)
 	}
@@ -83,7 +83,7 @@ func BenchmarkCodeEncoder(b *testing.B) {
 		b.StartTimer()
 	}
 	b.RunParallel(func(pb *testing.PB) {
-		enc := NewEncoder(ioutil.Discard)
+		enc := NewEncoder(io.Discard)
 		for pb.Next() {
 			if err := enc.Encode(&codeStruct); err != nil {
 				b.Fatal("Encode:", err)
@@ -138,13 +138,13 @@ func BenchmarkDecoderStream(b *testing.B) {
 	var buf bytes.Buffer
 	dec := NewDecoder(&buf)
 	buf.WriteString(`"` + strings.Repeat("x", 1000000) + `"` + "\n\n\n")
-	var x interface{}
+	var x any
 	if err := dec.Decode(&x); err != nil {
 		b.Fatal("Decode:", err)
 	}
 	ones := strings.Repeat(" 1\n", 300000) + "\n\n\n"
 	b.StartTimer()
-	for i := 0; i < b.N; i++ {
+	for i := range b.N {
 		if i%300000 == 0 {
 			buf.WriteString(ones)
 		}