Use QPDFUsage exception for cli, json, and QPDFJob errors

Jay Berkenbilt
1 parent 764feb0f
Showing 18 changed files with 99 additions and 81 deletions
ChangeLog
include/qpdf/JSONHandler.hh
include/qpdf/QPDFArgParser.hh
include/qpdf/QPDFExc.hh
include/qpdf/QPDFJob.hh
include/qpdf/QPDFSystemError.hh
include/qpdf/QPDFUsage.hh
libqpdf/JSONHandler.cc
libqpdf/QPDFArgParser.cc
libqpdf/QPDFJob.cc
libqpdf/QPDFJob_argv.cc
libqpdf/QPDFJob_config.cc
libqpdf/QPDFSystemError.cc
libqpdf/QPDFUsage.cc
libqpdf/build.mk
libtests/arg_parser.cc
libtests/json_handler.cc
qpdf/qpdf.cc
+2022-01-28  Jay Berkenbilt  <ejb@ql.org>
+
+	* Add QPDFUsage exception, which is thrown by JSONHandler,
+	QPDFArgParser, and QPDFJob to indicate command-line usage or job
+	configuration errors.
+
 2022-01-22  Jay Berkenbilt  <ejb@ql.org>
  
 	* Add QUtil::make_shared_cstr to return a std::shared_ptr<char>
@@ -28,7 +28,6 @@
 #include <string>
 #include <map>
 #include <functional>
-#include <stdexcept>
 #include <memory>
  
 // This class allows a sax-like walk through a JSON object with
