Page MenuHomePhabricator

bench: Don't return a bool from main

Authored by Fabien on Jul 31 2019, 14:21.


Return `EXIT_SUCCESS` from `main()` on error, not the bool `false`
(introduced in #13112). This is the correct value to return on error,
and also shuts up a clang warning.

Also add a final return for clarity.

Backport of core PR13349

Test Plan

With clang:

ninja bench-bitcoin

Ensure there is no -Wmain warning during the build.

Diff Detail

rABC Bitcoin ABC
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Fabien created this revision.Jul 31 2019, 14:21
Herald added a reviewer: Restricted Project. · View Herald TranscriptJul 31 2019, 14:21
deadalnix accepted this revision.Jul 31 2019, 18:21
This revision is now accepted and ready to land.Jul 31 2019, 18:21
This revision was automatically updated to reflect the committed changes.