diff options
author | Jay Berkenbilt <ejb@ql.org> | 2022-07-31 16:34:05 +0200 |
---|---|---|
committer | Jay Berkenbilt <ejb@ql.org> | 2022-07-31 22:23:17 +0200 |
commit | 5f4224f31a500452a4f97f36ed57351b41ca0114 (patch) | |
tree | be355db5a4fcc8334410baf76c7eb1330578faa8 /libqpdf/QPDFJob_config.cc | |
parent | 80acfc3826704064db8cc2f6af0c338b3aa557e7 (diff) | |
download | qpdf-5f4224f31a500452a4f97f36ed57351b41ca0114.tar.zst |
Simplify --json-output
Now --json-output just changes defaults. Allow output file with --json.
Diffstat (limited to 'libqpdf/QPDFJob_config.cc')
-rw-r--r-- | libqpdf/QPDFJob_config.cc | 14 |
1 files changed, 2 insertions, 12 deletions
diff --git a/libqpdf/QPDFJob_config.cc b/libqpdf/QPDFJob_config.cc index 9a2b3a84..8a9c1470 100644 --- a/libqpdf/QPDFJob_config.cc +++ b/libqpdf/QPDFJob_config.cc @@ -244,7 +244,6 @@ QPDFJob::Config::json(std::string const& parameter) if ((o.m->json_version < 1) || (o.m->json_version > JSON::LATEST)) { usage(std::string("unsupported json version ") + parameter); } - o.m->require_outfile = false; return this; } @@ -297,14 +296,7 @@ QPDFJob::Config* QPDFJob::Config::jsonOutput(std::string const& parameter) { o.m->json_output = true; - if (parameter.empty() || (parameter == "latest")) { - o.m->json_version = JSON::LATEST; - } else { - o.m->json_version = QUtil::string_to_int(parameter.c_str()); - } - if ((o.m->json_version < 2) || (o.m->json_version > JSON::LATEST)) { - usage(std::string("unsupported json output version ") + parameter); - } + json(parameter); if (!o.m->json_stream_data_set) { // No need to set json_stream_data_set -- that indicates // explicit use of --json-stream-data. @@ -313,9 +305,7 @@ QPDFJob::Config::jsonOutput(std::string const& parameter) if (!o.m->decode_level_set) { o.m->decode_level = qpdf_dl_none; } - if (o.m->json_keys.empty()) { - o.m->json_keys.insert("qpdf"); - } + o.m->json_keys.insert("qpdf"); return this; } |