mbox series

[FFmpeg-devel,v2,0/2] Simon & Schuster Interactive VAG demuxer + decoder.

Message ID 20200206065801.28118-1-zane@zanevaniperen.com
Headers show
Series Simon & Schuster Interactive VAG demuxer + decoder. | expand

Message

Zane van Iperen Feb. 6, 2020, 6:58 a.m. UTC 
Hi all,

This patchset adds support for the VAG container and ADPCM variant used
by some Simon & Schuster Interactive games such as 'Real War',
and 'Real War: Rogue States'.

It has been tested against VAG files from both games.

v2:
  - simplify the demuxer
  - fix error in header structure

Some things to note:
* SSI's VAG has no relation to the existing PS2 VAG.
  I've named it 'kvag' (after its tag), but am open to suggestions if this
  is inappropriate (ssi_vag?).

Zane


Zane van Iperen (2):
  avcodec: add decoder for Simon & Schuster Interactive's ADPCM variant
  avformat: add demuxer for Simon & Schuster Interactive's VAG format

 libavcodec/Makefile      |   1 +
 libavcodec/adpcm.c       |  10 ++++
 libavcodec/allcodecs.c   |   1 +
 libavcodec/avcodec.h     |   1 +
 libavcodec/codec_desc.c  |   7 +++
 libavcodec/version.h     |   4 +-
 libavformat/Makefile     |   1 +
 libavformat/allformats.c |   1 +
 libavformat/kvag.c       | 117 +++++++++++++++++++++++++++++++++++++++
 libavformat/version.h    |   2 +-
 10 files changed, 142 insertions(+), 3 deletions(-)
 create mode 100644 libavformat/kvag.c

Comments

Paul B Mahol Feb. 6, 2020, 8:55 a.m. UTC | #1 
LGTM

On 2/6/20, Zane van Iperen <zane@zanevaniperen.com> wrote:
> Hi all,
>
> This patchset adds support for the VAG container and ADPCM variant used
> by some Simon & Schuster Interactive games such as 'Real War',
> and 'Real War: Rogue States'.
>
> It has been tested against VAG files from both games.
>
> v2:
>   - simplify the demuxer
>   - fix error in header structure
>
> Some things to note:
> * SSI's VAG has no relation to the existing PS2 VAG.
>   I've named it 'kvag' (after its tag), but am open to suggestions if this
>   is inappropriate (ssi_vag?).
>
> Zane
>
>
> Zane van Iperen (2):
>   avcodec: add decoder for Simon & Schuster Interactive's ADPCM variant
>   avformat: add demuxer for Simon & Schuster Interactive's VAG format
>
>  libavcodec/Makefile      |   1 +
>  libavcodec/adpcm.c       |  10 ++++
>  libavcodec/allcodecs.c   |   1 +
>  libavcodec/avcodec.h     |   1 +
>  libavcodec/codec_desc.c  |   7 +++
>  libavcodec/version.h     |   4 +-
>  libavformat/Makefile     |   1 +
>  libavformat/allformats.c |   1 +
>  libavformat/kvag.c       | 117 +++++++++++++++++++++++++++++++++++++++
>  libavformat/version.h    |   2 +-
>  10 files changed, 142 insertions(+), 3 deletions(-)
>  create mode 100644 libavformat/kvag.c
>
> --
> 2.17.1
>
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
Michael Niedermayer Feb. 6, 2020, 2:29 p.m. UTC | #2 
On Thu, Feb 06, 2020 at 09:55:41AM +0100, Paul B Mahol wrote:
> LGTM

will apply

thx

[...]