fix string map and add toString

Author: 9il

source/mir/algebraic_alias/json.d

@@ -124,4 +124,8 @@ unittest
 
     JsonAlgebraic[string] aa = object.toAA;
     object = StringMap!JsonAlgebraic(aa);
+
+    JsonAlgebraic fromAA = ["a" : JsonAlgebraic(3), "b" : JsonAlgebraic("b")];
+    assert(fromAA.get!(StringMap!JsonAlgebraic)["a"] == 3);
+    assert(fromAA.get!(StringMap!JsonAlgebraic)["b"] == "b");
 }

source/mir/serde.d

@@ -529,8 +529,6 @@ alias serdeGetIgnoreOutIf(alias symbol) = naryFun!(TemplateArgsOf!(getUDA!(symbo
 /++
 Attributes to conditional ignore field during serialization.
 
-The attribute can be combined with $(LREF serdeOrderedIn), $(LREF serdeRealOrderedIn).
-
 The predicate should be aplied to the aggregate value, not to the member.
 
 See_also: $(LREF serdeIgnoreIfAggregate) $(LREF serdeIgnoreOutIf), $(LREF serdeIgnoreInIfAggregate)
@@ -544,7 +542,7 @@ alias serdeGetIgnoreOutIfAggregate(alias symbol) = naryFun!(TemplateArgsOf!(getU
 /++
 Attributes to conditional ignore field during deserialization.
 
-The predicate should be aplied to the aggregate value, not to the member.
+The attribute should be combined with $(LREF serdeRealOrderedIn) applied on the aggregate.
 
 See_also: $(LREF serdeIgnoreIfAggregate) $(LREF serdeIgnoreOutIfAggregate) $(LREF serdeIgnoreIn)
 +/
@@ -557,7 +555,7 @@ alias serdeGetIgnoreInIfAggregate(alias symbol) = naryFun!(TemplateArgsOf!(getUD
 /++
 Attributes to conditional ignore field during serialization and deserialization.
 
-The attribute can be combined with $(LREF serdeOrderedIn), $(LREF serdeRealOrderedIn).
+The attribute should be combined with $(LREF serdeRealOrderedIn) applied on the aggregate.
 
 The predicate should be aplied to the aggregate value, not to the member.
 

source/mir/string_map.d

@@ -111,6 +111,34 @@ struct StringMap(T, U = uint)
         assert(map.findPos("key") == 0);
     }
 
+    ///
+    string toString() const
+    {
+        import mir.format: stringBuf;
+        stringBuf buffer;
+        toString(buffer);
+        return buffer.data.idup;
+    }
+
+    ///ditto
+    void toString(W)(scope ref W w) const
+    {
+        bool next;
+        w.put('[');
+        import mir.format: printEscaped, EscapeFormat, print;
+        foreach (i, ref value; values)
+        {
+            if (next)
+                w.put(`, `);
+            next = true;
+            w.put('\"');
+            printEscaped!(char, EscapeFormat.ion)(w, keys[i]);
+            w.put(`": `);
+            print(w, value);
+        }
+        w.put(']');
+    }
+
     /++
     Constructs an associative array using keys and values.
     Params:
@@ -704,7 +732,7 @@ private struct StructImpl(T, U = uint)
     this()(string[] keys, T[] values) @trusted pure nothrow
     {
         import mir.array.allocation: array;
-        import mir.ndslice.sorting: sort;
+        import mir.ndslice.sorting: makeIndex;
         import mir.ndslice.topology: iota, indexed;
         import mir.string_table: smallerStringFirst;
 
@@ -714,13 +742,9 @@ private struct StructImpl(T, U = uint)
         _length = keys.length;
         _keys = keys.ptr;
         _values = values.ptr;
-        _indices = keys.length.iota!U.array.ptr;
+        _indices = keys.makeIndex!(U, smallerStringFirst).ptr;
         auto sortedKeys = _keys.indexed(indices);
-        sortedKeys.sort!smallerStringFirst;
-        size_t maxKeyLength;
-        foreach (ref key; keys)
-            if (key.length > maxKeyLength)
-                maxKeyLength = key.length;
+        size_t maxKeyLength = sortedKeys[$ - 1].length;
         _lengthTable = new U[maxKeyLength + 2];
 
         size_t ski;