Move `JobSetter` to `QPDF::Doc` and update references

Relocate `JobSetter` to `QPDF::Doc` for improved encapsulation of job-related logic. Adjust all references to use the new placement.

Move `JobSetter` to `QPDF::Doc` and update references
Relocate `JobSetter` to `QPDF::Doc` for improved encapsulation of job-related logic. Adjust all references to use the new placement.
m-holger
1 parent c030d31c
Showing 3 changed files with 14 additions and 16 deletions
include/qpdf/QPDF.hh
libqpdf/QPDFJob.cc
libqpdf/qpdf/QPDF_private.hh
@@ -791,7 +791,6 @@ class QPDF
     class Doc;
     class StreamCopier;
-    class JobSetter;
     inline bool reconstructed_xref() const;
     inline Doc& doc();
@@ -30,6 +30,18 @@
 using namespace qpdf;
+// JobSetter class is restricted to QPDFJob.
+class QPDF::Doc::JobSetter
+{
+  public:
+    // Enable enhanced warnings for pdf file checking.
+    static void
+    setCheckMode(QPDF& qpdf, bool val)
+    {
+        qpdf.m->check_mode = val;
+    }
+};
+
 namespace
 {
     class ImageOptimizer final: public QPDFObjectHandle::StreamDataProvider
@@ -734,7 +746,7 @@ QPDFJob::doCheck(QPDF&amp; pdf)
     bool okay = true;
     auto& cout = *m->log->getInfo();
     cout << "checking " << m->infile_name() << "\n";
-    QPDF::JobSetter::setCheckMode(pdf, true);
+    QPDF::Doc::JobSetter::setCheckMode(pdf, true);
     try {
         int extension_level = pdf.getExtensionLevel();
         cout << "PDF Version: " << pdf.getPDFVersion();
@@ -344,6 +344,7 @@ class QPDF::PatternFinder final: public InputSource::Finder
 class QPDF::Doc
 {
   public:
+    class JobSetter;
     class ParseGuard;
     class Resolver;
     class Streams;
@@ -520,20 +521,6 @@ class QPDF::Doc::Resolver
     }
 };
-// JobSetter class is restricted to QPDFJob.
-class QPDF::JobSetter
-{
-    friend class QPDFJob;
-
-  private:
-    // Enable enhanced warnings for pdf file checking.
-    static void
-    setCheckMode(QPDF& qpdf, bool val)
-    {
-        qpdf.m->check_mode = val;
-    }
-};
-
 inline bool
 QPDF::reconstructed_xref() const
 {