search

c-blake / cligen

Nim library to infer/generate command-line-interfaces / option / argument parsing; Docs at
https://c-blake.github.io/cligen/
ISC License
496 stars 23 forks source link

[Bug] nim 1.0.6/1.4.8 doesn't have isNaN/isnan #199

Closed BlackLotus closed 2 years ago

BlackLotus commented 2 years ago

strUt.nim tries to import isNaN. I wrote a simple one liner as a workaround, but I guess it's not elegant enough to integrate it into cligen...

c-blake commented 2 years ago

Oh? Oops. What I usually do here is a when declared(isNaN) with the impl from the new stdib copied in if not declared and that can work. I can do this tomorrow AM, or you can try. Happy to coach a PR.

BlackLotus commented 2 years ago

Sorry for the delayed answer. Like I said I had it fixed in my build, but I don't feel very comfortable with nim yet. My solution was way short/simpler and probably not good enough at all. Anyway. Thank you very much :)

c-blake commented 2 years ago

Sure. Easy to fix. Out of curiosity, what in cligen/strUt are you trying to use?