From owner-cvs-all@FreeBSD.ORG Tue Oct 25 10:16:23 2005 Return-Path: X-Original-To: cvs-all@freebsd.org Delivered-To: cvs-all@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id B5C6116A420 for ; Tue, 25 Oct 2005 10:16:23 +0000 (GMT) (envelope-from rik@cronyx.ru) Received: from hanoi.cronyx.ru (hanoi.cronyx.ru [144.206.181.53]) by mx1.FreeBSD.org (Postfix) with ESMTP id C501843D4C for ; Tue, 25 Oct 2005 10:16:21 +0000 (GMT) (envelope-from rik@cronyx.ru) Received: (from root@localhost) by hanoi.cronyx.ru (8.13.0/vak/3.0) id j9PADGYC080711 for cvs-all@freebsd.org.checked; Tue, 25 Oct 2005 14:13:16 +0400 (MSD) (envelope-from rik@cronyx.ru) Received: from [144.206.181.94] (hi.cronyx.ru [144.206.181.94]) by hanoi.cronyx.ru (8.13.0/vak/3.0) with ESMTP id j9PAAgo7080662; Tue, 25 Oct 2005 14:10:43 +0400 (MSD) (envelope-from rik@cronyx.ru) Message-ID: <435E04AA.3080605@cronyx.ru> Date: Tue, 25 Oct 2005 14:10:50 +0400 From: Roman Kurakin User-Agent: Mozilla Thunderbird 0.9 (Windows/20041103) X-Accept-Language: en-us, en MIME-Version: 1.0 To: obrien@freebsd.org References: <200509271657.j8RGvj2n015326@repoman.freebsd.org> <20051024072423.GA34909@dragon.NUXI.org> <200510241034.27073.jhb@freebsd.org> <20051025052018.GE55418@dragon.NUXI.org> In-Reply-To: <20051025052018.GE55418@dragon.NUXI.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: Roman Kurakin , cvs-src@freebsd.org, src-committers@freebsd.org, cvs-all@freebsd.org, John Baldwin Subject: Re: cvs commit: src/sys/dev/cp if_cp.c X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 25 Oct 2005 10:16:23 -0000 David O'Brien wrote: >On Mon, Oct 24, 2005 at 10:34:25AM -0400, John Baldwin wrote: > > >>On Monday 24 October 2005 03:24 am, David O'Brien wrote: >> >> >>>On Tue, Sep 27, 2005 at 04:57:45PM +0000, Roman Kurakin wrote: >>> >>> >>>>rik 2005-09-27 16:57:45 UTC >>>> FreeBSD src repository >>>> Modified files: >>>> sys/dev/cp if_cp.c >>>> Log: >>>> Restore if_cp.c 1.27 >>>> >>>> >... > > >>>You should not have backed out my commit without discussing it with me >>>and understanding the reason for the change. >>>Do it again and I *will* be taking it Core. >>> >>> >>Looks like he added some function prototypes and moved the cdevsw up. Does i >>compile now with gcc 4.0? It seems that his changes were a lot simpler and >>didn't destroy nearly as much CVS history as your changes. It would really >>be preferable to use simpler solutions rather than destroying version history >>with really big diffs. >> >> > >Doesn't matter -- it was a clear back out of my recent commit. >src/MAINTAINERS doesn't list any of these drivers, so what was his > > I do not want to list them in src/MAINTAINERS cause I think it is better to allow any commits. Cases such this one is rare. Since I didn't get reply from you within a week and I do not like function relocation and didn't find explanation from commit log I've backout function relocation, not fix of forward variable declaration. The commit log didn't state what was the problem variable forward declaration or/and function forward declaration. rik >authority in unilaterally backing out my commit? >It is also port portable to define static functions early in a file, >before they are used. > > >