[MPlayer-dev-eng] [PATCH 2/3] Fix some warnings with Objective-C code and declarations
Alexander Strasser
eclipse7 at gmx.net
Mon Jul 15 21:26:44 CEST 2013
On 2013-07-15 21:13 +0200, Alexander Strasser wrote:
> On 2013-07-15 08:09 +0800, Xidorn Quan wrote:
> > On Sun, Jul 14, 2013 at 8:38 PM, Jiang Jiang <gzjjgod at gmail.com> wrote:
> >
> > > ---
> > > libvo/osx_objc_common.h | 2 +-
> > > libvo/osx_objc_common.m | 4 ++--
> > > libvo/vo_corevideo.h | 2 +-
> > > libvo/vo_corevideo.m | 7 ++++---
> > > 4 files changed, 8 insertions(+), 7 deletions(-)
> > >
> [...]
> >
> > LGTM.
>
> IIRC you are on OSX and you can commit to the MPlayer repository.
>
> So it should be fine for you to apply this if you think it is correct.
> Do not forget the attribution to the author, it can happen easily.
>
> Maybe splitting the patch into rename method and include part
> would be better as well as is including the compiler warning in
> the body of the commit message.
>
> I just forgot because I cut out the patch from the email, but would
> configureSize be a better name than configWidth?
^^^^^^^^^
Oooops, I meant configSize no need to spell out configure of course.
Alexander
More information about the MPlayer-dev-eng
mailing list