From 5f82fe2d62ed82baf33034b240098d1c03078098 Mon Sep 17 00:00:00 2001 From: Angel Ezquerra Date: Thu, 21 Mar 2024 20:49:01 +0100 Subject: [PATCH] Remove `imported and not used: 'sequninit' [UnusedImport]` warning This warning only happens with devel. The fix is to make sequninit.nim "used" when newSeqUninit is not needed. --- src/arraymancer/private/sequninit.nim | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/arraymancer/private/sequninit.nim b/src/arraymancer/private/sequninit.nim index 8b6daf0d..4bb397d9 100644 --- a/src/arraymancer/private/sequninit.nim +++ b/src/arraymancer/private/sequninit.nim @@ -14,7 +14,11 @@ # from Nim https://github.com/nim-lang/Nim/pull/22739 on the stdlib provides a # `newSeqUninit` for types supporting `supportsCopyMem` -when not declared(newSeqUninit): +when declared(newSeqUninit): + # Avoid an "imported and not used: 'sequninit' [UnusedImport]" warning + # when nim has a built-in newSeqUninit function (i.e. post nim 2.0.2) + {.used.} +else: func newSeqUninit*[T](len: Natural): seq[T] {.inline.} = ## Creates an uninitialzed seq. ## Contrary to newSequnitialized in system.nim this works for any subtype T