[FFmpeg-devel] [PATCH] doc/examples/muxing: make compatible with C++

Stefano Sabatini stefasab at gmail.com
Wed Mar 19 10:40:35 CET 2014


On date Wednesday 2014-03-19 11:45:08 +0530, anshul encoded:
[...]
> From dd883b6260e9622ab030dca3afc98e651c5cfd01 Mon Sep 17 00:00:00 2001
> From: Anshul Maheshwari <er.anshul.maheshwari at gmail.com>
> Date: Wed, 19 Mar 2014 11:42:06 +0530
> Subject: [PATCH] added CPlusPlus examples
> 
> ---
>  MAINTAINERS                            |   5 +
>  Makefile                               |   7 +-
>  doc/Makefile                           |  17 +-
>  doc/examples_cpp/Makefile              |  44 ++
>  doc/examples_cpp/README                |  23 +
>  doc/examples_cpp/avcodec.cpp           | 659 ++++++++++++++++++++++++++++
>  doc/examples_cpp/avio_reading.cpp      | 134 ++++++
>  doc/examples_cpp/demuxing_decoding.cpp | 387 +++++++++++++++++
>  doc/examples_cpp/filter_audio.cpp      | 365 ++++++++++++++++
>  doc/examples_cpp/filtering_audio.cpp   | 283 +++++++++++++
>  doc/examples_cpp/filtering_video.cpp   | 263 ++++++++++++
>  doc/examples_cpp/metadata.cpp          |  58 +++
>  doc/examples_cpp/muxing.cpp            | 614 +++++++++++++++++++++++++++
>  doc/examples_cpp/remuxing.cpp          | 173 ++++++++
>  doc/examples_cpp/resampling_audio.cpp  | 218 ++++++++++
>  doc/examples_cpp/scaling_video.cpp     | 143 +++++++
>  doc/examples_cpp/transcode_aac.cpp     | 754 +++++++++++++++++++++++++++++++++
>  doc/examples_cpp/transcoding.cpp       | 602 ++++++++++++++++++++++++++
>  18 files changed, 4746 insertions(+), 3 deletions(-)
>  create mode 100644 doc/examples_cpp/Makefile
>  create mode 100644 doc/examples_cpp/README
>  create mode 100644 doc/examples_cpp/avcodec.cpp
>  create mode 100644 doc/examples_cpp/avio_reading.cpp
>  create mode 100644 doc/examples_cpp/demuxing_decoding.cpp
>  create mode 100644 doc/examples_cpp/filter_audio.cpp
>  create mode 100644 doc/examples_cpp/filtering_audio.cpp
>  create mode 100644 doc/examples_cpp/filtering_video.cpp
>  create mode 100644 doc/examples_cpp/metadata.cpp
>  create mode 100644 doc/examples_cpp/muxing.cpp
>  create mode 100644 doc/examples_cpp/remuxing.cpp
>  create mode 100644 doc/examples_cpp/resampling_audio.cpp
>  create mode 100644 doc/examples_cpp/scaling_video.cpp
>  create mode 100644 doc/examples_cpp/transcode_aac.cpp
>  create mode 100644 doc/examples_cpp/transcoding.cpp

As documentation maintainer I'm against this, since it's going to be a
hell to maintain, and I don't want to duplicate code changes/tests and
have to keep C/C++ code in synch.

What I suggest instead: we extend documentation on the wiki regarding
FFmpeg support with C++, so that the user knows exactly what
conversions are required when porting the code to C++.
-- 
FFmpeg = Forgiving and Fanciful Multimedia Ponderous Ecletic God


More information about the ffmpeg-devel mailing list