Minor fixes

[pr0g]

This is a simple PR with some minor fixes I spotted while reading through the source code.

• Minor typo fixes
• Minor grammatical fixes
• Removal of trailing whitespace
• Removal of tab characters
• Removal of redundant semicolons (e.g. ; at end of a namespace)
• std::move of constructor parameter passed by value (to avoid additional copies)
• use of false literal instead of 0
• Removal of redundant override keyword when used with final
• use cassert and cstdint C++ headers as opposed to assert.h and stdint.h C headers in TaskScheduler.h

(The change does not include switching .cpp files to use nullptr instead of NULL, but I would happily do that 🙂 It also looks like RotateLeft is unused and could be removed).

I also am not sure what the comment...

// check if have tasks inside threadState change but before waiting

on line 1164 of TaskScheduler.cpp means exactly, if this could be rephrased in a separate change that would be great!

I won't be offended at all if you aren't interested in the changes (they're pretty much all superficial) or if you'd rather implement them yourself and not merge this PR that's also fine. I just thought I might as well share it in case it was useful as I was noodling around with this in my fork.

Thanks!

[dougbinks]

This is a lot in one PR so may take me some time to consider.

Note that cassert on windows includes yvals_core.h which is 2000 lines long on top of assert.h. cstdint is better, but still adds many more lines so I'm not in favour of swapping unless there's good reason to do so.

[pr0g]

I can split it up and only include one part at a time, so don't worry about going through the whole thing right now. I'll close this one for now and open a separate one with only 1 set of changes per PR.