Reconfigured options for sym_upload to not treat h and ? flags as invalid options.

Reconfigured options for sym_upload to not treat h and ? flags as invalid options.

I'm submitting this on behalf of Andrew Liu.

R=mmandlis@chromium.org

Committed: https://code.google.com/p/google-breakpad/source/detail?r=1462

[ivanpe, 2015-06-20 00:27:51]

ivanpe@chromium.org changed reviewers:
+ jessicag@chromium.org, mmandlis@chromium.org

[mmandlis, 2015-06-22 16:51:59]

lgtm

https://codereview.chromium.org/1196733004/diff/1/src/tools/linux/symupload/s...
File src/tools/linux/symupload/sym_upload.cc (right):

https://codereview.chromium.org/1196733004/diff/1/src/tools/linux/symupload/s...
src/tools/linux/symupload/sym_upload.cc:199: break;
nit - do you really need the "break;" after "exit(0);"?

[ivanpe, 2015-06-22 21:18:10]

https://codereview.chromium.org/1196733004/diff/1/src/tools/linux/symupload/s...
File src/tools/linux/symupload/sym_upload.cc (right):

https://codereview.chromium.org/1196733004/diff/1/src/tools/linux/symupload/s...
src/tools/linux/symupload/sym_upload.cc:199: break;
On 2015/06/22 16:51:59, mmandlis wrote:
> nit - do you really need the "break;" after "exit(0);"?

I think this is a good safely practice so I'm fine with it.

[ivanpe, 2015-06-22 21:21:44]

Committed patchset #1 (id:1) manually as r1462 (presubmit successful).