Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 28 Apr 2004 11:06:13 -0400
From:      John Baldwin <jhb@FreeBSD.org>
To:        Ruslan Ermilov <ru@FreeBSD.org>
Cc:        cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/boot/i386/boot2 boot1.S sio.S src/sys/boot/i386/btx/btx btx.S src/sys/boot/i386/btx/btxldr btxldr.S src/sys/boot/i386/libi386 amd64_tramp.S src/sys/boot/i386/pxeldr pxeldr.S
Message-ID:  <200404281106.13952.jhb@FreeBSD.org>
In-Reply-To: <200404281431.i3SEVjtb016009@repoman.freebsd.org>
References:  <200404281431.i3SEVjtb016009@repoman.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Wednesday 28 April 2004 10:31 am, Ruslan Ermilov wrote:
> ru          2004/04/28 07:31:44 PDT
>
>   FreeBSD src repository
>
>   Modified files:
>     sys/boot/i386/boot2  boot1.S sio.S
>     sys/boot/i386/btx/btx btx.S
>     sys/boot/i386/btx/btxldr btxldr.S
>     sys/boot/i386/libi386 amd64_tramp.S
>     sys/boot/i386/pxeldr pxeldr.S
>   Log:
>   After talking to Bruce Evans and reading more standards specs,
>   switch to using C99-style comments everywhere in preprocessed
>   assembler.  The reason is that lines starting with the regexp
>   '^[[:space:]]#' are treated as preprocessing directives, and
>   while it seems to work now with GCC, it's not necessarily has
>   to work.  Use C99 comments `//' for the trailing comments to
>   save whitespace.

Well, that just killed any local diff's anyone had to the boot code for a 
trivial reason.  as(1) worked just fine, I don't see why it's such a cardinal 
sin to use as(1) to compile asm files.

Thanks for not even asking before hosing the history in cvs annotate:

> grep jhb /usr/src/MAINTAINERS
cdboot          jhb     Pre-commit review requested.
pxeboot         jhb     Pre-commit review requested.
witness         jhb     Pre-commit review requested.
sys/boot/i386/cdboot/Makefile:MAINTAINER=       jhb@FreeBSD.org
sys/boot/i386/pxeldr/Makefile:MAINTAINER=       jhb@FreeBSD.org

-- 
John Baldwin <jhb@FreeBSD.org>  <><  http://www.FreeBSD.org/~jhb/
"Power Users Use the Power to Serve"  =  http://www.FreeBSD.org



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