[vlc-commits] input: deprecate b_dead, b_eof and b_error

Rémi Denis-Courmont git at videolan.org
Thu Jun 4 23:10:33 CEST 2015


vlc | branch: master | Rémi Denis-Courmont <remi at remlab.net> | Fri Jun  5 00:09:00 2015 +0300| [729adddade7a63cc895d715a928dcfa1d9b954be] | committer: Rémi Denis-Courmont

input: deprecate b_dead, b_eof and b_error

Those values are writable by the input thread, and thus inaccessible
to other threads. Any remaining use of those (in VLM, podcast and
interfaces notably) is an undefined memory access, a bug.

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=729adddade7a63cc895d715a928dcfa1d9b954be
---

 include/vlc_input.h |    9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/include/vlc_input.h b/include/vlc_input.h
index 475a6d2..0ce44d8 100644
--- a/include/vlc_input.h
+++ b/include/vlc_input.h
@@ -219,17 +219,16 @@ typedef struct input_resource_t input_resource_t;
 
 /**
  * Main structure representing an input thread. This structure is mostly
- * private. The only public fields are READ-ONLY. You must use the helpers
- * to modify them
+ * private. The only public fields are read-only and constant.
  */
 struct input_thread_t
 {
     VLC_COMMON_MEMBERS
 
-    bool b_error;
-    bool b_eof;
+    bool b_error VLC_DEPRECATED;
+    bool b_eof VLC_DEPRECATED;
     bool b_preparsing;
-    bool b_dead;
+    bool b_dead VLC_DEPRECATED;
 
     /* All other data is input_thread is PRIVATE. You can't access it
      * outside of src/input */



More information about the vlc-commits mailing list