Date: Fri, 28 Mar 2008 09:35:18 -0700 From: Doug Barton <dougb@FreeBSD.org> To: Roman Divacky <rdivacky@FreeBSD.org> Cc: Pav Lucistnik <pav@FreeBSD.org>, soralx@cydem.org, freebsd-ports@FreeBSD.org Subject: Re: [patch] pkg_delete(1) speedup Message-ID: <47ED1E46.3050409@FreeBSD.org> In-Reply-To: <20080328082241.GA48417@freebsd.org> References: <1206548309.78560.69.camel@pav.hide.vol.cz> <20080326201416.GA34807@freebsd.org> <47EC5589.7070207@FreeBSD.org> <20080328082241.GA48417@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Roman Divacky wrote:
> there are 3 style-only changes in the patch.. 3 lines.. I didnt consider
> to make two separate patches for testing/review :)
>
> I'll do it if you insist....
Think of it as good practice. :) Seriously though, it does make the
CVS history a little easier to examine. Your best bet is to generate a
diff that contains only the style changes, then diff the substantive
changes against the new version. That way if you have to make changes
to the substance later on down the road you won't have to worry about
reapplying the style changes.
hth,
Doug
--
This .signature sanitized for your protection
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?47ED1E46.3050409>