jyapayne/Nim
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Allow futureLogging in release builds (#21448)

Browse source 
* Add test case

* Fixes 21447: Keeps stackTrace around when using futureLogging

* Remove extra whitespace
This commit is contained in:
Jake Leahy 2023-03-03 06:27:10 +11:00 • committed by GitHub
commit d4d28f2ffe
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 7 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
lib/pure/asyncfutures.nim
View file

@ -29,7 +29,7 @@ type
finished: bool finished: bool
error*: ref Exception ## Stored exception error*: ref Exception ## Stored exception
errorStackTrace*: string errorStackTrace*: string
when not defined(release): when not defined(release) or defined(futureLogging):
stackTrace: seq[StackTraceEntry] ## For debugging purposes only. stackTrace: seq[StackTraceEntry] ## For debugging purposes only.
id: int id: int
fromProc: string fromProc: string

6
tests/async/t21447.nim Normal file
View file

@ -0,0 +1,6 @@
discard """
action: "compile"
cmd: "nim c -d:release -d:futureLogging $file"
"""
import std/asyncdispatch