mockgen: file should be closed if failed to write

uber-go / mock

GoMock is a mocking framework for the Go programming language.

Apache License 2.0

1.85k stars 106 forks source link

mockgen: file should be closed if failed to write #40

Closed alexandear closed 9 months ago

alexandear commented 1 year ago

This PR fixes a situation when f.Close() is not deferred when dst.Write returns an error. This is because of log.Fatalf internally calls os.Exit. Also, if is inverted to simplify the code.