Thread: A very small typo in the comment
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
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. +
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. +