Remove unneeded returns in multiple methods

1. According to EntityFormInterface the save() method has a return value of type int. (I brought that one back)
2. The return type of validateForm() and submitForm() is not specified on the FormInterface - so even though it doesn't seem to be used anywhere I'm wondering if it really hurts if we pass through the return value.

Did your IDE or some tool notify you about potential issues?