From: Stephen Amell <mrstephenamell@gmail.com>
To: pgsql-hackers@postgresql.org
Subject: Quitting the thes
Date: Mon, 13 May 2019 16:52:15 -0300
Message-ID: <539fa271-21b3-777e-a468-d96cffe9c768@gmail.com> (raw)
Hi all,
Here my first patch for postgres. Starting by an easy thing, I correct
the duplicated "the the" strings from comments on some files.
- src/backend/executor/execExpr.c
- src/include/c.h
- src/include/jit/llvmjit_emit.h
- src/include/nodes/execnodes.h
- src/include/replication/logical.h
Any feedback are welcome!
Thanks a lot,
diff --git a/src/backend/executor/execExpr.c b/src/backend/executor/execExpr.cindex 0fb31f5..0a7b2b8 100644--- a/src/backend/executor/execExpr.c+++ b/src/backend/executor/execExpr.c@@ -2361,7 +2361,7 @@ get_last_attnums_walker(Node *node, LastAttnumInfo *info)
* Compute additional information for EEOP_*_FETCHSOME ops.
*
* The goal is to determine whether a slot is 'fixed', that is, every
- * evaluation of the the expression will have the same type of slot, with an+ * evaluation of the expression will have the same type of slot, with an
* equivalent descriptor.
*/
static void
diff --git a/src/include/c.h b/src/include/c.hindex 33c9518..0f89c02 100644--- a/src/include/c.h+++ b/src/include/c.h@@ -1129,7 +1129,7 @@ typedef union PGAlignedXLogBlock
* Please note IT IS NOT SAFE to cast constness away if the result will ever
* be modified (it would be undefined behaviour). Doing so anyway can cause
* compiler misoptimizations or runtime crashes (modifying readonly memory).
- * It is only safe to use when the the result will not be modified, but API+ * It is only safe to use when the result will not be modified, but API
* design or language restrictions prevent you from declaring that
* (e.g. because a function returns both const and non-const variables).
*
diff --git a/src/include/jit/llvmjit_emit.h b/src/include/jit/llvmjit_emit.hindex 9569da6..71a8625 100644--- a/src/include/jit/llvmjit_emit.h+++ b/src/include/jit/llvmjit_emit.h@@ -218,7 +218,7 @@ l_mcxt_switch(LLVMModuleRef mod, LLVMBuilderRef b, LLVMValueRef nc)
}
/*
- * Return pointer to the the argno'th argument nullness.+ * Return pointer to the argno'th argument nullness.
*/
static inline LLVMValueRef
l_funcnullp(LLVMBuilderRef b, LLVMValueRef v_fcinfo, size_t argno)
@@ -236,7 +236,7 @@ l_funcnullp(LLVMBuilderRef b, LLVMValueRef v_fcinfo, size_t argno)
}
/*
- * Return pointer to the the argno'th argument datum.+ * Return pointer to the argno'th argument datum.
*/
static inline LLVMValueRef
l_funcvaluep(LLVMBuilderRef b, LLVMValueRef v_fcinfo, size_t argno)
diff --git a/src/include/nodes/execnodes.h b/src/include/nodes/execnodes.hindex ff33287..4b1635e 100644--- a/src/include/nodes/execnodes.h+++ b/src/include/nodes/execnodes.h@@ -593,7 +593,7 @@ typedef struct EState
* and with which options. es_jit is created on-demand when JITing is
* performed.
*
- * es_jit_combined_instr is the the combined, on demand allocated,+ * es_jit_combined_instr is the combined, on demand allocated,
* instrumentation from all workers. The leader's instrumentation is kept
* separate, and is combined on demand by ExplainPrintJITSummary().
*/
diff --git a/src/include/replication/logical.h b/src/include/replication/logical.hindex 0a2a63a..0daa38c 100644--- a/src/include/replication/logical.h+++ b/src/include/replication/logical.h@@ -47,7 +47,7 @@ typedef struct LogicalDecodingContext
/*
* Marks the logical decoding context as fast forward decoding one. Such a
- * context does not have plugin loaded so most of the the following+ * context does not have plugin loaded so most of the following
* properties are unused.
*/
bool fast_forward;
Attachments:
[text/plain] the-thes.patch (3.2K, ../539fa271-21b3-777e-a468-d96cffe9c768@gmail.com/2-the-thes.patch)
download | inline diff:diff --git a/src/backend/executor/execExpr.c b/src/backend/executor/execExpr.cindex 0fb31f5..0a7b2b8 100644--- a/src/backend/executor/execExpr.c+++ b/src/backend/executor/execExpr.c@@ -2361,7 +2361,7 @@ get_last_attnums_walker(Node *node, LastAttnumInfo *info)
* Compute additional information for EEOP_*_FETCHSOME ops.
*
* The goal is to determine whether a slot is 'fixed', that is, every
- * evaluation of the the expression will have the same type of slot, with an+ * evaluation of the expression will have the same type of slot, with an
* equivalent descriptor.
*/
static void
diff --git a/src/include/c.h b/src/include/c.hindex 33c9518..0f89c02 100644--- a/src/include/c.h+++ b/src/include/c.h@@ -1129,7 +1129,7 @@ typedef union PGAlignedXLogBlock
* Please note IT IS NOT SAFE to cast constness away if the result will ever
* be modified (it would be undefined behaviour). Doing so anyway can cause
* compiler misoptimizations or runtime crashes (modifying readonly memory).
- * It is only safe to use when the the result will not be modified, but API+ * It is only safe to use when the result will not be modified, but API
* design or language restrictions prevent you from declaring that
* (e.g. because a function returns both const and non-const variables).
*
diff --git a/src/include/jit/llvmjit_emit.h b/src/include/jit/llvmjit_emit.hindex 9569da6..71a8625 100644--- a/src/include/jit/llvmjit_emit.h+++ b/src/include/jit/llvmjit_emit.h@@ -218,7 +218,7 @@ l_mcxt_switch(LLVMModuleRef mod, LLVMBuilderRef b, LLVMValueRef nc)
}
/*
- * Return pointer to the the argno'th argument nullness.+ * Return pointer to the argno'th argument nullness.
*/
static inline LLVMValueRef
l_funcnullp(LLVMBuilderRef b, LLVMValueRef v_fcinfo, size_t argno)
@@ -236,7 +236,7 @@ l_funcnullp(LLVMBuilderRef b, LLVMValueRef v_fcinfo, size_t argno)
}
/*
- * Return pointer to the the argno'th argument datum.+ * Return pointer to the argno'th argument datum.
*/
static inline LLVMValueRef
l_funcvaluep(LLVMBuilderRef b, LLVMValueRef v_fcinfo, size_t argno)
diff --git a/src/include/nodes/execnodes.h b/src/include/nodes/execnodes.hindex ff33287..4b1635e 100644--- a/src/include/nodes/execnodes.h+++ b/src/include/nodes/execnodes.h@@ -593,7 +593,7 @@ typedef struct EState
* and with which options. es_jit is created on-demand when JITing is
* performed.
*
- * es_jit_combined_instr is the the combined, on demand allocated,+ * es_jit_combined_instr is the combined, on demand allocated,
* instrumentation from all workers. The leader's instrumentation is kept
* separate, and is combined on demand by ExplainPrintJITSummary().
*/
diff --git a/src/include/replication/logical.h b/src/include/replication/logical.hindex 0a2a63a..0daa38c 100644--- a/src/include/replication/logical.h+++ b/src/include/replication/logical.h@@ -47,7 +47,7 @@ typedef struct LogicalDecodingContext
/*
* Marks the logical decoding context as fast forward decoding one. Such a
- * context does not have plugin loaded so most of the the following+ * context does not have plugin loaded so most of the following
* properties are unused.
*/
bool fast_forward;
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-translators@postgresql.org
Cc: mrstephenamell@gmail.com, pgsql-hackers@postgresql.org
Subject: Re: Quitting the thes
In-Reply-To: <539fa271-21b3-777e-a468-d96cffe9c768@gmail.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox