summary refs log tree commit diff stats
path: root/tests
diff options
context:
space:
mode:
authorTimothee Cour <timothee.cour2@gmail.com>2021-05-07 10:36:41 -0700
committerGitHub <noreply@github.com>2021-05-07 19:36:41 +0200
commit98c29c01eb91a0c6ce7da09380a272eebe6bca6f (patch)
treec7b9da12072df412fb52dffc3ac9f4c7c9b73be5 /tests
parent51f3ef6cb8ac9fbbc9533c1de2af219dc4ecabe7 (diff)
downloadNim-98c29c01eb91a0c6ce7da09380a272eebe6bca6f.tar.gz
fix #14873 properly by skipping `abi` field in importc type (#17944)
* fix #14873 properly by skipping `abi` field in importc type

* add test

* fix test for windows
Diffstat (limited to 'tests')
-rw-r--r--tests/stdlib/uselocks.nim18
1 files changed, 17 insertions, 1 deletions
diff --git a/tests/stdlib/uselocks.nim b/tests/stdlib/uselocks.nim
index e9d23f9d9..e82de8adf 100644
--- a/tests/stdlib/uselocks.nim
+++ b/tests/stdlib/uselocks.nim
@@ -11,5 +11,21 @@ proc use* (m: var MyType): int =
     result = 3
 
 block:
+  # bug #14873
   var l: Lock
-  doAssert $l == "()"
+  doAssert ($l).len > 0
+    # on posix, "()", on windows, something else, but that shouldn't be part of the spec
+    # what matters is that `$` doesn't cause the codegen bug mentioned
+
+when true: # intentional
+  # bug https://github.com/nim-lang/Nim/issues/14873#issuecomment-784241605
+  type
+    Test = object
+      path: string # Removing this makes both cases work.
+      lock: Lock
+  # A: This is not fine.
+  var a = Test()
+  proc main(): void =
+    # B: This is fine.
+    var b = Test()
+  main()