1st: It’s just an extra safety check whether InputComponent is null or not, nothing major - we are calling the base function using Super::SetupPlayerInputComponent(InputComponent); so the check(…) will happen already.
2st: Had a quick look, and doesn’t look like this required to be virtual afterall!