From 89af92333c86bc5911598e80ce756dd894d9f844 Mon Sep 17 00:00:00 2001 From: Rosen Penev Date: Sun, 21 Jan 2024 18:07:46 -0800 Subject: [PATCH] clang-tidy: use dynamic_cast Found with cppcoreguidelines-pro-type-static-cast-downcast Signed-off-by: Rosen Penev --- tests/test_id3v2framefactory.cpp | 36 ++++++++++++++++---------------- 1 file changed, 18 insertions(+), 18 deletions(-) diff --git a/tests/test_id3v2framefactory.cpp b/tests/test_id3v2framefactory.cpp index 10de6352..98728a16 100644 --- a/tests/test_id3v2framefactory.cpp +++ b/tests/test_id3v2framefactory.cpp @@ -225,13 +225,13 @@ public: factory); }, [](const File &f) { - return static_cast(f).hasID3v2Tag(); + return dynamic_cast(f).hasID3v2Tag(); }, [](File &f) { - return static_cast(f).ID3v2Tag(true); + return dynamic_cast(f).ID3v2Tag(true); }, [](File &f) { - return static_cast(f).strip(); + return dynamic_cast(f).strip(); } ); } @@ -249,13 +249,13 @@ public: factory); }, [](const File &f) { - return static_cast(f).hasID3v2Tag(); + return dynamic_cast(f).hasID3v2Tag(); }, [](File &f) { - return static_cast(f).ID3v2Tag(true); + return dynamic_cast(f).ID3v2Tag(true); }, [](File &f) { - static_cast(f).strip(); + dynamic_cast(f).strip(); return f.save(); } ); @@ -274,13 +274,13 @@ public: TrueAudio::Properties::Average, factory); }, [](const File &f) { - return static_cast(f).hasID3v2Tag(); + return dynamic_cast(f).hasID3v2Tag(); }, [](File &f) { - return static_cast(f).ID3v2Tag(true); + return dynamic_cast(f).ID3v2Tag(true); }, [](File &f) { - static_cast(f).strip(); + dynamic_cast(f).strip(); return f.save(); } ); @@ -299,13 +299,13 @@ public: fileName, true, RIFF::WAV::Properties::Average, factory); }, [](const File &f) { - return static_cast(f).hasID3v2Tag(); + return dynamic_cast(f).hasID3v2Tag(); }, [](File &f) { - return static_cast(f).ID3v2Tag(); + return dynamic_cast(f).ID3v2Tag(); }, [](File &f) { - static_cast(f).strip(); + dynamic_cast(f).strip(); return true; } ); @@ -324,10 +324,10 @@ public: fileName, true, RIFF::AIFF::Properties::Average, factory); }, [](const File &f) { - return static_cast(f).hasID3v2Tag(); + return dynamic_cast(f).hasID3v2Tag(); }, [](File &f) { - return static_cast(f).tag(); + return dynamic_cast(f).tag(); }, [](File &f) { f.setProperties({}); @@ -352,7 +352,7 @@ public: return !f.tag()->isEmpty(); }, [](File &f) { - return static_cast(f).tag(); + return dynamic_cast(f).tag(); }, [](File &f) { f.setProperties({}); @@ -374,13 +374,13 @@ public: fileName, true, DSDIFF::Properties::Average, factory); }, [](const File &f) { - return static_cast(f).hasID3v2Tag(); + return dynamic_cast(f).hasID3v2Tag(); }, [](File &f) { - return static_cast(f).ID3v2Tag(true); + return dynamic_cast(f).ID3v2Tag(true); }, [](File &f) { - static_cast(f).strip(); + dynamic_cast(f).strip(); return true; } );