Switch object rename around to be more intuitive.

Author: technicallyagd

readme.md

@@ -28,7 +28,7 @@ let tim = Person(name: "Tim", job: "Fluffer")
 
 tim.lunpack(name, job) # creates name, job with let and assign respective member values to them
 
-tim.lunpack(job, name = otherName) # you can also unpack into custom names using '='
+tim.lunpack(job, otherName = name) # you can also unpack into custom names using '='
 
 # will not invoke proc chain multiple times
 tim.someProcWithSideEffects(arg).lunpack(name, job)
@@ -40,6 +40,8 @@ tim.someProcWithSideEffects(arg).lunpack(name, job)
 #   job = someUniqueSym1212498.job
 ```
 
+See `tests/test1.nim` for more usages.
+
 ## TODO
 
 - Docs

src/unpack.nim

@@ -27,8 +27,8 @@ proc unpackInternal(srcNode, dests: NimNode; sec,
       case dest.kind:
       of nnkExprEqExpr:
         var newNode = stNode.copyNimTree
-        newNode[0] = dest[1]
-        newNode[^1] = nnkDotExpr.newTree(src, dest[0])
+        newNode[0] = dest[0]
+        newNode[^1] = nnkDotExpr.newTree(src, dest[1])
         section.add(newNode)
       else:
         var newNode = stNode.copyNimTree

tests/test1.nim

@@ -88,11 +88,11 @@ suite "Object meber unpacking":
     check secreteCounter == 1
 
   test "should be able to rename object member with '=' sign":
-    tim.lunpack(name = otherName)
+    tim.lunpack(otherName = name)
 
     check otherName == timName
 
-    tim.lunpack(job, name = yetAnotherName) # and is order-agnostic.
+    tim.lunpack(job, yetAnotherName = name) # and is order-agnostic.
 
     check yetAnotherName == timName
     check job == fluffer