Remove superfluous helper variable in Randomizer::getBytes()

[joshuaruesweg]

I am currently looking at the random extension to extend it in the future. In the process, I noticed that an auxiliary variable was introduced at this point, which is not needed in my eyes. I don't know if the compiler doesn't optimise this variable anyway, but I wanted to make the pull request anyway, because I think the code is more understandable this way.

[cmb69]

Thank you for the PR! I'd leave that as is (for better readability regarding the names and the types (unsigned vs. signed)), but leave that to @zeriyoshi and @TimWolla to decide.

[TimWolla]

for better readability regarding the names and the types (unsigned vs. signed)

I agree it's a little more obvious regarding the types. I just had to look up how exactly a comparison between unsigned and signed works. However I find the use of two different variable names for the same value confusing. Especially since the trailing NUL byte is added based on length, not required_size. So that would be a “yes” from me.

Looking at the function I'd also move the declaration of result into the while() loop, as it is not used outside the loop, I've added such a commit.

I'll let zeriyoshi have the final review.

[zeriyoshi]

Thanks for your help in improving it! Looks like a good change to me.