I could restore that sanity check (but checking with the `size` value I compute immediately after), it's just that it seems somewhat redundant. We just asked for a buffer large at least `capacity` and got a non-`FAILED(hr)` response, I'd be quite surprised if whatever serviced the allocation request lied to us.
Yeah, Hence like I said—I don't mind calling it excessive and removing it, but then we don't really need the GetMaxLength() either.