Nim icon indicating copy to clipboard operation
Nim copied to clipboard

Published 20 hours ago • verified publisher icon nim-lang

support cstring in `case`

Open metagn opened this issue 2 years ago • 4 comments

closes https://github.com/nim-lang/RFCs/issues/350

Again a bit sloppy, but should work.

Can add experimental switch or whatever if desired.

metagn avatar Aug 01 '22 13:08 metagn

Does it works when the cstring is nil ?.

juancarlospaco avatar Aug 01 '22 14:08 juancarlospaco

I thought it would be complicated to make it work but it's not

metagn avatar Aug 01 '22 14:08 metagn

This is awesome! Since this will be in version 2.0 it doesn't need an experimental switch. It does need a changelog entry though and documentation updates.

Araq avatar Aug 04 '22 09:08 Araq

Done

metagn avatar Aug 04 '22 12:08 metagn

Thanks for your hard work on this PR! The lines below are statistics of the Nim compiler built from a95b6391fd353074daf2dbfed4d73e8d57f314ca

Hint: mm: orc; threads: on; opt: speed; options: -d:release 163966 lines; 15.268s; 842.176MiB peakmem

github-actions[bot] avatar Sep 01 '22 16:09 github-actions[bot]