[Ffmpeg-devel] Re: [PATCH] RTP/RTSP and MPEG4-AAC audio (Repost)

Dave Dodge dododge
Fri May 6 20:46:46 CEST 2005


On Fri, May 06, 2005 at 06:06:52PM +0200, Romain DEGEZ wrote:
> The fact is that with Michael answer, I can't figure out what is 
> "cosmetical" and what is not :-/

>From a very quick glance through the patches I see a few whitespace
changes.  For example:

-        get_word_sep(transport_protocol, sizeof(transport_protocol),
-                     "/", &p);
+        get_word_sep(transport_protocol, sizeof(transport_protocol), "/", &p);

and:

 } 
  
-
 static AVInputFormat sdp_demux = {

I'm not one of the source maintainers, but I'm guessing this sort of
thing would be considered a cosmetic change.  Some projects don't like
having whitespace changes and real code changes mixed into the same
patch.

                                                  -Dave Dodge





More information about the ffmpeg-devel mailing list