fixes #22163; use {.push warning[BareExcept]:off.} to override settings temporarily (#21390)

* use `{.push warning[BareExcept]:off.}` to override settings temporarily

* likewise, suppress expect
This commit is contained in:
ringabout
2023-06-27 08:03:52 +08:00
committed by GitHub
parent 4546f5dfe5
commit 4ce3a68e79
2 changed files with 10 additions and 6 deletions

View File

@@ -98,7 +98,7 @@ template doAssertRaises*(exception: typedesc, code: untyped) =
const begin = "expected raising '" & astToStr(exception) & "', instead"
const msgEnd = " by: " & astToStr(code)
template raisedForeign {.gensym.} = raiseAssert(begin & " raised foreign exception" & msgEnd)
{.warning[BareExcept]:off.}
{.push warning[BareExcept]:off.}
when Exception is exception:
try:
if true:
@@ -117,6 +117,6 @@ template doAssertRaises*(exception: typedesc, code: untyped) =
mixin `$` # alternatively, we could define $cstring in this module
raiseAssert(begin & " raised '" & $e.name & "'" & msgEnd)
except: raisedForeign()
{.warning[BareExcept]:on.}
{.pop.}
if wrong:
raiseAssert(begin & " nothing was raised" & msgEnd)