More c++ tidying.
[dcpomatic.git] / src / lib / util.cc
index 37b03c83605fa73c1131b6f15ee3216cd3bf2ecd..0aa7e7a28c3919b756eca61a839cbaea3a66d8f2 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2012-2020 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2012-2021 Carl Hetherington <cth@carlh.net>
 
     This file is part of DCP-o-matic.
 
@@ -103,7 +103,8 @@ using std::cout;
 using std::bad_alloc;
 using std::set_terminate;
 using std::make_pair;
-using boost::shared_ptr;
+using std::shared_ptr;
+using std::make_shared;
 using boost::thread;
 using boost::optional;
 using boost::lexical_cast;
@@ -241,7 +242,7 @@ DCPOMATIC_DISABLE_WARNINGS
 LONG WINAPI
 exception_handler(struct _EXCEPTION_POINTERS * info)
 {
-       FILE* f = fopen_boost (backtrace_file, "w");
+       auto f = fopen_boost (backtrace_file, "w");
        fprintf (f, "C-style exception %d\n", info->ExceptionRecord->ExceptionCode);
        fclose(f);
 
@@ -370,7 +371,7 @@ DCPOMATIC_ENABLE_WARNINGS
        /* Add our library directory to the libltdl search path so that
           xmlsec can find xmlsec1-openssl.
        */
-       boost::filesystem::path lib = directory_containing_executable().parent_path();
+       auto lib = directory_containing_executable().parent_path();
        lib /= "Frameworks";
        setenv ("LTDL_LIBRARY_PATH", lib.c_str (), 1);
 #endif
@@ -398,7 +399,7 @@ DCPOMATIC_ENABLE_WARNINGS
                "Hello dolly", dcp::NONE, dcp::Colour(), dcp::Time(), dcp::Time()
                );
        subs.push_back (StringText(ss, 0));
-       render_text (subs, list<shared_ptr<Font> >(), dcp::Size(640, 480), DCPTime(), 24);
+       render_text (subs, list<shared_ptr<Font>>(), dcp::Size(640, 480), DCPTime(), 24);
 #endif
 
        Ratio::setup_ratios ();
@@ -479,7 +480,7 @@ digest_head_tail (vector<boost::filesystem::path> files, boost::uintmax_t size)
        char* p = buffer.get ();
        int i = 0;
        while (i < int64_t (files.size()) && to_do > 0) {
-               FILE* f = fopen_boost (files[i], "rb");
+               auto f = fopen_boost (files[i], "rb");
                if (!f) {
                        throw OpenFileError (files[i].string(), errno, OpenFileError::READ);
                }
@@ -499,7 +500,7 @@ digest_head_tail (vector<boost::filesystem::path> files, boost::uintmax_t size)
        p = buffer.get ();
        i = files.size() - 1;
        while (i >= 0 && to_do > 0) {
-               FILE* f = fopen_boost (files[i], "rb");
+               auto f = fopen_boost (files[i], "rb");
                if (!f) {
                        throw OpenFileError (files[i].string(), errno, OpenFileError::READ);
                }
@@ -592,7 +593,7 @@ short_audio_channel_name (int c)
                _("DBP"),
                _("DBS"),
                "",
-               _("Sign")
+               ""
        };
 
        return channels[c];
@@ -606,7 +607,7 @@ valid_image_file (boost::filesystem::path f)
                return false;
        }
 
-       string ext = f.extension().string();
+       auto ext = f.extension().string();
        transform (ext.begin(), ext.end(), ext.begin(), ::tolower);
        return (
                ext == ".tif" || ext == ".tiff" || ext == ".jpg" || ext == ".jpeg" ||
@@ -623,7 +624,7 @@ valid_sound_file (boost::filesystem::path f)
                return false;
        }
 
-       string ext = f.extension().string();
+       auto ext = f.extension().string();
        transform (ext.begin(), ext.end(), ext.begin(), ::tolower);
        return (ext == ".wav" || ext == ".mp3" || ext == ".aif" || ext == ".aiff");
 }
@@ -631,7 +632,7 @@ valid_sound_file (boost::filesystem::path f)
 bool
 valid_j2k_file (boost::filesystem::path f)
 {
-       string ext = f.extension().string();
+       auto ext = f.extension().string();
        transform (ext.begin(), ext.end(), ext.begin(), ::tolower);
        return (ext == ".j2k" || ext == ".j2c" || ext == ".jp2");
 }
