[vlc-devel] [RFC] [PATCH 0/2] IOMX support for Android

Martin Storsjö martin at martin.st
Wed Sep 21 14:49:31 CEST 2011


This is the reworked IOMX support for Android.

Compared to the code in the previous version, this keeps
omxil.c as plain C (and only has some very minimal changes,
to hook in the IOMX code), and contains all the C++ code in a
separate file, which implements a normal OMX interface
outwards, which omxil.c then uses, relaying the calls to IOMX.

This is still very much a RFC patchset. Things that might
need attention:
- Missing check for the right headers in configure.ac
- This creates a separate module "iomx", which builds from the
  same source files as the "omxil" module. Is this ok, or would
  it better to keep it named the same, but enabling this behaviour
  only if some configure flag is specified? The separate module
  has some ugly workaround to conditionally set a define when
  building omxil.c for the iomx module.
- Coding style (I'm not that used to the VLC style, sorry)

Martin Storsjö (2):
  omxil: Make GetAudioParamSize nonstatic
  omxil: Allow using IOMX on Android

 configure.ac                      |   12 ++
 modules/codec/omxil/Modules.am    |    4 +
 modules/codec/omxil/iomx.cpp      |  394 +++++++++++++++++++++++++++++++++++++
 modules/codec/omxil/iomx.h        |   33 +++
 modules/codec/omxil/omxil-iomx.c  |    2 +
 modules/codec/omxil/omxil.c       |    7 +
 modules/codec/omxil/omxil_utils.h |    1 +
 modules/codec/omxil/utils.c       |    2 +-
 8 files changed, 454 insertions(+), 1 deletions(-)
 create mode 100644 modules/codec/omxil/iomx.cpp
 create mode 100644 modules/codec/omxil/iomx.h
 create mode 100644 modules/codec/omxil/omxil-iomx.c

-- 
1.7.2.5




More information about the vlc-devel mailing list