Fix for grammatical error in code-comment

Started by Emil Igglandalmost 9 years ago2 messages
#1Emil Iggland
emil@iggland.com
1 attachment(s)

I found a grammatical error in one of the code comments.

The comment is made regarding the run-time of the algorithm, and
references big-O performance.
However the comment text uses "effective", instead of the probably more
correct "efficient".

The original comment reads as though the code would not do its job
correctly, rather than just not very quickly.

Regards
Emil

Attachments:

grammar_fix.patchtext/plain; charset=UTF-8; name=grammar_fix.patchDownload
--- src/backend/commands/publicationcmds.c.orig	2017-03-22 14:24:10.838593000 +0100
+++ src/backend/commands/publicationcmds.c	2017-03-22 14:25:14.410045000 +0100
@@ -525,7 +525,7 @@
 		myrelid = RelationGetRelid(rel);
 		/*
 		 * filter out duplicates when user specifies "foo, foo"
-		 * Note that this algorithm is know to not be very effective (O(N^2))
+		 * Note that this algorithm is known to not be very efficient (O(N^2))
 		 * but given that it only works on list of tables given to us by user
 		 * it's deemed acceptable.
 		 */
#2Magnus Hagander
magnus@hagander.net
In reply to: Emil Iggland (#1)
Re: Fix for grammatical error in code-comment

On Wed, Mar 22, 2017 at 2:47 PM, Emil Iggland <emil@iggland.com> wrote:

I found a grammatical error in one of the code comments.

The comment is made regarding the run-time of the algorithm, and
references big-O performance.
However the comment text uses "effective", instead of the probably more
correct "efficient".

The original comment reads as though the code would not do its job
correctly, rather than just not very quickly.

Applied, thanks.

--
Magnus Hagander
Me: http://www.hagander.net/
Work: http://www.redpill-linpro.com/