Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 25 Aug 2005 04:09:52 +0100 (BST)
From:      Robert Watson <rwatson@FreeBSD.org>
To:        Ben Kaduk <minimarmot@gmail.com>
Cc:        cvs-src@freebsd.org, src-committers@freebsd.org, cvs-all@freebsd.org
Subject:   Re: cvs commit: src/sys/net if_var.h
Message-ID:  <20050825040704.C11335@fledge.watson.org>
In-Reply-To: <47d0403c05082419435293e9d4@mail.gmail.com>
References:  <200508241326.j7ODQFeU043971@repoman.freebsd.org> <47d0403c05082419435293e9d4@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help

On Thu, 25 Aug 2005, Ben Kaduk wrote:

> On 8/24/05, Robert Watson <rwatson@freebsd.org> wrote:
>> rwatson     2005-08-24 13:26:15 UTC
>>
>>   FreeBSD src repository
>>
>>   Modified files:        (Branch: RELENG_6)
>>     sys/net              if_var.h
>>   Log:
>>   Merge if_var.h:1.101 from HEAD to RELENG_6:
>>
>>     - Rename ifmaof_ifpforaddr() to if_findmulti(); assert if_addr_mtx.
>>       Staticize.
>>
>>     Problem reported by:    Ed Maste <emaste at phaedrus dot sandvine dot ca>
>>
>>   Approved by:    re (scottl)
>
> Pardon my ignorance, but how can removing a single line and not adding 
> anything result in a rename operation?

This is actually a second part of a larger commit made earlier in the day, 
but missed in the first pass -- take a look at if.c:1.234.2.6.  The 
function was renamed and staticized, so the prototype in if_var.h was no 
longer needed.  The commit message is a bit broad as a result (only the 
staticization is relevant).

Robert N M Watson



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20050825040704.C11335>