Add to_upper() method to util.{cc,h}
authorCarl Hetherington <cth@carlh.net>
Fri, 2 Apr 2021 21:44:58 +0000 (23:44 +0200)
committerCarl Hetherington <cth@carlh.net>
Sun, 4 Apr 2021 18:48:35 +0000 (20:48 +0200)
src/lib/film.cc
src/lib/util.cc
src/lib/util.h

index b34c852952bf9c12e22e3b7059d9435192058072..1f188021b8babbb082bf9a82f2611465638726e8 100644 (file)
@@ -919,9 +919,8 @@ Film::isdcf_name (bool if_created_now) const
 
        auto audio_langs = audio_languages();
        auto audio_language = (audio_langs.empty() || !audio_langs.front().language()) ? "XX" : audio_langs.front().language()->subtag();
-       transform (audio_language.begin(), audio_language.end(), audio_language.begin(), ::toupper);
 
-       d += "_" + audio_language;
+       d += "_" + to_upper (audio_language);
 
        /* I'm not clear on the precise details of the convention for CCAP labelling;
           for now I'm just appending -CCAP if we have any closed captions.
@@ -945,7 +944,7 @@ Film::isdcf_name (bool if_created_now) const
                if (burnt_in) {
                        transform (lang.begin(), lang.end(), lang.begin(), ::tolower);
                } else {
-                       transform (lang.begin(), lang.end(), lang.begin(), ::toupper);
+                       lang = to_upper (lang);
                }
 
                d += "-" + lang;
@@ -959,8 +958,7 @@ Film::isdcf_name (bool if_created_now) const
 
        if (_release_territory) {
                auto territory = _release_territory->subtag();
-               transform (territory.begin(), territory.end(), territory.begin(), ::toupper);
-               d += "_" + territory;
+               d += "_" + to_upper (territory);
                if (_ratings.empty ()) {
                        d += "-NR";
                } else {
index 24ea42b8f2e0e945a31488a414e42ca1f4017f57..d3511e8c7e34e0b1c5f5744147b18d83fb944837 100644 (file)
@@ -1167,3 +1167,12 @@ default_font_file ()
 
        return liberation_normal;
 }
+
+
+string
+to_upper (string s)
+{
+       transform (s.begin(), s.end(), s.begin(), ::toupper);
+       return s;
+}
+
index ceb30701c3bbb956819bd60f30f3116e67021e87..7f8106f3c543775bfb10e62a04c00af7422fa9bd 100644 (file)
@@ -122,6 +122,7 @@ extern void copy_in_bits (boost::filesystem::path from, boost::filesystem::path
 extern dcp::Size scale_for_display (dcp::Size s, dcp::Size display_container, dcp::Size film_container);
 extern dcp::DecryptedKDM decrypt_kdm_with_helpful_error (dcp::EncryptedKDM kdm);
 extern boost::filesystem::path default_font_file ();
+extern std::string to_upper (std::string s);
 
 template <class T>
 std::list<T>