search

flyx / NimYAML

YAML implementation for Nim
https://nimyaml.org
Other
186 stars 36 forks source link

Catchable exception warnings #133

Closed jfilby closed 1 year ago

jfilby commented 1 year ago

When I write code that call's this package's load() function I get warnings:

/.../nexus/src/nexus/cmd/service/generate/modules/module_utils.nim(118, 9) template/generic instantiation of `load` from here
/home/jasonfi/.nimble/pkgs/yaml-1.1.0/yaml/serialization.nim(1383, 12) template/generic instantiation of `yAssert` from here
/home/jasonfi/.nimble/pkgs/yaml-1.1.0/yaml/private/internal.nim(42, 9) Warning: The bare except clause is deprecated; use `except CatchableError:` instead [BareExcept]

I tried wrapping the call in try/except CatchableError, but it didn't help. I think the problem is in the YAML module. This is quite a new warning. I'm using Nim v1.6.12.

flyx commented 1 year ago

This is already fixed but has not found its way in a release yet. Probably should do one.

flyx commented 1 year ago

Fixed in released NimYAML 2.0.0.