[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] possibly -> possible

From: Hyrum K. Wright <hyrum_wright_at_mail.utexas.edu>
Date: Tue, 7 Apr 2009 08:00:24 -0500

On Apr 7, 2009, at 3:54 AM, Edmund Wong wrote:

> Hi,
>
> I'm following the advice of the hacking.html and
> am reading up on the different header files. Came
> across a minor spelling mistake. I took the liberty
> of making a patch. Since I'm learning, is this
> the correct way of doing things here or am I just
> being too nitpicky? I figured it would help make
> the comments better.

Not too nitpicky at all. A would bet that a nontrivial number of
revisions in our repository are spelling, whitespace, or documentation
tweaks. Having readable docs and help text makes using and fixing the
software much easier in the long run.

That being said, if you run across several of these in the course of
your travels through the code, it's often much easier for you and for
reviewers to include them in one patch, rather than N patches.

Thanks for the hard work!

-Hyrum

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1576997
Received on 2009-04-07 15:00:42 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.