summary refs log tree commit diff stats
diff options
context:
space:
mode:
authorJake Leahy <jake@leahy.dev>2023-03-03 06:27:10 +1100
committerGitHub <noreply@github.com>2023-03-02 14:27:10 -0500
commitd4d28f2ffe522d2509dccfa7be0eef709732d93f (patch)
tree4f19bcc1c594ce8933474e82343f0222f2af1a00
parent38d299dfc0c9f1315cf412aa76cb041642d62f07 (diff)
downloadNim-d4d28f2ffe522d2509dccfa7be0eef709732d93f.tar.gz
Allow `futureLogging` in release builds (#21448)
* Add test case

* Fixes 21447: Keeps stackTrace around when using futureLogging

* Remove extra whitespace
-rw-r--r--lib/pure/asyncfutures.nim2
-rw-r--r--tests/async/t21447.nim6
2 files changed, 7 insertions, 1 deletions
diff --git a/lib/pure/asyncfutures.nim b/lib/pure/asyncfutures.nim
index edf5f7277..51aaaca36 100644
--- a/lib/pure/asyncfutures.nim
+++ b/lib/pure/asyncfutures.nim
@@ -29,7 +29,7 @@ type
     finished: bool
     error*: ref Exception              ## Stored exception
     errorStackTrace*: string
-    when not defined(release):
+    when not defined(release) or defined(futureLogging):
       stackTrace: seq[StackTraceEntry] ## For debugging purposes only.
       id: int
       fromProc: string
diff --git a/tests/async/t21447.nim b/tests/async/t21447.nim
new file mode 100644
index 000000000..e4f7ae31f
--- /dev/null
+++ b/tests/async/t21447.nim
@@ -0,0 +1,6 @@
+discard """
+  action: "compile"
+  cmd: "nim c -d:release -d:futureLogging $file"
+"""
+
+import std/asyncdispatch