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

src/tools/linux/symupload/sym_upload.cc

 // Copyright (c) 2006, Google Inc.
 // All rights reserved.
 //
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 // notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
@@ skipping 174 matching lines @@
 }
 
 //=============================================================================
 static void
 SetupOptions(int argc, const char *argv[], Options *options) {
   extern int optind;
   int ch;
 
   while ((ch = getopt(argc, (char * const *)argv, "u:v:x:h?")) != -1) {
     switch (ch) {
+      case 'h':
+      case '?':
+        Usage(argc, argv);
+        exit(0);
+        break;

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

nit - do you really need the "break;" after "exit(0);"?

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

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

       case 'u':
         options->proxy_user_pwd = optarg;
         break;
       case 'v':
         options->version = optarg;
         break;
       case 'x':
         options->proxy = optarg;
         break;
 
@@ skipping 15 matching lines @@
   options->uploadURLStr = argv[optind + 1];
 }
 
 //=============================================================================
 int main(int argc, const char* argv[]) {
   Options options;
   SetupOptions(argc, argv, &options);
   Start(&options);
   return options.success ? 0 : 1;
 }