Nikolay Sivov (@nsivov) commented about dlls/scrrun/filesystem.c:
static HRESULT WINAPI textstream_WriteBlankLines(ITextStream *iface, LONG lines) {
- FIXME("%p, %ld stub\n", iface, lines);
- struct textstream *This = impl_from_ITextStream(iface);
I agree it's inconsistent, for newer changes I've been trying to use meaningful names instead of "This".
Looks good otherwise.