Merge pull request #194 from alex-taylor/issue-193

Changed file-open mode from 'chunked' to 'no buffer' to match new behaviour in Kodi 21
kodi-pvr · Dec 22, 2024 · 7125ca4 · 7125ca4
2 parents a5658e0 + 4821deb
commit 7125ca4
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 2 deletions.
diff --git a/pvr.argustv/addon.xml.in b/pvr.argustv/addon.xml.in
@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <addon
   id="pvr.argustv"
-  version="21.0.2"
+  version="21.1.0"
   name="ARGUS TV client"
   provider-name="Fred Hoogduin, Marcel Groothuis">
   <requires>@ADDON_DEPENDS@</requires>

diff --git a/pvr.argustv/changelog.txt b/pvr.argustv/changelog.txt
@@ -1,3 +1,6 @@
+v21.1.0
+- Updated file-open mode to ADDON_READ_NO_BUFFER for compatibility with Kodi 21
+
 v21.0.2
 - Translations updates from Weblate
 	- es_es, et_ee

diff --git a/src/lib/tsreader/FileReader.cpp b/src/lib/tsreader/FileReader.cpp
@@ -51,6 +51,9 @@ long FileReader::SetFileName(const std::string& fileName)
 //
 long FileReader::OpenFile()
 {
+  // Temporary definition - this flag was added in Kodi 21, but doesn't appear in filesystem.h until Kodi 22.
+  int ADDON_READ_NO_BUFFER = 0x200;
+
   int Tmo = 25; //5 in MediaPortal
 
   // Is the file already opened
@@ -72,7 +75,7 @@ long FileReader::OpenFile()
   do
   {
     kodi::Log(ADDON_LOG_INFO, "FileReader::OpenFile() %s.", m_fileName.c_str());
-    if (m_file.OpenFile(m_fileName, ADDON_READ_CHUNKED))
+    if (m_file.OpenFile(m_fileName, ADDON_READ_NO_BUFFER))
     {
       break;
     }