[FFmpeg-devel] [patch]add mmsh protocol and extract common code for mmst.c

[Stefano Sabatini]

On date Friday 2010-08-20 21:42:57 +0800, zhentan feng encoded:
> Hi
> 
> On Fri, Aug 20, 2010 at 12:16 AM, Stefano Sabatini <
> stefano.sabatini-lala at poste.it> wrote:
> 
> > On date Thursday 2010-08-19 23:55:40 +0800, zhentan feng encoded:
> > [...]
> >
> > > +    snprintf(headers, sizeof(headers),
> > > +        "Accept: */*\r\n"
> > > +        USERAGENT
> > > +        "Host: %s:%d\r\n"
> > > +        "Pragma: no-cache,rate=1.000000,stream-time=0,"
> > > +        "stream-offset=0:0,request-context=%u,max-duration=0\r\n"
> > > +        CLIENTGUID
> > > +        "Connection: Close\r\n\r\n",
> > > +        host, port, mmsh->request_seq++);
> >
> > ditto
> >
> what's the problem with this item? The indent is 4 characters.
> I split into several lines for more readable as HTTP message type.

I mean:

    snprintf(headers, sizeof(headers),
             "Accept: */*\r\n"
             USERAGENT
             "Host: %s:%d\r\n"
             "Pragma: no-cache,rate=1.000000,stream-time=0,"
             "stream-offset=0:0,request-context=%u,max-duration=0\r\n"
             CLIENTGUID
             "Connection: Close\r\n\r\n",
             host, port, mmsh->request_seq++);

that is following params on the following lines are aligned to the
next character after "(".
  
> btw:I try to find the rule to follow according to ffprobe.c code style,
> but I am still not clear what's the NOT weird indent?
> could you explain it more?

Regards.
-- 
FFmpeg = Fascinating & Funny Mastodontic Puristic Explosive God