[MPlayer-dev-eng] RN5 authentication for rtsp

The Wanderer inverseparadox at comcast.net
Thu Mar 27 01:05:04 CET 2008


Paul wrote:

> On Wed, 26 Mar 2008, Nico Sabbi wrote:

>> Il Wednesday 26 March 2008 19:40:38 Paul ha scritto:
>>> 
>>> Here's an updated patch with some bound checking on strings and
>>> proper gargabe collection. There are no cosmetic changes
>> 
>> 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.

The extra indentation is necessary, yes, but it makes it harder to tell
which lines have actually changed and which lines have merely been
reindented, and so makes it harder to review the patch (or see what has
changed, later, when looking at the commit log). Because of that, the
standard around these parts is that all such cosmetic changes must be
made in separate patches, to be applied after the patch which makes the
functional changes.

-- 
    The Wanderer

    My usual .sig is on vacation while I adjust to my new computer



More information about the MPlayer-dev-eng mailing list