Approved

using namespace std; is OK in test snippets. It is not OK, for Codewars specific reasons, in preloaded and in solution snippets.

I don't think the translator will fix something that old. None of the issues seem blocking. If you want, you can fix it yourself.

fixed there

fixed there

It is argued that knowing the function signature should be enough because target platform is well known, and ABI can be naturally detemined based on this information.

Having said that, I can see value of having more detailed info in a white kata, where users are expected to not be that experienced.

Approved

Thanks, fixed :)

fixed + better assertion messages + made the initial code compile

Approved.

You're right.
I just didn't know that float(...) ignores surrounding whitespaces.
There is always something new to learn😀

merged

Its a duplicate of one of the issues below.

Fixed.

If you use lodash, it can be _.sample(charset) and no problems with indexing :)

EDIT: fixed