diff options
author | Adam Szmigin <smidge@xsco.net> | 2020-07-08 23:33:30 +0100 |
---|---|---|
committer | Adam Szmigin <smidge@xsco.net> | 2020-07-08 23:33:30 +0100 |
commit | 61e8db96d77022c3861cf6a12ad0a7c1aad7c836 (patch) | |
tree | 6d6e613d104411045b7277fd7b630482f550dcf4 | |
parent | c650b488f5939c924a8559e2f89c1af222a0263e (diff) |
Remove internal use of Boost Optional
-rw-r--r-- | meson.build | 1 | ||||
-rw-r--r-- | src/djinterop/crate.cpp | 6 | ||||
-rw-r--r-- | src/djinterop/database.cpp | 6 | ||||
-rw-r--r-- | src/djinterop/enginelibrary/el_crate_impl.cpp | 12 | ||||
-rw-r--r-- | src/djinterop/enginelibrary/el_crate_impl.hpp | 6 | ||||
-rw-r--r-- | src/djinterop/enginelibrary/el_database_impl.cpp | 16 | ||||
-rw-r--r-- | src/djinterop/enginelibrary/el_database_impl.hpp | 6 | ||||
-rw-r--r-- | src/djinterop/enginelibrary/el_track_impl.cpp | 152 | ||||
-rw-r--r-- | src/djinterop/enginelibrary/el_track_impl.hpp | 109 | ||||
-rw-r--r-- | src/djinterop/enginelibrary/performance_data_format.cpp | 21 | ||||
-rw-r--r-- | src/djinterop/enginelibrary/performance_data_format.hpp | 15 | ||||
-rw-r--r-- | src/djinterop/impl/crate_impl.hpp | 7 | ||||
-rw-r--r-- | src/djinterop/impl/database_impl.hpp | 9 | ||||
-rw-r--r-- | src/djinterop/impl/track_impl.hpp | 97 | ||||
-rw-r--r-- | src/djinterop/impl/util.cpp | 4 | ||||
-rw-r--r-- | src/djinterop/impl/util.hpp | 15 | ||||
-rw-r--r-- | src/djinterop/track.cpp | 132 |
17 files changed, 280 insertions, 334 deletions
diff --git a/meson.build b/meson.build index c38bab5..cd5a227 100644 --- a/meson.build +++ b/meson.build @@ -6,7 +6,6 @@ project( default_options: ['cpp_std=c++17', 'default_library=both']) cpp_compiler = meson.get_compiler('cpp') -add_global_arguments('-D_MODERN_SQLITE_BOOST_OPTIONAL_SUPPORT', language: 'cpp') if cpp_compiler.get_id() == 'msvc' # Ask MSVC to populate the __cplusplus macro properly. add_global_arguments('/Zc:__cplusplus', language: 'cpp') diff --git a/src/djinterop/crate.cpp b/src/djinterop/crate.cpp index 55f5541..5da4ad3 100644 --- a/src/djinterop/crate.cpp +++ b/src/djinterop/crate.cpp @@ -82,7 +82,7 @@ std::string crate::name() const std::optional<crate> crate::parent() const { - return from_boost_optional(pimpl_->parent()); + return pimpl_->parent(); } void crate::remove_track(track tr) const @@ -97,13 +97,13 @@ void crate::set_name(std::string name) const void crate::set_parent(std::optional<crate> parent) const { - pimpl_->set_parent(to_boost_optional(parent)); + pimpl_->set_parent(parent); } std::optional<crate> crate::sub_crate_by_name( const std::string& name) const { - return from_boost_optional(pimpl_->sub_crate_by_name(name)); + return pimpl_->sub_crate_by_name(name); } std::vector<track> crate::tracks() const diff --git a/src/djinterop/database.cpp b/src/djinterop/database.cpp index c1f06fb..1a224f8 100644 --- a/src/djinterop/database.cpp +++ b/src/djinterop/database.cpp @@ -39,7 +39,7 @@ transaction_guard database::begin_transaction() const std::optional<crate> database::crate_by_id(int64_t id) const { - return from_boost_optional(pimpl_->crate_by_id(id)); + return pimpl_->crate_by_id(id); } std::vector<crate> database::crates() const @@ -95,12 +95,12 @@ std::vector<crate> database::root_crates() const std::optional<crate> database::root_crate_by_name( const std::string& name) const { - return from_boost_optional(pimpl_->root_crate_by_name(name)); + return pimpl_->root_crate_by_name(name); } std::optional<track> database::track_by_id(int64_t id) const { - return from_boost_optional(pimpl_->track_by_id(id)); + return pimpl_->track_by_id(id); } std::vector<track> database::tracks() const diff --git a/src/djinterop/enginelibrary/el_crate_impl.cpp b/src/djinterop/enginelibrary/el_crate_impl.cpp index 3403fee..f3a2360 100644 --- a/src/djinterop/enginelibrary/el_crate_impl.cpp +++ b/src/djinterop/enginelibrary/el_crate_impl.cpp @@ -211,7 +211,7 @@ bool el_crate_impl::is_valid() std::string el_crate_impl::name() { - boost::optional<std::string> name; + std::optional<std::string> name; storage_->db << "SELECT title FROM Crate WHERE id = ?" << id() >> [&](std::string title) { if (!name) @@ -231,9 +231,9 @@ std::string el_crate_impl::name() return *name; } -boost::optional<crate> el_crate_impl::parent() +std::optional<crate> el_crate_impl::parent() { - boost::optional<crate> parent; + std::optional<crate> parent; storage_->db << "SELECT crateParentId FROM CrateParentList WHERE crateOriginId " "= ? AND crateParentId <> crateOriginId" @@ -298,7 +298,7 @@ void el_crate_impl::set_name(std::string name) trans.commit(); } -void el_crate_impl::set_parent(boost::optional<crate> parent) +void el_crate_impl::set_parent(std::optional<crate> parent) { el_transaction_guard_impl trans{storage_}; @@ -323,9 +323,9 @@ void el_crate_impl::set_parent(boost::optional<crate> parent) trans.commit(); } -boost::optional<crate> el_crate_impl::sub_crate_by_name(const std::string& name) +std::optional<crate> el_crate_impl::sub_crate_by_name(const std::string& name) { - boost::optional<crate> cr; + std::optional<crate> cr; storage_->db << "SELECT cr.id FROM Crate cr " "JOIN CrateParentList cpl ON (cpl.crateOriginId = cr.id) " "WHERE cr.title = ? " diff --git a/src/djinterop/enginelibrary/el_crate_impl.hpp b/src/djinterop/enginelibrary/el_crate_impl.hpp index 9bf657c..55a24df 100644 --- a/src/djinterop/enginelibrary/el_crate_impl.hpp +++ b/src/djinterop/enginelibrary/el_crate_impl.hpp @@ -41,11 +41,11 @@ public: std::vector<crate> descendants() override; bool is_valid() override; std::string name() override; - boost::optional<crate> parent() override; + std::optional<crate> parent() override; void remove_track(track tr) override; void set_name(std::string name) override; - void set_parent(boost::optional<crate> parent) override; - boost::optional<crate> sub_crate_by_name(const std::string& name) override; + void set_parent(std::optional<crate> parent) override; + std::optional<crate> sub_crate_by_name(const std::string& name) override; std::vector<track> tracks() override; private: diff --git a/src/djinterop/enginelibrary/el_database_impl.cpp b/src/djinterop/enginelibrary/el_database_impl.cpp index a9b6090..84193b4 100644 --- a/src/djinterop/enginelibrary/el_database_impl.cpp +++ b/src/djinterop/enginelibrary/el_database_impl.cpp @@ -70,9 +70,9 @@ transaction_guard el_database_impl::begin_transaction() std::make_unique<el_transaction_guard_impl>(storage_)}; } -boost::optional<crate> el_database_impl::crate_by_id(int64_t id) +std::optional<crate> el_database_impl::crate_by_id(int64_t id) { - boost::optional<crate> cr; + std::optional<crate> cr; storage_->db << "SELECT COUNT(*) FROM Crate WHERE id = ?" << id >> [&](int64_t count) { if (count == 1) @@ -162,7 +162,7 @@ track el_database_impl::create_track(std::string relative_path) << "REPLACE INTO MetaData (id, type, text) VALUES (?, ?, ?)"; for (int64_t type : {1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 12, 13, 15, 16}) { - boost::optional<std::string> text; + std::optional<std::string> text; switch (type) { case 10: @@ -193,7 +193,7 @@ track el_database_impl::create_track(std::string relative_path) "type, value) VALUES (?, ?, ?)"; for (int64_t type = 1; type <= 11 /* 12 */; ++type) { - boost::optional<int64_t> value; + std::optional<int64_t> value; switch (type) { case 5: value = 0; break; @@ -258,9 +258,9 @@ std::vector<crate> el_database_impl::root_crates() return results; } -boost::optional<crate> el_database_impl::root_crate_by_name(const std::string& name) +std::optional<crate> el_database_impl::root_crate_by_name(const std::string& name) { - boost::optional<crate> cr; + std::optional<crate> cr; storage_->db << "SELECT cr.id FROM Crate cr " "JOIN CrateParentList cpl ON (cpl.crateOriginId = cr.id) " "WHERE cr.title = ? " @@ -273,9 +273,9 @@ boost::optional<crate> el_database_impl::root_crate_by_name(const std::string& n return cr; } -boost::optional<track> el_database_impl::track_by_id(int64_t id) +std::optional<track> el_database_impl::track_by_id(int64_t id) { - boost::optional<track> tr; + std::optional<track> tr; storage_->db << "SELECT COUNT(*) FROM Track WHERE id = ?" << id >> [&](int64_t count) { if (count == 1) diff --git a/src/djinterop/enginelibrary/el_database_impl.hpp b/src/djinterop/enginelibrary/el_database_impl.hpp index 9e16314..3249bdd 100644 --- a/src/djinterop/enginelibrary/el_database_impl.hpp +++ b/src/djinterop/enginelibrary/el_database_impl.hpp @@ -33,7 +33,7 @@ public: el_database_impl(std::shared_ptr<el_storage> storage); transaction_guard begin_transaction() override; - boost::optional<djinterop::crate> crate_by_id(int64_t id) override; + std::optional<djinterop::crate> crate_by_id(int64_t id) override; std::vector<djinterop::crate> crates() override; std::vector<djinterop::crate> crates_by_name( const std::string& name) override; @@ -45,9 +45,9 @@ public: void remove_crate(djinterop::crate cr) override; void remove_track(djinterop::track tr) override; std::vector<djinterop::crate> root_crates() override; - boost::optional<djinterop::crate> root_crate_by_name( + std::optional<djinterop::crate> root_crate_by_name( const std::string& name) override; - boost::optional<djinterop::track> track_by_id(int64_t id) override; + std::optional<djinterop::track> track_by_id(int64_t id) override; std::vector<djinterop::track> tracks() override; std::vector<djinterop::track> tracks_by_relative_path( const std::string& relative_path) override; diff --git a/src/djinterop/enginelibrary/el_track_impl.cpp b/src/djinterop/enginelibrary/el_track_impl.cpp index 6356dc7..7a15837 100644 --- a/src/djinterop/enginelibrary/el_track_impl.cpp +++ b/src/djinterop/enginelibrary/el_track_impl.cpp @@ -37,10 +37,10 @@ using std::chrono::system_clock; namespace { -boost::optional<system_clock::time_point> to_time_point( - boost::optional<int64_t> timestamp) +std::optional<system_clock::time_point> to_time_point( + std::optional<int64_t> timestamp) { - boost::optional<system_clock::time_point> result; + std::optional<system_clock::time_point> result; if (timestamp) { result = system_clock::time_point{seconds(*timestamp)}; @@ -48,10 +48,10 @@ boost::optional<system_clock::time_point> to_time_point( return result; } -boost::optional<int64_t> to_timestamp( - boost::optional<system_clock::time_point> time) +std::optional<int64_t> to_timestamp( + std::optional<system_clock::time_point> time) { - boost::optional<int64_t> result; + std::optional<int64_t> result; if (time) { result = duration_cast<seconds>(time->time_since_epoch()).count(); @@ -89,10 +89,10 @@ el_track_impl::el_track_impl(std::shared_ptr<el_storage> storage, int64_t id) { } -boost::optional<std::string> el_track_impl::get_metadata_str( +std::optional<std::string> el_track_impl::get_metadata_str( metadata_str_type type) { - boost::optional<std::string> result; + std::optional<std::string> result; storage_->db << "SELECT text FROM MetaData WHERE id = ? AND " "type = ? AND text IS NOT NULL" << id() << static_cast<int64_t>(type) >> @@ -113,7 +113,7 @@ boost::optional<std::string> el_track_impl::get_metadata_str( } void el_track_impl::set_metadata_str( - metadata_str_type type, boost::optional<std::string> content) + metadata_str_type type, std::optional<std::string> content) { if (content) { @@ -134,9 +134,9 @@ void el_track_impl::set_metadata_str( << id() << static_cast<int64_t>(type) << content; } -boost::optional<int64_t> el_track_impl::get_metadata_int(metadata_int_type type) +std::optional<int64_t> el_track_impl::get_metadata_int(metadata_int_type type) { - boost::optional<int64_t> result; + std::optional<int64_t> result; storage_->db << "SELECT value FROM MetaDataInteger WHERE id = " "? AND type = ? AND value IS NOT NULL" << id() << static_cast<int64_t>(type) >> @@ -157,7 +157,7 @@ boost::optional<int64_t> el_track_impl::get_metadata_int(metadata_int_type type) } void el_track_impl::set_metadata_int( - metadata_int_type type, boost::optional<int64_t> content) + metadata_int_type type, std::optional<int64_t> content) { storage_->db << "REPLACE INTO MetaDataInteger (id, type, value) VALUES (?, ?, ?)" @@ -262,20 +262,20 @@ void el_track_impl::set_adjusted_main_cue(double sample_offset) trans.commit(); } -boost::optional<std::string> el_track_impl::album() +std::optional<std::string> el_track_impl::album() { return get_metadata_str(metadata_str_type::album); } -void el_track_impl::set_album(boost::optional<std::string> album) +void el_track_impl::set_album(std::optional<std::string> album) { set_metadata_str(metadata_str_type::album, album); } -boost::optional<int64_t> el_track_impl::album_art_id() +std::optional<int64_t> el_track_impl::album_art_id() { int64_t cell = get_cell<int64_t>("idAlbumArt"); - boost::optional<int64_t> album_art_id; + std::optional<int64_t> album_art_id; if (cell < 1) { // TODO (haslersn): Throw something. @@ -287,7 +287,7 @@ boost::optional<int64_t> el_track_impl::album_art_id() return album_art_id; } -void el_track_impl::set_album_art_id(boost::optional<int64_t> album_art_id) +void el_track_impl::set_album_art_id(std::optional<int64_t> album_art_id) { if (album_art_id && *album_art_id <= 1) { @@ -297,23 +297,23 @@ void el_track_impl::set_album_art_id(boost::optional<int64_t> album_art_id) // 1 is the magic number for "no album art" } -boost::optional<std::string> el_track_impl::artist() +std::optional<std::string> el_track_impl::artist() { return get_metadata_str(metadata_str_type::artist); } -void el_track_impl::set_artist(boost::optional<std::string> artist) +void el_track_impl::set_artist(std::optional<std::string> artist) { set_metadata_str(metadata_str_type::artist, artist); } -boost::optional<double> el_track_impl::average_loudness() +std::optional<double> el_track_impl::average_loudness() { return get_track_data().average_loudness; } void el_track_impl::set_average_loudness( - boost::optional<double> average_loudness) + std::optional<double> average_loudness) { el_transaction_guard_impl trans{storage_}; auto track_d = get_track_data(); @@ -322,25 +322,25 @@ void el_track_impl::set_average_loudness( trans.commit(); } -boost::optional<int64_t> el_track_impl::bitrate() +std::optional<int64_t> el_track_impl::bitrate() { - return get_cell<boost::optional<int64_t>>("bitrate"); + return get_cell<std::optional<int64_t>>("bitrate"); } -void el_track_impl::set_bitrate(boost::optional<int64_t> bitrate) +void el_track_impl::set_bitrate(std::optional<int64_t> bitrate) { set_cell("bitrate", bitrate); } -boost::optional<double> el_track_impl::bpm() +std::optional<double> el_track_impl::bpm() { - return get_cell<boost::optional<double>>("bpmAnalyzed"); + return get_cell<std::optional<double>>("bpmAnalyzed"); } -void el_track_impl::set_bpm(boost::optional<double> bpm) +void el_track_impl::set_bpm(std::optional<double> bpm) { set_cell("bpmAnalyzed", bpm); - boost::optional<int64_t> ceiled_bpm; + std::optional<int64_t> ceiled_bpm; if (bpm) { ceiled_bpm = static_cast<int64_t>(std::ceil(*bpm)); @@ -348,22 +348,22 @@ void el_track_impl::set_bpm(boost::optional<double> bpm) set_cell("bpm", ceiled_bpm); } -boost::optional<std::string> el_track_impl::comment() +std::optional<std::string> el_track_impl::comment() { return get_metadata_str(metadata_str_type::comment); } -void el_track_impl::set_comment(boost::optional<std::string> comment) +void el_track_impl::set_comment(std::optional<std::string> comment) { set_metadata_str(metadata_str_type::comment, comment); } -boost::optional<std::string> el_track_impl::composer() +std::optional<std::string> el_track_impl::composer() { return get_metadata_str(metadata_str_type::composer); } -void el_track_impl::set_composer(boost::optional<std::string> composer) +void el_track_impl::set_composer(std::optional<std::string> composer) { set_metadata_str(metadata_str_type::composer, composer); } @@ -414,21 +414,21 @@ void el_track_impl::set_default_main_cue(double sample_offset) trans.commit(); } -boost::optional<milliseconds> el_track_impl::duration() +std::optional<milliseconds> el_track_impl::duration() { - boost::optional<milliseconds> result; + std::optional<milliseconds> result; auto smp = sampling(); if (smp) { double secs = smp->sample_count / smp->sample_rate; return milliseconds{static_cast<int64_t>(1000 * secs)}; } - auto secs = get_cell<boost::optional<int64_t>>("length"); + auto secs = get_cell<std::optional<int64_t>>("length"); if (secs) { return milliseconds{*secs * 1000}; } - return boost::none; + return std::nullopt; } std::string el_track_impl::file_extension() @@ -444,23 +444,23 @@ std::string el_track_impl::filename() return get_filename(rel_path); } -boost::optional<std::string> el_track_impl::genre() +std::optional<std::string> el_track_impl::genre() { return get_metadata_str(metadata_str_type::genre); } -void el_track_impl::set_genre(boost::optional<std::string> genre) +void el_track_impl::set_genre(std::optional<std::string> genre) { set_metadata_str(metadata_str_type::genre, genre); } -boost::optional<hot_cue> el_track_impl::hot_cue_at(int32_t index) +std::optional<hot_cue> el_track_impl::hot_cue_at(int32_t index) { auto quick_cues_d = get_quick_cues_data(); return std::move(quick_cues_d.hot_cues[index]); } -void el_track_impl::set_hot_cue_at(int32_t index, boost::optional<hot_cue> cue) +void el_track_impl::set_hot_cue_at(int32_t index, std::optional<hot_cue> cue) { el_transaction_guard_impl trans{storage_}; auto quick_cues_d = get_quick_cues_data(); @@ -469,13 +469,13 @@ void el_track_impl::set_hot_cue_at(int32_t index, boost::optional<hot_cue> cue) trans.commit(); } -std::array<boost::optional<hot_cue>, 8> el_track_impl::hot_cues() +std::array<std::optional<hot_cue>, 8> el_track_impl::hot_cues() { auto quick_cues_d = get_quick_cues_data(); return std::move(quick_cues_d.hot_cues); } -void el_track_impl::set_hot_cues(std::array<boost::optional<hot_cue>, 8> cues) +void el_track_impl::set_hot_cues(std::array<std::optional<hot_cue>, 8> cues) { el_transaction_guard_impl trans{storage_}; // TODO (haslersn): The following can be optimized because in this case we @@ -486,11 +486,11 @@ void el_track_impl::set_hot_cues(std::array<boost::optional<hot_cue>, 8> cues) trans.commit(); } -boost::optional<track_import_info> el_track_impl::import_info() +std::optional<track_import_info> el_track_impl::import_info() { if (get_cell<int64_t>("isExternalTrack") == 0) { - return boost::none; + return std::nullopt; } return track_import_info{get_cell<std::string>("uuidOfExternalDatabase"), get_cell<int64_t>("idTrackInExternalDatabase")}; @@ -499,7 +499,7 @@ boost::optional<track_import_info> el_track_impl::import_info() } void el_track_impl::set_import_info( - const boost::optional<track_import_info>& import_info) + const std::optional<track_import_info>& import_info) { if (import_info) { @@ -533,9 +533,9 @@ bool el_track_impl::is_valid() return valid; } -boost::optional<musical_key> el_track_impl::key() +std::optional<musical_key> el_track_impl::key() { - boost::optional<musical_key> result; + std::optional<musical_key> result; auto key_num = get_metadata_int(metadata_int_type::musical_key); if (key_num) { @@ -544,9 +544,9 @@ boost::optional<musical_key> el_track_impl::key() return result; } -void el_track_impl::set_key(boost::optional<musical_key> key) +void el_track_impl::set_key(std::optional<musical_key> key) { - boost::optional<int64_t> key_num; + std::optional<int64_t> key_num; if (key) { key_num = static_cast<int64_t>(*key); @@ -560,7 +560,7 @@ void el_track_impl::set_key(boost::optional<musical_key> key) trans.commit(); } -boost::optional<system_clock::time_point> el_track_impl::last_accessed_at() +std::optional<system_clock::time_point> el_track_impl::last_accessed_at() { // TODO (haslersn): Is there a difference between @@ -571,7 +571,7 @@ boost::optional<system_clock::time_point> el_track_impl::last_accessed_at() } void el_track_impl::set_last_accessed_at( - boost::optional<system_clock::time_point> accessed_at) + std::optional<system_clock::time_point> accessed_at) { if (accessed_at) { @@ -590,34 +590,34 @@ void el_track_impl::set_last_accessed_at( } else { - set_metadata_int(metadata_int_type::last_accessed_ts, boost::none); + set_metadata_int(metadata_int_type::last_accessed_ts, std::nullopt); } } -boost::optional<system_clock::time_point> el_track_impl::last_modified_at() +std::optional<system_clock::time_point> el_track_impl::last_modified_at() { return to_time_point(get_metadata_int(metadata_int_type::last_modified_ts)); } void el_track_impl::set_last_modified_at( - boost::optional<system_clock::time_point> modified_at) + std::optional<system_clock::time_point> modified_at) { set_metadata_int( metadata_int_type::last_modified_ts, to_timestamp(modified_at)); } -boost::optional<system_clock::time_point> el_track_impl::last_played_at() +std::optional<system_clock::time_point> el_track_impl::last_played_at() { return to_time_point(get_metadata_int(metadata_int_type::last_played_ts)); } void el_track_impl::set_last_played_at( - boost::optional<system_clock::time_point> played_at) + std::optional<system_clock::time_point> played_at) { - static boost::optional<std::string> zero{"0"}; - static boost::optional<std::string> one{"1"}; + static std::optional<std::string> zero{"0"}; + static std::optional<std::string> one{"1"}; set_metadata_str(metadata_str_type::ever_played, played_at ? one : zero); set_metadata_int( metadata_int_type::last_played_ts, to_timestamp(played_at)); @@ -632,13 +632,13 @@ void el_track_impl::set_last_played_at( } } -boost::optional<loop> el_track_impl::loop_at(int32_t index) +std::optional<loop> el_track_impl::loop_at(int32_t index) { auto loops_d = get_loops_data(); return std::move(loops_d.loops[index]); } -void el_track_impl::set_loop_at(int32_t index, boost::optional<loop> l) +void el_track_impl::set_loop_at(int32_t index, std::optional<loop> l) { el_transaction_guard_impl trans{storage_}; auto loops_d = get_loops_data(); @@ -647,13 +647,13 @@ void el_track_impl::set_loop_at(int32_t index, boost::optional<loop> l) trans.commit(); } -std::array<boost::optional<loop>, 8> el_track_impl::loops() +std::array<std::optional<loop>, 8> el_track_impl::loops() { auto loops_d = get_loops_data(); return std::move(loops_d.loops); } -void el_track_impl::set_loops(std::array<boost::optional<loop>, 8> cues) +void el_track_impl::set_loops(std::array<std::optional<loop>, 8> cues) { el_transaction_guard_impl trans{storage_}; loops_data loops_d; @@ -668,12 +668,12 @@ std::vector<waveform_entry> el_track_impl::overview_waveform() return std::move(overview_waveform_d.waveform); } -boost::optional<std::string> el_track_impl::publisher() +std::optional<std::string> el_track_impl::publisher() { return get_metadata_str(metadata_str_type::publisher); } -void el_track_impl::set_publisher(boost::optional<std::string> publisher) +void el_track_impl::set_publisher(std::optional<std::string> publisher) { set_metadata_str(metadata_str_type::publisher, publisher); } @@ -711,16 +711,16 @@ void el_track_impl::set_relative_path(std::string relative_path) set_metadata_str(metadata_str_type::file_extension, extension); } -boost::optional<sampling_info> el_track_impl::sampling() +std::optional<sampling_info> el_track_impl::sampling() { return get_track_data().sampling; } -void el_track_impl::set_sampling(boost::optional<sampling_info> sampling) +void el_track_impl::set_sampling(std::optional<sampling_info> sampling) { el_transaction_guard_impl trans{storage_}; - boost::optional<int64_t> secs; + std::optional<int64_t> secs; if (sampling) { secs = static_cast<int64_t>( @@ -738,7 +738,7 @@ void el_track_impl::set_sampling(boost::optional<sampling_info> sampling) } else { - set_metadata_str(metadata_str_type::duration_mm_ss, boost::none); + set_metadata_str(metadata_str_type::duration_mm_ss, std::nullopt); } set_cell("length", secs); set_cell("lengthCalculated", secs); @@ -781,22 +781,22 @@ void el_track_impl::set_sampling(boost::optional<sampling_info> sampling) trans.commit(); } -boost::optional<std::string> el_track_impl::title() +std::optional<std::string> el_track_impl::title() { return get_metadata_str(metadata_str_type::title); } -void el_track_impl::set_title(boost::optional<std::string> title) +void el_track_impl::set_title(std::optional<std::string> title) { set_metadata_str(metadata_str_type::title, title); } -boost::optional<int32_t> el_track_impl::track_number() +std::optional<int32_t> el_track_impl::track_number() { - return get_cell<boost::optional<int32_t>>("playOrder"); + return get_cell<std::optional<int32_t>>("playOrder"); } -void el_track_impl::set_track_number(boost::optional<int32_t> track_number) +void el_track_impl::set_track_number(std::optional<int32_t> track_number) { set_cell("playOrder", track_number); } @@ -844,12 +844,12 @@ void el_track_impl::set_waveform(std::vector<waveform_entry> waveform) trans.commit(); } -boost::optional<int32_t> el_track_impl::year() +std::optional<int32_t> el_track_impl::year() { - return get_cell<boost::optional<int32_t>>("year"); + return get_cell<std::optional<int32_t>>("year"); } -void el_track_impl::set_year(boost::optional<int32_t> year) +void el_track_impl::set_year(std::optional<int32_t> year) { set_cell("year", year); } diff --git a/src/djinterop/enginelibrary/el_track_impl.hpp b/src/djinterop/enginelibrary/el_track_impl.hpp index bf82ee6..dd556ac 100644 --- a/src/djinterop/enginelibrary/el_track_impl.hpp +++ b/src/djinterop/enginelibrary/el_track_impl.hpp @@ -19,12 +19,11 @@ #include <chrono> #include <memory> +#include <optional> #include <stdexcept> #include <string> #include <vector> -#include <boost/optional.hpp> - #include <djinterop/enginelibrary.hpp> #include <djinterop/enginelibrary/el_storage.hpp> #include <djinterop/enginelibrary/performance_data_format.hpp> @@ -68,18 +67,18 @@ class el_track_impl : public djinterop::track_impl public: el_track_impl(std::shared_ptr<el_storage> storage, int64_t id); - boost::optional<std::string> get_metadata_str(metadata_str_type type); + std::optional<std::string> get_metadata_str(metadata_str_type type); void set_metadata_str( - metadata_str_type type, boost::optional<std::string> content); + metadata_str_type type, std::optional<std::string> content); void set_metadata_str(metadata_str_type type, const std::string& content); - boost::optional<int64_t> get_metadata_int(metadata_int_type type); + std::optional<int64_t> get_metadata_int(metadata_int_type type); void set_metadata_int( - metadata_int_type type, boost::optional<int64_t> content); + metadata_int_type type, std::optional<int64_t> content); template <typename T> T get_cell(const char* column_name) { - boost::optional<T> result; + std::optional<T> result; storage_->db << (std::string{"SELECT "} + column_name + " FROM Track WHERE id = ?") << id() >> @@ -112,7 +111,7 @@ public: template <typename T> T get_perfdata(const char* column_name) { - boost::optional<T> result; + std::optional<T> result; storage_->db << (std::string{"SELECT "} + column_name + " From PerformanceData WHERE id = ?") << id() >> @@ -217,79 +216,79 @@ public: void set_adjusted_beatgrid(std::vector<beatgrid_marker> beatgrid) override; double adjusted_main_cue() override; void set_adjusted_main_cue(double sample_offset) override; - boost::optional<std::string> album() override; - void set_album(boost::optional<std::string> album) override; - boost::optional<int64_t> album_art_id() override; - void set_album_art_id(boost::optional<int64_t> album_art_id) override; - boost::optional<std::string> artist() override; - void set_artist(boost::optional<std::string> artist) override; - boost::optional<double> average_loudness() override; + std::optional<std::string> album() override; + void set_album(std::optional<std::string> album) override; + std::optional<int64_t> album_art_id() override; + void set_album_art_id(std::optional<int64_t> album_art_id) override; + std::optional<std::string> artist() override; + void set_artist(std::optional<std::string> artist) override; + std::optional<double> average_loudness() override; void set_average_loudness( - boost::optional<double> average_loudness) override; - boost::optional<int64_t> bitrate() override; - void set_bitrate(boost::optional<int64_t> bitrate) override; - boost::optional<double> bpm() override; - void set_bpm(boost::optional<double> bpm) override; - boost::optional<std::string> comment() override; - void set_comment(boost::optional<std::string> comment) override; - boost::optional<std::string> composer() override; - void set_composer(boost::optional<std::string> composer) override; + std::optional<double> average_loudness) override; + std::optional<int64_t> bitrate() override; + void set_bitrate(std::optional<int64_t> bitrate) override; + std::optional<double> bpm() override; + void set_bpm(std::optional<double> bpm) override; + std::optional<std::string> comment() override; + void set_comment(std::optional<std::string> comment) override; + std::optional<std::string> composer() override; + void set_composer(std::optional<std::string> composer) override; std::vector<djinterop::crate> containing_crates() override; database db() override; std::vector<beatgrid_marker> default_beatgrid() override; void set_default_beatgrid(std::vector<beatgrid_marker> beatgrid) override; double default_main_cue() override; void set_default_main_cue(double sample_offset) override; - boost::optional<std::chrono::milliseconds> duration() override; + std::optional<std::chrono::milliseconds> duration() override; std::string file_extension() override; std::string filename() override; - boost::optional<std::string> genre() override; - void set_genre(boost::optional<std::string> genre) override; - boost::optional<hot_cue> hot_cue_at(int32_t index) override; - void set_hot_cue_at(int32_t index, boost::optional<hot_cue> cue) override; - std::array<boost::optional<hot_cue&g |