bugfix: LinkedList.of() and Stream.of() (without parameters) would re…

…turn a list with a single item, undefined. That call was impossible in typescript (wouldn't compile) but it was possible in javascript. Now make it that it returns an empty list as you would expect, while retaining the intended feature, that if there are parameters, the compiler knows we're dealing with a non-empty list (thanks to typescript overloads).

src/LinkedList.ts

@@ -48,8 +48,10 @@ export class LinkedListStatic {
     /**
      * Create a LinkedList with the elements you give.
      */
-    of<T>(elt:T,...elts:T[]): ConsLinkedList<T> {
-        return new ConsLinkedList(elt, LinkedList.ofIterable(elts));
+    of<T>(elt: T, ...elts:T[]): ConsLinkedList<T>;
+    of<T>(...elts:T[]): LinkedList<T>;
+    of<T>(...elts:T[]): LinkedList<T> {
+        return LinkedList.ofIterable(elts);
     }
 
     /**

src/Stream.ts

@@ -54,8 +54,10 @@ export class StreamStatic {
     /**
      * Create a Stream with the elements you give.
      */
-    of<T>(elt: T, ...elts:T[]): ConsStream<T> {
-        return new ConsStream(elt, Lazy.of(()=>Stream.ofIterable(elts)));
+    of<T>(elt:T, ...elts:T[]): ConsStream<T>;
+    of<T>(...elts:T[]): Stream<T>;
+    of<T>(...elts:T[]): Stream<T> {
+        return Stream.ofIterable(elts);
     }
 
     /**

tests/Seq.ts

@@ -24,6 +24,8 @@ export function runTests(seqName: string,
     const nonEmptySeqName = nonEmptySeqName_ || seqName;
     CollectionTest.runTests(seqName, of, empty);
     describe(seqName + " creation", () => {
+        it("handles of() without parameters ok", () => assert.deepEqual(
+            [], of().toArray()));
         it("creates from a JS array", () => assert.deepEqual(
             ["a","b", "c"],
             ofIterable<string>(["a","b","c"]).toArray()));