[MPlayer-dev-eng] RN5 authentication for rtsp

Uoti Urpala uoti.urpala at pp1.inet.fi
Wed Mar 26 23:28:55 CET 2008


On Wed, 2008-03-26 at 18:10 -0400, Paul wrote:
> > at a quick glance there are still cosmetics in the initial part of your patch
> 
> I can't see it. The extra indentation is neccessary IMHO.

It is necessary for the final version. However you reindent a fairly
large block of code, and it's not immediately obvious which part is only
supposed to be reindented and which otherwise changed. In this case I
think "diff -b" output is easier to read. Of course it's easy for
everyone to generate diff -b output themselves if they first apply the
patch, but it's harder to read the patch directly from the mail reader.

Attached is "git diff -b" output with the patch applied.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: real.c.patch2.diff-b
Type: text/x-patch
Size: 3205 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20080327/2a9fe47e/attachment.bin>


More information about the MPlayer-dev-eng mailing list