tree dedaf070afd37a1e6c9192e467bfe4f128b665b9
parent e19b2848ce8b0bb793232b36ac8953f82e892c5d
author Victor Hsieh <victorhsieh@google.com> 1639678922 -0800
committer Victor Hsieh <victorhsieh@google.com> 1639679720 -0800

odrefresh should exit non-zero for invalid argument

This fixes two cases.

1. "Expected 1 argument, but have %d."
2. unknown action

Because of 1, the loop is actually not necessary (as pointed out by the
compiler), so removed.

With 2 now exits, the default return is never eached, thus deleted.

Bug: 205750213
Test: atest art_standalone_odrefresh_tests
Test: atest ComposHostTestCases
Change-Id: Ifc198e888045e633a63a918509faf9a0fe1e24fb