On 2014-09-25 16:48, Frans Klaver wrote:
On Thu, Sep 25, 2014 at 3:41 PM,  <[email protected]> wrote:
Subject: [PATCH 1/1] Changed "&" with "and" in Documentation/applying-patches.txt.

Documentation/SubmittingPatches says:

Describe your changes in imperative mood, e.g. "make xyzzy do frotz"
instead of "[This patch] makes xyzzy do frotz" or "[I] changed xyzzy
to do frotz", as if you are giving orders to the codebase to change
its behaviour.

Thank you for your feedback Frans.

So for example a better description would have been: "use 'and' instead of '&'"?


From: Oscar Utbult <[email protected]>

http://english.stackexchange.com/questions/3127/when-to-use-instead-of-and

It might be that a link is enough, but I would prefer to have an
excerpt from that, possibly with the link as reference.

You are right. I will provide one in the future.

Oscar


Thanks,
Frans


Signed-off-by: Oscar Utbult <[email protected]>
---
 Documentation/applying-patches.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/applying-patches.txt b/Documentation/applying-patches.txt
index a083ba3..77df55b 100644
--- a/Documentation/applying-patches.txt
+++ b/Documentation/applying-patches.txt
@@ -94,7 +94,7 @@ Common errors when patching
 ---
When patch applies a patch file it attempts to verify the sanity of the
 file in different ways.
-Checking that the file looks like a valid patch file & checking the code +Checking that the file looks like a valid patch file and checking the code around the bits being modified matches the context provided in the patch are
 just two of the basic sanity checks patch does.

--
1.8.0.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to