From cea961e0801edd6eca01f2e82e63ef7d1de6bcfa Mon Sep 17 00:00:00 2001
From: Sebastien Binet <binet@cern.ch>
Date: Thu, 19 Nov 2020 11:15:43 +0100
Subject: [PATCH] npz: do not sort keys, use archive/zip.File order

---
 npz/npz_example_test.go              | 4 ++--
 npz/reader.go                        | 3 ---
 testdata/data_float64_corder.npz.txt | 8 ++++----
 testdata/data_float64_forder.npz.txt | 8 ++++----
 4 files changed, 10 insertions(+), 13 deletions(-)

diff --git a/npz/npz_example_test.go b/npz/npz_example_test.go
index 09fcefc..9cea4f3 100644
--- a/npz/npz_example_test.go
+++ b/npz/npz_example_test.go
@@ -39,8 +39,8 @@ func ExampleOpen() {
 	fmt.Printf("arr1: %v\n", f1)
 
 	// Output:
-	// arr0.npy: Header{Major:1, Minor:0, Descr:{Type:<f8, Fortran:false, Shape:[2 3]}}
 	// arr1.npy: Header{Major:1, Minor:0, Descr:{Type:<f8, Fortran:false, Shape:[6 1]}}
+	// arr0.npy: Header{Major:1, Minor:0, Descr:{Type:<f8, Fortran:false, Shape:[2 3]}}
 	// arr0: [0 1 2 3 4 5]
 	// arr1: [0 1 2 3 4 5]
 }
@@ -82,8 +82,8 @@ func ExampleReader() {
 	fmt.Printf("arr1: %v\n", f1)
 
 	// Output:
-	// arr0.npy: Header{Major:1, Minor:0, Descr:{Type:<f8, Fortran:false, Shape:[2 3]}}
 	// arr1.npy: Header{Major:1, Minor:0, Descr:{Type:<f8, Fortran:false, Shape:[6 1]}}
+	// arr0.npy: Header{Major:1, Minor:0, Descr:{Type:<f8, Fortran:false, Shape:[2 3]}}
 	// arr0: [0 1 2 3 4 5]
 	// arr1: [0 1 2 3 4 5]
 }
diff --git a/npz/reader.go b/npz/reader.go
index ca2f1e7..99d76d7 100644
--- a/npz/reader.go
+++ b/npz/reader.go
@@ -9,7 +9,6 @@ import (
 	"fmt"
 	"io"
 	"os"
-	"sort"
 
 	"github.com/sbinet/npyio/npy"
 )
@@ -69,7 +68,6 @@ func Open(name string) (*Reader, error) {
 	for i, f := range rz.File {
 		keys[i] = f.Name
 	}
-	sort.Strings(keys)
 
 	return &Reader{
 		r:    r,
@@ -91,7 +89,6 @@ func NewReader(r io.ReaderAt, size int64) (*Reader, error) {
 	for i, f := range rz.File {
 		keys[i] = f.Name
 	}
-	sort.Strings(keys)
 
 	return &Reader{
 		r:    r,
diff --git a/testdata/data_float64_corder.npz.txt b/testdata/data_float64_corder.npz.txt
index 8d80da1..a563dd6 100644
--- a/testdata/data_float64_corder.npz.txt
+++ b/testdata/data_float64_corder.npz.txt
@@ -1,9 +1,9 @@
 ================================================================================
 file: testdata/data_float64_corder.npz
-entry: arr0.npy
-npy-header: Header{Major:1, Minor:0, Descr:{Type:<f8, Fortran:false, Shape:[2 3]}}
-data = [0 1 2 3 4 5]
-
 entry: arr1.npy
 npy-header: Header{Major:1, Minor:0, Descr:{Type:<f8, Fortran:false, Shape:[6 1]}}
 data = [0 1 2 3 4 5]
+
+entry: arr0.npy
+npy-header: Header{Major:1, Minor:0, Descr:{Type:<f8, Fortran:false, Shape:[2 3]}}
+data = [0 1 2 3 4 5]
diff --git a/testdata/data_float64_forder.npz.txt b/testdata/data_float64_forder.npz.txt
index 630bdf5..5da49a0 100644
--- a/testdata/data_float64_forder.npz.txt
+++ b/testdata/data_float64_forder.npz.txt
@@ -1,9 +1,9 @@
 ================================================================================
 file: testdata/data_float64_forder.npz
-entry: arr0.npy
-npy-header: Header{Major:1, Minor:0, Descr:{Type:<f8, Fortran:true, Shape:[2 3]}}
-data = [0 1 2 3 4 5]
-
 entry: arr1.npy
 npy-header: Header{Major:1, Minor:0, Descr:{Type:<f8, Fortran:true, Shape:[6 1]}}
 data = [0 1 2 3 4 5]
+
+entry: arr0.npy
+npy-header: Header{Major:1, Minor:0, Descr:{Type:<f8, Fortran:true, Shape:[2 3]}}
+data = [0 1 2 3 4 5]