remove unnecessary dup

Author: narimiran

src/nimble.nim

@@ -1480,7 +1480,7 @@ proc validateDevModeDepsWorkingCopiesBeforeLock(
     }
 
   # Remove not errors from the errors set.
-  for name, error in common.dup(errors):
+  for name, error in errors:
     if error.kind in notAnErrorSet:
       errors.del name
 
@@ -1699,7 +1699,7 @@ proc sync(options: Options) =
   var errors: ValidationErrors
   findValidationErrorsOfDevDepsWithLockFile(pkgInfo, options, errors)
 
-  for name, error in common.dup(errors):
+  for name, error in errors:
     if error.kind == vekWorkingCopyNeedsSync:
       if not options.action.listOnly:
         syncWorkingCopy(name, error.path, pkgInfo, options)

src/nimblepkg/common.nim

@@ -52,8 +52,6 @@ template newClone*[T: not ref](obj: T): ref T =
   result[] = obj
   result
 
-proc dup*[T](obj: T): T = obj
-
 proc `$`*(p: ptr | ref): string = cast[int](p).toHex
   ## Converts the pointer `p` to its hex string representation.