Commit f4598797d578ac42362a24a562bdbfb0ea73e3a9
1 parent
41c839e2
Refactor `QPDFJob::setWriterOptions`: replace `std::shared_ptr` with `std::make_…
…shared` for progress reporters to improve clarity and efficiency.
Showing
1 changed file
with
3 additions
and
5 deletions
libqpdf/QPDFJob.cc
| @@ -2960,15 +2960,13 @@ QPDFJob::setWriterOptions(QPDFWriter& w) | @@ -2960,15 +2960,13 @@ QPDFJob::setWriterOptions(QPDFWriter& w) | ||
| 2960 | if (m->progress) { | 2960 | if (m->progress) { |
| 2961 | if (m->progress_handler) { | 2961 | if (m->progress_handler) { |
| 2962 | w.registerProgressReporter( | 2962 | w.registerProgressReporter( |
| 2963 | - std::shared_ptr<QPDFWriter::ProgressReporter>( | ||
| 2964 | - new QPDFWriter::FunctionProgressReporter(m->progress_handler))); | 2963 | + std::make_shared<QPDFWriter::FunctionProgressReporter>(m->progress_handler)); |
| 2965 | } else { | 2964 | } else { |
| 2966 | char const* outfilename = | 2965 | char const* outfilename = |
| 2967 | !m->outfilename.empty() ? m->outfilename.data() : "standard output"; | 2966 | !m->outfilename.empty() ? m->outfilename.data() : "standard output"; |
| 2968 | w.registerProgressReporter( | 2967 | w.registerProgressReporter( |
| 2969 | - std::shared_ptr<QPDFWriter::ProgressReporter>( | ||
| 2970 | - // line-break | ||
| 2971 | - new ProgressReporter(*m->log->getInfo(), m->message_prefix, outfilename))); | 2968 | + std::make_shared<ProgressReporter>( |
| 2969 | + *m->log->getInfo(), m->message_prefix, outfilename)); | ||
| 2972 | } | 2970 | } |
| 2973 | } | 2971 | } |
| 2974 | } | 2972 | } |