Check if user provided more than one option for pdfium_test.

samples/pdfium_test.cc

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <limits.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 
 #include <list>
@@ skipping 171 matching lines @@
   }
   printf("Unsupported feature: %s.\n", feature.c_str());
 }
 
 bool ParseCommandLine(int argc, const char* argv[], OutputFormat* output_format,
                       std::list<const char*>* files) {
   *output_format = OUTPUT_NONE;
   files->clear();
 
   int cur_arg = 1;
-  if (cur_arg < argc) {
+  for (; cur_arg < argc; ++cur_arg) {
     if (strcmp(argv[cur_arg], "--ppm") == 0)
       *output_format = OUTPUT_PPM;
 #ifdef _WIN32
-    if (strcmp(argv[cur_arg], "--emf") == 0)
+    else if (strcmp(argv[cur_arg], "--emf") == 0)
       *output_format = OUTPUT_EMF;
-    if (strcmp(argv[cur_arg], "--bmp") == 0)
+    else if (strcmp(argv[cur_arg], "--bmp") == 0)
       *output_format = OUTPUT_BMP;
 #endif
-    if (*output_format != OUTPUT_NONE)
-      cur_arg++;
+    else
+      break;
   }
 
-  if (cur_arg >= argc)
+  if (cur_arg > 2)  // Multiple options.
+    return false;
+
+  if (cur_arg >= argc)  // No input files.
     return false;
 
   for (int i = cur_arg; i < argc; i++)
     files->push_back(argv[i]);
 
   return true;
 }
 
 class TestLoader {
  public:
@@ skipping 102 matching lines @@
     const char* buffer =
         reinterpret_cast<const char*>(FPDFBitmap_GetBuffer(bitmap));
 
     switch (format) {
 #ifdef _WIN32
       case OUTPUT_BMP:
         WriteBmp(name, i, buffer, stride, width, height);
         break;
 
       case OUTPUT_EMF:
         WriteEmf(page, name, i);

[Bo Xu, 2014/08/20 05:25:59]

When do WriteEmf, bitmap is created but never used and we use render to DC, so
it would take twice time for rendering. For batch testing, this maybe an issue.

         break;
 #endif
       case OUTPUT_PPM:
         WritePpm(name, i, buffer, stride, width, height);
         break;
       default:
         break;
     }
 
     FPDFBitmap_Destroy(bitmap);
@@ skipping 10 matching lines @@
   FPDFAvail_Destroy(pdf_avail);
 
   printf("Loaded, parsed and rendered %d pages.\n", page_count);
 }
 
 int main(int argc, const char* argv[]) {
   v8::V8::InitializeICU();
   OutputFormat format = OUTPUT_NONE;
   std::list<const char*> files;
   if (!ParseCommandLine(argc, argv, &format, &files)) {
-    printf("Usage: pdfium_test [OPTIONS] [FILE]\n");
+    printf("Usage: pdfium_test [OPTION] [FILE]...\n");
     printf("--ppm    write page images <pdf-name>.<page-number>.ppm\n");
 #ifdef _WIN32
     printf("--bmp    write page images <pdf-name>.<page-number>.bmp\n");
     printf("--emf    write page meta files <pdf-name>.<page-number>.emf\n");
 #endif
     return 1;
   }
 
   FPDF_InitLibrary(NULL);
 
@@ skipping 23 matching lines @@
     } else {
       RenderPdf(filename, pBuf, len, format);
     }
     free(pBuf);
   }
 
   FPDF_DestroyLibrary();
 
   return 0;
 }