[MPlayer-cvslog] r36858 - in trunk: gui/dialog/equalizer.c gui/dialog/menu.c gui/dialog/playlis

Reimar Döffinger Reimar.Doeffinger at gmx.de
Mon Feb 17 00:42:32 CET 2014


On Mon, Feb 17, 2014 at 12:33:58AM +0100, Ingo Brückl wrote:
> Reimar Döffinger wrote on Sun, 16 Feb 2014 17:42:52 +0100:
> 
> > On Sun, Feb 16, 2014 at 04:44:36PM +0100, reimar wrote:
> >> Author: reimar
> >> Date: Sun Feb 16 16:44:36 2014
> >> New Revision: 36858
> >>
> >> Log:
> >> Avoid many pointer cast warnings.
> >>
> >> Fixes most occurrences of
> >> "cast from pointer to integer of different size"
> >> and
> >> "cast to pointer from integer of different size"
> >> warnings.
> 
> > Please tell me if I should send patches for review to
> > the list first for this kind of thing.
> > It seemed simple enough to me, with a not too high risk
> > of causing conflicts so I decided to just commit.
> 
> In general I'm fine with it, but the sloppy GTK casts should be fixed
> differently. There are GTK macros for that. I'll go over it.
> 
> BTW, although apparently not missing, should stdint.h be included?

Several of those files already used int*_t so I concluded it was
assumed to be there anyway.
It might also be that these files include inttypes.h instead which
is fine as well.


More information about the MPlayer-cvslog mailing list