Thread: A very small typo in the comment

A very small typo in the comment

From
"孟庆钟"
Date:
Hi,
      I am a student in China. I have read some source code of postgreSQL, though only a little. I think maybe there is typo in this file: src\backend\catalog\index.c. At line 649 in v9.2.0, line 653 in V9.2.2. The sentence is " *indexInfo: same info executor uses to insert into the index", I think it should not be 'same', but 'some'.
      I am not sure if my thoughts is right.
      Sorry for my poor English.

      Good Luck From Beijing!

      Meng Qingzhong

Re: A very small typo in the comment

From
Bruce Momjian
Date:
On Sat, Jan  5, 2013 at 12:13:27AM +0800, 孟庆钟 wrote:
> Hi,
>       I am a student in China. I have read some source code of postgreSQL,
> though only a little. I think maybe there is typo in this file: src\backend\
> catalog\index.c. At line 649 in v9.2.0, line 653 in V9.2.2. The sentence is "
> *indexInfo: same info executor uses to insert into the index", I think it
> should not be 'same', but 'some'.
>       I am not sure if my thoughts is right.
>       Sorry for my poor English.

You are correct.  I have fixed it in the PG 9.3 source tree.  Thanks.

--  Bruce Momjian  <bruce@momjian.us>        http://momjian.us EnterpriseDB
http://enterprisedb.com
 + It's impossible for everything to be true. +



Re: A very small typo in the comment

From
Bruce Momjian
Date:
On Mon, Jan  7, 2013 at 07:46:46PM -0500, Bruce Momjian wrote:
> On Sat, Jan  5, 2013 at 12:13:27AM +0800, 孟庆钟 wrote:
> > Hi,
> >       I am a student in China. I have read some source code of postgreSQL,
> > though only a little. I think maybe there is typo in this file: src\backend\
> > catalog\index.c. At line 649 in v9.2.0, line 653 in V9.2.2. The sentence is "
> > *indexInfo: same info executor uses to insert into the index", I think it
> > should not be 'same', but 'some'.
> >       I am not sure if my thoughts is right.
> >       Sorry for my poor English.
> 
> You are correct.  I have fixed it in the PG 9.3 source tree.  Thanks.

Oh, I see Tom's later comment now.  The change was not made to 9.3.

--  Bruce Momjian  <bruce@momjian.us>        http://momjian.us EnterpriseDB
http://enterprisedb.com
 + It's impossible for everything to be true. +