Re: [Gstreamer-openmax] [PATCH 0/2] Some random clean-ups

[Felipe C. <fel...@no...>]

Hi Rob,

Sorry for the delay.

ro...@ti... wrote:
> On Wed, Nov 3, 2010 at 8:56 PM, Rob Clark <ro...@ti...> wrote:
> > Use common submodule and pre-commit hook, and cosemetic gstomx.h whitespace
> > change.
> >
> > Rob Clark (2):
> >  use common submodule
> >  minor formatting cleanup
> >
> >  .gitignore               |    3 +
> >  .gitmodules              |    3 +
> >  autogen.sh               |   60 ++++++++--
> >  build-aux/gst-autogen.sh |  294 ----------------------------------------------
> >  common                   |    1 +
> >  configure.ac             |   17 ++-
> >  omx/gstomx.h             |    6 +-
> >  shave-libtool.in         |   69 -----------
> >  shave.in                 |   79 ------------
> >  9 files changed, 74 insertions(+), 458 deletions(-)
> >  create mode 100644 .gitmodules
> >  delete mode 100644 build-aux/gst-autogen.sh
> >  create mode 160000 common
> >  delete mode 100644 shave-libtool.in
> >  delete mode 100644 shave.in
> >
> >
> 
> So I take it that silence means that no one has any objections to this
> patchset or the tempate caps patchset?

I don't like the common subdir one, but the rest look good. Thanks.

-- 
Felipe Contreras