@@ -39,15 +38,8 @@
 class JSONHandler
 {
   public:
-    // Error exception is thrown if there are any errors validating
-    // the JSON object.
-    class QPDF_DLL_CLASS Error: public std::runtime_error
-    {
-      public:
-        QPDF_DLL
-        Error(std::string const&);
-    };
-
+    // A QPDFUsage exception is thrown if there are any errors
+    // validating the JSON object.
     QPDF_DLL
     JSONHandler();
  
@@ -55,10 +47,10 @@ class JSONHandler
     ~JSONHandler() = default;
  
     // Based on the type of handler, expect the object to be of a
-    // certain type. JSONHandler::Error is thrown otherwise. Multiple
-    // handlers may be registered, which allows the object to be of
-    // various types. If an anyHandler is added, no other handler will
-    // be called. There is no "final" handler -- if the top-level is a
+    // certain type. QPDFUsage is thrown otherwise. Multiple handlers
+    // may be registered, which allows the object to be of various
+    // types. If an anyHandler is added, no other handler will be
+    // called. There is no "final" handler -- if the top-level is a
     // dictionary or array, just use its end handler.
  
     typedef std::function<void(
@@ -106,6 +98,8 @@ class JSONHandler
   private:
     JSONHandler(JSONHandler const&) = delete;
  
+    static void usage(std::string const& msg);
+
     struct Handlers
     {
         Handlers() :
@@ -29,7 +29,6 @@
 #include <map>
 #include <vector>
 #include <functional>
-#include <stdexcept>
 #include <sstream>
  
 // This is not a general-purpose argument parser. It is tightly
@@ -56,15 +55,6 @@
 class QPDFArgParser
 {
   public:
-    // Usage exception is thrown if there are any errors parsing
-    // arguments
-    class QPDF_DLL_CLASS Usage: public std::runtime_error
-    {
-      public:
-        QPDF_DLL
-        Usage(std::string const&);
-    };
-
     // progname_env is used to override argv[0] when figuring out the
     // name of the executable for setting up completion. This may be
     // needed if the program is invoked by a wrapper.
@@ -72,8 +62,8 @@ class QPDFArgParser
     QPDFArgParser(int argc, char* argv[], char const* progname_env);
  
     // Calls exit(0) if a help option is given or if in completion
-    // mode. If there are argument parsing errors,
-    // QPDFArgParser::Usage is thrown.
+    // mode. If there are argument parsing errors, QPDFUsage is
+    // thrown.
     QPDF_DLL
     void parseArgs();
  
@@ -39,7 +39,7 @@ class QPDF_DLL_CLASS QPDFExc: public std::runtime_error
 	    qpdf_offset_t offset,
 	    std::string const& message);
     QPDF_DLL
-    virtual ~QPDFExc() throw ()
+    virtual ~QPDFExc() noexcept
     {
     }
  
@@ -43,15 +43,8 @@ class QPDFWriter;
 class QPDFJob
 {
   public:
-    // ConfigError exception is thrown if there are any usage-like
-    // errors when calling Config methods.
-    class QPDF_DLL_CLASS ConfigError: public std::runtime_error
-    {
-      public:
-        QPDF_DLL
-        ConfigError(std::string const&);
-    };
-
+    // QPDFUsage is thrown if there are any usage-like errors when
+    // calling Config methods.
     QPDF_DLL
     QPDFJob();
  
@@ -318,7 +311,9 @@ class QPDFJob
     friend class Config;
  
     // Return a top-level configuration item. See CONFIGURATION above
-    // for details.
+    // for details. If an invalid configuration is created (such as
+    // supplying contradictory options, omitting an input file, etc.),
+    // QPDFUsage is thrown.
     QPDF_DLL
     std::shared_ptr<Config> config();
  
@@ -404,6 +399,7 @@ class QPDFJob
     };
  
     // Helper functions
+    static void usage(std::string const& msg);
     static JSON json_schema(std::set<std::string>* keys = 0);
     static void parse_object_id(
         std::string const& objspec, bool& trailer, int& obj, int& gen);
@@ -36,7 +36,7 @@ class QPDF_DLL_CLASS QPDFSystemError: public std::runtime_error
     QPDFSystemError(std::string const& description,
                     int system_errno);
     QPDF_DLL
-    virtual ~QPDFSystemError() throw ();
+    virtual ~QPDFSystemError() noexcept;
  
     // To get a complete error string, call what(), provided by
     // std::exception.  The accessors below return the original values
+// Copyright (c) 2005-2021 Jay Berkenbilt
+//
+// This file is part of qpdf.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//   http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+//
+// Versions of qpdf prior to version 7 were released under the terms
+// of version 2.0 of the Artistic License. At your option, you may
+// continue to consider qpdf to be licensed under those terms. Please
+// see the manual for additional information.
+
+#ifndef QPDFUSAGE_HH
+#define QPDFUSAGE_HH
+
+#include <qpdf/DLL.h>
+
+#include <string>
+#include <stdexcept>
+
+class QPDF_DLL_CLASS QPDFUsage: public std::runtime_error
+{
+  public:
+    QPDF_DLL
+    QPDFUsage(std::string const& msg);
+    QPDF_DLL
+    virtual ~QPDFUsage() noexcept = default;
+};
+
+#endif // QPDFUSAGE_HH
 #include <qpdf/JSONHandler.hh>
 #include <qpdf/QUtil.hh>
 #include <qpdf/QTC.hh>
-
-JSONHandler::Error::Error(std::string const& msg) :
-    std::runtime_error(msg)
-{
-}
+#include <qpdf/QPDFUsage.hh>
  
 JSONHandler::JSONHandler() :
     m(new Members())
@@ -17,6 +13,12 @@ JSONHandler::Members::Members()
 }
  
 void
+JSONHandler::usage(std::string const& msg)
+{
+    throw QPDFUsage(msg);
+}
+
+void
 JSONHandler::addAnyHandler(json_handler_t fn)
 {
     this->m->h.any_handler = fn;
@@ -128,9 +130,8 @@ JSONHandler::handle(std::string const&amp; path, JSON j)
                 else
                 {
                     QTC::TC("libtests", "JSONHandler unexpected key");
-                    throw Error(
-                        "JSON handler found unexpected key " + k +
-                        " in object at " + path);
+                    usage("JSON handler found unexpected key " + k +
+                          " in object at " + path);
                 }
             }
             else
@@ -163,7 +164,6 @@ JSONHandler::handle(std::string const&amp; path, JSON j)
         // different if this code were trying to be part of a
         // general-purpose JSON package.
         QTC::TC("libtests", "JSONHandler unhandled value");
-        throw Error("JSON handler: value at " + path +
-                    " is not of expected type");
+        usage("JSON handler: value at " + path + " is not of expected type");
     }
 }
@@ -2,15 +2,11 @@
 #include <qpdf/QUtil.hh>
 #include <qpdf/QIntC.hh>
 #include <qpdf/QTC.hh>
+#include <qpdf/QPDFUsage.hh>
 #include <iostream>
 #include <cstring>
 #include <cstdlib>
  
-QPDFArgParser::Usage::Usage(std::string const& msg) :
-    std::runtime_error(msg)
-{
-}
-
 QPDFArgParser::Members::Members(
     int argc, char* argv[], char const* progname_env) :
  
@@ -424,7 +420,7 @@ QPDFArgParser::usage(std::string const&amp; message)
         // This will cause bash to fall back to regular file completion.
         exit(0);
     }
-    throw Usage(message);
+    throw QPDFUsage(message);
 }
  
 void
@@ -25,21 +25,16 @@
 #include <qpdf/QPDFOutlineDocumentHelper.hh>
 #include <qpdf/QPDFAcroFormDocumentHelper.hh>
 #include <qpdf/QPDFExc.hh>
+#include <qpdf/QPDFUsage.hh>
 #include <qpdf/QPDFSystemError.hh>
 #include <qpdf/QPDFCryptoProvider.hh>
 #include <qpdf/QPDFEmbeddedFileDocumentHelper.hh>
 #include <qpdf/QPDFArgParser.hh>
