Revert "Fixes #10490. Don't return from safe_read() early on stopreads if we haven't read any bytes, this looks like an eof to higher levels of code."

I misread a report from Jim Stichnot, it was oldfile and not stopreads causing problems.

This reverts commit 393295c.

Author: daniel-kristjansson

mythtv/libs/libmythtv/fileringbuffer.cpp

@@ -429,6 +429,9 @@ int FileRingBuffer::safe_read(int fd, void *data, uint sz)
         return 0;
     }
 
+    if (stopreads)
+        return 0;
+
     while (tot < sz)
     {
         ret = read(fd2, (char *)data + tot, sz - tot);
@@ -467,7 +470,7 @@ int FileRingBuffer::safe_read(int fd, void *data, uint sz)
                 break;
             }
         }
-        if (stopreads && (tot > 0))
+        if (stopreads)
             break;
         if (tot < sz)
             usleep(60000);