These are chat archives for kbknapp/clap-rs

24th
Jan 2018
Casey Rodarmor
@casey
Jan 24 2018 11:54
@kbknapp Sorry for the slow response! It's kind of a silly test on my part, I think I'll just remove it. Would it be possible to sort the arguments, or find some other stable ordering, so that output is deterministic?
Kevin K.
@kbknapp
Jan 24 2018 13:39
Potentially. I'll start looking at it again, the reason it was removed was there is a pretty big perf boost by doing it this non-deterministic way. There might be a way to get the best of both worlds though, so I'll look into it :)
Casey Rodarmor
@casey
Jan 24 2018 14:51
Can you point me to the code?
If there's a big perf boost, then agreed it might not be worth it.
Kevin K.
@kbknapp
Jan 24 2018 14:58
the testing code, or the code that changed?
Here's the commit: kbknapp/clap-rs@03e413d
Casey Rodarmor
@casey
Jan 24 2018 15:13
Yup, the code that changed, thanks!