Make parse_common_args() not exit.
Instead, return value of false is interpreted as a need to return
immediately.
Stuff in src/contrib/ is broken by this, and existing user tools may
behave a bit weirdly if they don't check the return value. If this is a
big deal, can add some extra backwards-compatibility glue, but not done
here.
Uncrustified the touched source files to fix indentation after batch
modification. This resulted in some additional clean-up, but didn't try
to separate that out from here.
Change-Id: I76c268297481bd125d14f4b4f7fa888f4da99f46