On Tue, Nov 29, 2016 at 1:34 PM, Diego Biurrun <di...@biurrun.de> wrote:
> index c823388..0fa3786 100644
> --- a/libavdevice/libdc1394.c
> +++ b/libavdevice/libdc1394.c
> @@ -20,26 +20,7 @@
>   * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 
> USA
>   */
>
> -#include "config.h"
> -
> -#if HAVE_LIBDC1394_2
>  #include <dc1394/dc1394.h>
> -#elif HAVE_LIBDC1394_1
> -#include <libraw1394/raw1394.h>
> -#include <libdc1394/dc1394_control.h>
> -
> -#define DC1394_VIDEO_MODE_320x240_YUV422 MODE_320x240_YUV422
> -#define DC1394_VIDEO_MODE_640x480_YUV411 MODE_640x480_YUV411
> -#define DC1394_VIDEO_MODE_640x480_YUV422 MODE_640x480_YUV422
> -#define DC1394_FRAMERATE_1_875 FRAMERATE_1_875
> -#define DC1394_FRAMERATE_3_75  FRAMERATE_3_75
> -#define DC1394_FRAMERATE_7_5   FRAMERATE_7_5
> -#define DC1394_FRAMERATE_15    FRAMERATE_15
> -#define DC1394_FRAMERATE_30    FRAMERATE_30
> -#define DC1394_FRAMERATE_60    FRAMERATE_60
> -#define DC1394_FRAMERATE_120   FRAMERATE_120
> -#define DC1394_FRAMERATE_240   FRAMERATE_240
> -#endif
>
>  #include "libavutil/imgutils.h"
>  #include "libavutil/internal.h"
> @@ -54,15 +35,9 @@
>
>  typedef struct dc1394_data {
>      AVClass *class;
> -#if HAVE_LIBDC1394_1
> -    raw1394handle_t handle;
> -    dc1394_cameracapture camera;
> -    int channel;
> -#elif HAVE_LIBDC1394_2
>      dc1394_t *d;
>      dc1394camera_t *camera;
>      dc1394video_frame_t *frame;
> -#endif
>      int current_frame;
>      int  frame_rate;        /**< frames per 1000 seconds (fps * 1000) */
>      char *video_size;       /**< String describing video size, set by a 
> private option. */
> @@ -104,9 +79,6 @@ struct dc1394_frame_rate {
>  #define OFFSET(x) offsetof(dc1394_data, x)
>  #define DEC AV_OPT_FLAG_DECODING_PARAM
>  static const AVOption options[] = {
> -#if HAVE_LIBDC1394_1
> -    { "channel", "", offsetof(dc1394_data, channel), AV_OPT_TYPE_INT, {.i64 
> = 0}, 0, INT_MAX, AV_OPT_FLAG_DECODING_PARAM },
> -#endif

should this option be kept around and converted into a warning, so not
to break random scripts still in the wild?

lgtm otherwise
-- 
Vittorio
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to