-
 #include <qpdf/QPDFWriter.hh>
 #include <qpdf/QIntC.hh>
  
 #include <qpdf/auto_job_schema.hh> // JOB_SCHEMA_DATA
  
-QPDFJob::ConfigError::ConfigError(std::string const& msg) :
-    std::runtime_error(msg)
-{
-}
-
 namespace
 {
     class ImageOptimizer: public QPDFObjectHandle::StreamDataProvider
@@ -452,6 +447,12 @@ QPDFJob::QPDFJob() :
  
  
 void
+QPDFJob::usage(std::string const& msg)
+{
+    throw QPDFUsage(msg);
+}
+
+void
 QPDFJob::setMessagePrefix(std::string const& message_prefix)
 {
     this->m->message_prefix = message_prefix;
@@ -548,7 +549,7 @@ QPDFJob::parseRotationParameter(std::string const&amp; parameter)
     }
     else
     {
-        throw ConfigError("invalid parameter to rotate: " + parameter);
+        usage("invalid parameter to rotate: " + parameter);
     }
 }
  
@@ -561,7 +562,7 @@ QPDFJob::parseNumrange(char const* range, int max)
     }
     catch (std::runtime_error& e)
     {
-        throw ConfigError(e.what());
+        usage(e.what());
     }
     return std::vector<int>();
 }
@@ -645,10 +646,6 @@ QPDFJob::createsOutput() const
 void
 QPDFJob::checkConfiguration()
 {
-    auto usage = [](char const* msg){
-        throw QPDFJob::ConfigError(msg);
-    };
-
     // QXXXQ messages are CLI-centric
     if (m->replace_input)
     {
@@ -480,7 +480,7 @@ ArgParser::parseOptions()
     {
         this->ap.parseArgs();
     }
-    catch (QPDFArgParser::Usage& e)
+    catch (std::runtime_error& e)
     {
         usage(e.what());
     }
@@ -3,11 +3,6 @@
 #include <qpdf/QTC.hh>
 #include <cstring>
  
-static void usage(std::string const& msg)
-{
-    throw QPDFJob::ConfigError(msg);
-}
-
 QPDFJob::Config*
 QPDFJob::Config::inputFile(char const* filename)
 {
@@ -10,7 +10,7 @@ QPDFSystemError::QPDFSystemError(std::string const&amp; description,
 {
 }
  
-QPDFSystemError::~QPDFSystemError() throw ()
+QPDFSystemError::~QPDFSystemError() noexcept
 {
 }
  
+#include <qpdf/QPDFUsage.hh>
+
+QPDFUsage::QPDFUsage(std::string const& msg) :
+    std::runtime_error(msg)
+{
+}
@@ -89,6 +89,7 @@ SRCS_libqpdf = \
 	libqpdf/QPDFStreamFilter.cc \
 	libqpdf/QPDFSystemError.cc \
 	libqpdf/QPDFTokenizer.cc \
+	libqpdf/QPDFUsage.cc \
 	libqpdf/QPDFWriter.cc \
 	libqpdf/QPDFXRefEntry.cc \
 	libqpdf/QPDF_Array.cc \
 #include <qpdf/QPDFArgParser.hh>
 #include <qpdf/QUtil.hh>
+#include <qpdf/QPDFUsage.hh>
 #include <iostream>
 #include <cstring>
 #include <cassert>
@@ -225,7 +226,7 @@ int main(int argc, char* argv[])
     {
         ap.parseArgs();
     }
-    catch (QPDFArgParser::Usage& e)
+    catch (QPDFUsage& e)
     {
         std::cerr << "usage: " << e.what() << std::endl;
         exit(2);
 #include <qpdf/JSONHandler.hh>
 #include <qpdf/QUtil.hh>
+#include <qpdf/QPDFUsage.hh>
 #include <iostream>
 #include <cassert>
  
@@ -120,7 +121,7 @@ static void test_errors()
             fn();
             assert(false);
         }
-        catch (JSONHandler::Error& e)
+        catch (QPDFUsage& e)
         {
             std::cout << msg << ": " << e.what() << std::endl;
         }
 #include <qpdf/QPDFJob.hh>
 #include <qpdf/QTC.hh>
 #include <qpdf/QUtil.hh>
-#include <qpdf/QPDFArgParser.hh>
+#include <qpdf/QPDFUsage.hh>
  
 #include <cstdio>
 #include <cstdlib>
@@ -48,11 +48,7 @@ int realmain(int argc, char* argv[])
         j.initializeFromArgv(argc, argv);
         j.run();
     }
-    catch (QPDFArgParser::Usage& e)
-    {
-        usageExit(e.what());
-    }
-    catch (QPDFJob::ConfigError& e)
+    catch (QPDFUsage& e)
     {
         usageExit(e.what());
     }