@@ -656,7 +657,7 @@ fit_ratio_within (float ratio, dcp::Size full_frame)
 void *
 wrapped_av_malloc (size_t s)
 {
-       void* p = av_malloc (s);
+       auto p = av_malloc (s);
        if (!p) {
                throw bad_alloc ();
        }
@@ -810,7 +811,7 @@ audio_channel_types (list<int> mapped, int channels)
        int non_lfe = 0;
        int lfe = 0;
 
-       BOOST_FOREACH (int i, mapped) {
+       for (auto i: mapped) {
                if (i >= channels) {
                        /* This channel is mapped but is not included in the DCP */
                        continue;
@@ -845,7 +846,7 @@ audio_channel_types (list<int> mapped, int channels)
 shared_ptr<AudioBuffers>
 remap (shared_ptr<const AudioBuffers> input, int output_channels, AudioMapping map)
 {
-       shared_ptr<AudioBuffers> mapped (new AudioBuffers (output_channels, input->frames()));
+       auto mapped = make_shared<AudioBuffers>(output_channels, input->frames());
        mapped->make_silent ();
 
        int to_do = min (map.input_channels(), input->channels());
@@ -953,8 +954,8 @@ void
 emit_subtitle_image (ContentTimePeriod period, dcp::SubtitleImage sub, dcp::Size size, shared_ptr<TextDecoder> decoder)
 {
        /* XXX: this is rather inefficient; decoding the image just to get its size */
-       FFmpegImageProxy proxy (sub.png_image());
-       shared_ptr<Image> image = proxy.image().image;
+       FFmpegImageProxy proxy (sub.png_image(), VIDEO_RANGE_FULL);
+       auto image = proxy.image().image;
        /* set up rect with height and width */
        dcpomatic::Rect<double> rect(0, 0, image->size().width / double(size.width), image->size().height / double(size.height));
 
@@ -996,7 +997,7 @@ show_jobs_on_console (bool progress)
 
                dcpomatic_sleep_seconds (5);
 
-               list<shared_ptr<Job> > jobs = JobManager::instance()->get();
+               auto jobs = JobManager::instance()->get();
 
                if (!first && progress) {
                        for (size_t i = 0; i < jobs.size(); ++i) {
@@ -1007,7 +1008,7 @@ show_jobs_on_console (bool progress)
 
                first = false;
 
-               BOOST_FOREACH (shared_ptr<Job> i, jobs) {
+               for (auto i: jobs) {
                        if (progress) {
                                cout << i->name();
                                if (!i->sub_name().empty()) {
@@ -1046,11 +1047,11 @@ show_jobs_on_console (bool progress)
 void
 copy_in_bits (boost::filesystem::path from, boost::filesystem::path to, boost::function<void (float)> progress)
 {
-       FILE* f = fopen_boost (from, "rb");
+       auto f = fopen_boost (from, "rb");
        if (!f) {
                throw OpenFileError (from, errno, OpenFileError::READ);
        }
-       FILE* t = fopen_boost (to, "wb");
+       auto t = fopen_boost (to, "wb");
        if (!t) {
                fclose (f);
                throw OpenFileError (to, errno, OpenFileError::WRITE);
@@ -1059,7 +1060,7 @@ copy_in_bits (boost::filesystem::path from, boost::filesystem::path to, boost::f
        /* on the order of a second's worth of copying */
        boost::uintmax_t const chunk = 20 * 1024 * 1024;
 
-       uint8_t* buffer = static_cast<uint8_t*> (malloc(chunk));
+       auto buffer = static_cast<uint8_t*> (malloc(chunk));
        if (!buffer) {
                throw std::bad_alloc ();
        }
@@ -1094,111 +1095,6 @@ copy_in_bits (boost::filesystem::path from, boost::filesystem::path to, boost::f
        free (buffer);
 }
 
-#ifdef DCPOMATIC_VARIANT_SWAROOP
-
-/* Make up a key from the machine UUID */
-dcp::Data
-key_from_uuid ()
-{
-       dcp::Data key (dcpomatic::crypto_key_length());
-       memset (key.data().get(), 0, key.size());
-       string const magic = command_and_read ("dcpomatic2_uuid");
-       strncpy ((char *) key.data().get(), magic.c_str(), dcpomatic::crypto_key_length());
-       return key;
-}
-
-/* swaroop chain file format:
- *
- *  0 [int16_t] IV length
- *  2 [int16_t] cert #1 length, or 0 for none
- *  4 [int16_t] cert #2 length, or 0 for none
- *  6 [int16_t] cert #3 length, or 0 for none
- *  8 [int16_t] cert #4 length, or 0 for none
- * 10 [int16_t] cert #5 length, or 0 for none
- * 12 [int16_t] cert #6 length, or 0 for none
- * 14 [int16_t] cert #7 length, or 0 for none
- * 16 [int16_t] cert #8 length, or 0 for none
- * 16 [int16_t] private key length
- * 20 IV
- *    cert #1
- *    cert #2
- *    cert #3
- *    cert #4
- *    cert #5
- *    cert #6
- *    cert #7
- *    cert #8
- *    private key
- */
-
-struct __attribute__ ((packed)) Header_ {
-       int16_t iv_length;
-       int16_t cert_length[8];
-       int16_t private_key_length;
-};
-
-typedef struct Header_ Header;
-
-shared_ptr<dcp::CertificateChain>
-read_swaroop_chain (boost::filesystem::path path)
-{
-       dcp::Data data (path);
-       Header* header = (Header *) data.data().get();
-       uint8_t* p = data.data().get() + sizeof(Header);
-
-       dcp::Data iv (p, header->iv_length);
-       p += iv.size();
-
-       shared_ptr<dcp::CertificateChain> cc (new dcp::CertificateChain());
-       for (int i = 0; i < 8; ++i) {
-               if (header->cert_length[i] == 0) {
-                       break;
-               }
-               dcp::Data c(p, header->cert_length[i]);
-               p += c.size();
-               cc->add (dcp::Certificate(dcpomatic::decrypt(c, key_from_uuid(), iv)));
-       }
-
-       dcp::Data k (p, header->private_key_length);
-       cc->set_key (dcpomatic::decrypt(k, key_from_uuid(), iv));
-       return cc;
-}
-
-void
-write_swaroop_chain (shared_ptr<const dcp::CertificateChain> chain, boost::filesystem::path output)
-{
-       scoped_array<uint8_t> buffer (new uint8_t[65536]);
-       Header* header = (Header *) buffer.get();
-       memset (header, 0, sizeof(Header));
-       uint8_t* p = buffer.get() + sizeof(Header);
-
-       dcp::Data iv = dcpomatic::random_iv ();
-       header->iv_length = iv.size ();
-       memcpy (p, iv.data().get(), iv.size());
-       p += iv.size();
-
-       int N = 0;
-       BOOST_FOREACH (dcp::Certificate i, chain->root_to_leaf()) {
-               dcp::Data e = dcpomatic::encrypt (i.certificate(true), key_from_uuid(), iv);
-               memcpy (p, e.data().get(), e.size());
-               p += e.size();
-               DCPOMATIC_ASSERT (N < 8);
-               header->cert_length[N] = e.size ();
-               ++N;
-       }
-
-       dcp::Data k = dcpomatic::encrypt (chain->key().get(), key_from_uuid(), iv);
-       memcpy (p, k.data().get(), k.size());
-       p += k.size();
-       header->private_key_length = k.size ();
-
-       FILE* f = fopen_boost (output, "wb");
-       checked_fwrite (buffer.get(), p - buffer.get(), f, output);
-       fclose (f);
-}
-
-#endif
-
 double
 db_to_linear (double db)
 {
@@ -1237,17 +1133,17 @@ decrypt_kdm_with_helpful_error (dcp::EncryptedKDM kdm)
                return dcp::DecryptedKDM (kdm, Config::instance()->decryption_chain()->key().get());
        } catch (dcp::KDMDecryptionError& e) {
                /* Try to flesh out the error a bit */
-               string const kdm_subject_name = kdm.recipient_x509_subject_name();
+               auto const kdm_subject_name = kdm.recipient_x509_subject_name();
                bool on_chain = false;
-               shared_ptr<const dcp::CertificateChain> dc = Config::instance()->decryption_chain();
-               BOOST_FOREACH (dcp::Certificate i, dc->root_to_leaf()) {
+               auto dc = Config::instance()->decryption_chain();
+               for (auto i: dc->root_to_leaf()) {
                        if (i.subject() == kdm_subject_name) {
                                on_chain = true;
                        }
                }
                if (!on_chain) {
                        throw KDMError (_("This KDM was not made for DCP-o-matic's decryption certificate."), e.what());
-               } else if (on_chain && kdm_subject_name != dc->leaf().subject()) {
+               } else if (kdm_subject_name != dc->leaf().subject()) {
                        throw KDMError (_("This KDM was made for DCP-o-matic but not for its leaf certificate."), e.what());
                } else {
                        throw;