1 files changed, 15 insertions, 51 deletions

diff --git a/src/demuxers/demux_smjpeg.c b/src/demuxers/demux_smjpeg.c
index 04a46fa78..6f0cb3832 100644
--- a/src/demuxers/demux_smjpeg.c
+++ b/src/demuxers/demux_smjpeg.c
@@ -39,10 +39,10 @@
 /* #define LOG_VERBOSE */
 /* #define LOG */
 
-#include "xine_internal.h"
-#include "xineutils.h"
-#include "compat.h"
-#include "demux.h"
+#include <xine/xine_internal.h>
+#include <xine/xineutils.h>
+#include <xine/compat.h>
+#include <xine/demux.h>
 #include "bswap.h"
 #include "group_games.h"
 
@@ -141,6 +141,8 @@ static int open_smjpeg_file(demux_smjpeg_t *this) {
       this->bih.biHeight = _X_BE_16(&header_chunk[10]);
       this->bih.biCompression = *(uint32_t *)&header_chunk[12];
       this->video_type = _x_fourcc_to_buf_video(this->bih.biCompression);
+      if (!this->video_type)
+        _x_report_video_fourcc (this->stream->xine, LOG_MODULE, this->bih.biCompression);
       break;
 
     case _SND_TAG:
@@ -160,6 +162,8 @@ static int open_smjpeg_file(demux_smjpeg_t *this) {
       } else {
         audio_codec = *(uint32_t *)&header_chunk[8];
         this->audio_type = _x_formattag_to_buf_audio(audio_codec);
+        if (!this->audio_type)
+          _x_report_audio_format_tag (this->stream->xine, LOG_MODULE, audio_codec);
       }
       break;
 
@@ -356,12 +360,6 @@ static int demux_smjpeg_seek (demux_plugin_t *this_gen, off_t start_pos, int sta
 }
 
 
-static void demux_smjpeg_dispose (demux_plugin_t *this_gen) {
-  demux_smjpeg_t *this = (demux_smjpeg_t *) this_gen;
-
-  free(this);
-}
-
 static int demux_smjpeg_get_status (demux_plugin_t *this_gen) {
   demux_smjpeg_t *this = (demux_smjpeg_t *) this_gen;
 
@@ -401,7 +399,7 @@ static demux_plugin_t *open_plugin (demux_class_t *class_gen, xine_stream_t *str
   this->demux_plugin.send_headers      = demux_smjpeg_send_headers;
   this->demux_plugin.send_chunk        = demux_smjpeg_send_chunk;
   this->demux_plugin.seek              = demux_smjpeg_seek;
-  this->demux_plugin.dispose           = demux_smjpeg_dispose;
+  this->demux_plugin.dispose           = default_demux_plugin_dispose;
   this->demux_plugin.get_status        = demux_smjpeg_get_status;
   this->demux_plugin.get_stream_length = demux_smjpeg_get_stream_length;
   this->demux_plugin.get_capabilities  = demux_smjpeg_get_capabilities;
@@ -412,19 +410,7 @@ static demux_plugin_t *open_plugin (demux_class_t *class_gen, xine_stream_t *str
 
   switch (stream->content_detection_method) {
 
-  case METHOD_BY_EXTENSION: {
-    const char *extensions, *mrl;
-
-    mrl = input->get_mrl (input);
-    extensions = class_gen->get_extensions (class_gen);
-
-    if (!_x_demux_check_extension (mrl, extensions)) {
-      free (this);
-      return NULL;
-    }
-  }
-  /* falling through is intended */
-
+  case METHOD_BY_MRL:
   case METHOD_BY_CONTENT:
   case METHOD_EXPLICIT:
 
@@ -443,39 +429,17 @@ static demux_plugin_t *open_plugin (demux_class_t *class_gen, xine_stream_t *str
   return &this->demux_plugin;
 }
 
-static const char *get_description (demux_class_t *this_gen) {
-  return "SMJPEG file demux plugin";
-}
-
-static const char *get_identifier (demux_class_t *this_gen) {
-  return "SMJPEG";
-}
-
-static const char *get_extensions (demux_class_t *this_gen) {
-  return "mjpg";
-}
-
-static const char *get_mimetypes (demux_class_t *this_gen) {
-  return NULL;
-}
-
-static void class_dispose (demux_class_t *this_gen) {
-  demux_smjpeg_class_t *this = (demux_smjpeg_class_t *) this_gen;
-
-  free (this);
-}
-
 void *demux_smjpeg_init_plugin (xine_t *xine, void *data) {
   demux_smjpeg_class_t     *this;
 
   this = calloc(1, sizeof(demux_smjpeg_class_t));
 
   this->demux_class.open_plugin     = open_plugin;
-  this->demux_class.get_description = get_description;
-  this->demux_class.get_identifier  = get_identifier;
-  this->demux_class.get_mimetypes   = get_mimetypes;
-  this->demux_class.get_extensions  = get_extensions;
-  this->demux_class.dispose         = class_dispose;
+  this->demux_class.description     = N_("SMJPEG file demux plugin");
+  this->demux_class.identifier      = "SMJPEG";
+  this->demux_class.mimetypes       = NULL;
+  this->demux_class.extensions      = "mjpg";
+  this->demux_class.dispose         = default_demux_class_dispose;
 
   return this;
 }