[MPlayer-cvslog] r32072 - trunk/udp_sync.h
    Reimar Döffinger 
    Reimar.Doeffinger at gmx.de
       
    Thu Sep  9 22:37:50 CEST 2010
    
    
  
On Thu, Sep 09, 2010 at 05:49:24PM +0200, Diego Biurrun wrote:
> > What do you think about this warning?
> > Should we try to disable it, should I add a
> > struct MPContext;
> > to that header or re-add the include?
> 
> Readd the #include.
> 
> > I'd rather have such tiny headers not include other rather huge ones.
> 
> If you worry about header size, reduce the size of the headers you
> worry about.  However, headers should work standalone and #include
> all of their dependencies, no matter how large.
It does work just fine stand-alone, there's just a (IMO) rather silly
gcc warning.
    
    
More information about the MPlayer-cvslog
mailing list