[multimedia/kid3] src/plugins/taglibmetadata: kid3-cli: Make Chapters and Event Timing Codes visible

Urs Fleisch <[email protected]>
Newsgroups gmane.comp.kde.cvs
Message-ID <[email protected]>
Git commit 1cceff125bfa42a5076143c6e502ea1d206fe33f by Urs Fleisch.
Committed on 20/07/2026 at 03:00.
Pushed by ufleisch into branch 'master'.

kid3-cli: Make Chapters and Event Timing Codes visible

M  +2    -2    src/plugins/taglibmetadata/taglibmatroskasupport.cpp
M  +1    -1    src/plugins/taglibmetadata/taglibmpegsupport.cpp

https://invent.kde.org/multimedia/kid3/-/commit/1cceff125bfa42a5076143c6e502ea1d206fe33f

diff --git a/src/plugins/taglibmetadata/taglibmatroskasupport.cpp b/src/plugins/taglibmetadata/taglibmatroskasupport.cpp
index 8ee1ffc0..a80024c2 100644
--- a/src/plugins/taglibmetadata/taglibmatroskasupport.cpp
+++ b/src/plugins/taglibmetadata/taglibmatroskasupport.cpp
@@ -271,7 +271,7 @@ void matroskaChapterEditionToFrame(
   const QString description = toSimpleTextOrJson(editionMap);
   frame.setExtendedType(
     Frame::ExtendedType(Frame::FT_Other, QLatin1String("Chapters")));
-  frame.setValue(description);
+  frame.setValue(description.isNull() ? QLatin1String("") : description);
 
   TagLib::String language;
   QVariantList synchedData;
@@ -337,7 +337,7 @@ TagLib::Matroska::ChapterEdition frameToMatroskaChapterEdition(const Frame& fram
   QListIterator it(synchedData);
   while (it.hasNext()) {
     auto time = static_cast<unsigned long long>(it.next().toDouble() * 1E6);
-    auto text = it.next().toString();
+    auto text = it.next().toString().trimmed();
     if (chapterDataIndex >= 0 && !chapterData[chapterDataIndex].timeEnd) {
       chapterData[chapterDataIndex].timeEnd = time;
       if (text.isEmpty()) {
diff --git a/src/plugins/taglibmetadata/taglibmpegsupport.cpp b/src/plugins/taglibmetadata/taglibmpegsupport.cpp
index 88a67698..b54bace6 100644
--- a/src/plugins/taglibmetadata/taglibmpegsupport.cpp
+++ b/src/plugins/taglibmetadata/taglibmpegsupport.cpp
@@ -979,7 +979,7 @@ QString getFieldsFromEtcoFrame(
   field.m_value = synchedData;
   fields.push_back(field);
 
-  return QString();
+  return QString(QLatin1String(""));
 }
 
 /**
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.