fix LockingTextReader: issues 13686 and 12320

[Issue 13686 (Reading unicode string with readf ("%s") produces a wrong
string)](https://issues.dlang.org/show_bug.cgi?id=13686) is fixed by reading all chars of a multibyte sequence and decoding.
Before, each char was mistaken for a dchar.

[Issue 12320 (std.stdio.LockingTextReader populates .front in .empty)](https://issues.dlang.org/show_bug.cgi?id=12320) is
fixed by moving the work from empty to popFront.
It's not a big deal, just that generally PRs should avoid gratuitous whitespace changes (unless the whole point of the PR is to fix whitespace issues).