Fixed (along with C translation warning) in this fork. Please approve.

Rust translation

Of course I need it, just search how it is used in floodfill and think of what happens if this check is skipped.

Well, exactly this is the case when &Vec<Vec<>> is better. I've started with

type Matrix = Vec<Vec<i32>>;

but then decided not to overcomplicate the code. We have a matrix type here, represented with Vec<Vec<>>, not an arbitrary slice of Vecs.

Also, there's nothing wrong with negatives in matrix, they should be allowed, but I don't see any actual reason to test them.

Added better diagnostic messages.

Fixed the incorrect function definition in solution setup.

Fixed

Approved

I've fixed C++ in https://www.codewars.com/kumite/6711202e8abc89c1527bbf8d
Can anyone merge?

Rust translation

Rust translation

C version doesn't need an additional argument n, numbers in string can be easily counted.

Rust translation

I don't see any sense of making it preloaded. Preloaded is something that can be used (and/or manipulated) in the task. And sample tests are for people to check their code by themselves; if they can't see what is checked, what the point of putting it into sample test at all?

It wasn't a suggestion because I've never said what to do with it.

And now I really have a suggestion :)

Rust translation