From 916d0c21af6153338a8064ff953b579ff87c6ba6 Mon Sep 17 00:00:00 2001
From: flywind <xzsflywind@gmail.com>
Date: Sun, 1 Aug 2021 17:19:43 +0800
Subject: [PATCH] fix #18620 (#18624)

* fix #18620

* add testcase
---
 lib/system/excpt.nim       |  5 +----
 tests/exception/t18620.nim | 17 +++++++++++++++++
 2 files changed, 18 insertions(+), 4 deletions(-)
 create mode 100644 tests/exception/t18620.nim

diff --git a/lib/system/excpt.nim b/lib/system/excpt.nim
index 9f65db2fe6..4814b77b48 100644
--- a/lib/system/excpt.nim
+++ b/lib/system/excpt.nim
@@ -524,10 +524,7 @@ proc getStackTrace(e: ref Exception): string =
 proc getStackTraceEntries*(e: ref Exception): seq[StackTraceEntry] =
   ## Returns the attached stack trace to the exception `e` as
   ## a `seq`. This is not yet available for the JS backend.
-  when not defined(nimSeqsV2):
-    shallowCopy(result, e.trace)
-  else:
-    result = move(e.trace)
+  shallowCopy(result, e.trace)
 
 proc getStackTraceEntries*(): seq[StackTraceEntry] =
   ## Returns the stack trace entries for the current stack trace.
diff --git a/tests/exception/t18620.nim b/tests/exception/t18620.nim
new file mode 100644
index 0000000000..ee23f8bac9
--- /dev/null
+++ b/tests/exception/t18620.nim
@@ -0,0 +1,17 @@
+discard """
+  matrix: "--gc:arc; --gc:refc"
+"""
+
+proc hello() =
+  raise newException(ValueError, "You are wrong")
+
+var flag = false
+
+try:
+  hello()
+except ValueError as e:
+  flag = true
+  doAssert len(getStackTraceEntries(e)) > 0
+  doAssert len(getStackTraceEntries(e)) > 0
+
+doAssert flag