Re: [HACKERS] A typo in mcxt.c
I'm happy to know such a thing. mcxt.c is very stable part of the code so those who don't know such things like me rarely comes. At Thu, 23 Feb 2017 22:59:51 -0500, Tom Lane wrote in <31880.1487908...@sss.pgh.pa.us> > Andres Freund writes: > > On 2017-02-23 14:26:07 -0600, Jim Nasby wrote: > >> On 2/23/17 6:38 AM, Thomas Munro wrote: > >>> That is an archaic way of contracting the same words differently: > > >> Given the number of non-native English speakers we have, it's probably > >> worth > >> changing it... > > > I'm a non-native speaker and I actually like discovering new language > > "features" every now and then. I think as long as it's not inhibiting > > understanding to much - which doesn't seem to be the case here - it's > > fine to keep things like this. > > While I don't recall it specifically, git blame says that comment is mine. > I'm pretty sure it's not a typo, but that the allusion to Hamlet was > intentional. I think it's good to have a bit of levity and external > references in our comments; cuts down on the boredom of reading totally > dry code. > > (But see commit d2783bee3 for one hazard of this sort of thing.) > > regards, tom lane -- Kyotaro Horiguchi NTT Open Source Software Center -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] A typo in mcxt.c
Andres Freund writes: > On 2017-02-23 14:26:07 -0600, Jim Nasby wrote: >> On 2/23/17 6:38 AM, Thomas Munro wrote: >>> That is an archaic way of contracting the same words differently: >> Given the number of non-native English speakers we have, it's probably worth >> changing it... > I'm a non-native speaker and I actually like discovering new language > "features" every now and then. I think as long as it's not inhibiting > understanding to much - which doesn't seem to be the case here - it's > fine to keep things like this. While I don't recall it specifically, git blame says that comment is mine. I'm pretty sure it's not a typo, but that the allusion to Hamlet was intentional. I think it's good to have a bit of levity and external references in our comments; cuts down on the boredom of reading totally dry code. (But see commit d2783bee3 for one hazard of this sort of thing.) regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] A typo in mcxt.c
On 2017-02-23 14:26:07 -0600, Jim Nasby wrote: > On 2/23/17 6:38 AM, Thomas Munro wrote: > > > I'm not so confident, but the "'tis" seems to me to be a typo of > > > "it's". > > That is an archaic way of contracting the same words differently: > > Given the number of non-native English speakers we have, it's probably worth > changing it... I'm a non-native speaker and I actually like discovering new language "features" every now and then. I think as long as it's not inhibiting understanding to much - which doesn't seem to be the case here - it's fine to keep things like this. - Andres -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] A typo in mcxt.c
On 2/23/17 6:38 AM, Thomas Munro wrote: I'm not so confident, but the "'tis" seems to me to be a typo of "it's". That is an archaic way of contracting the same words differently: Given the number of non-native English speakers we have, it's probably worth changing it... -- Jim Nasby, Data Architect, Blue Treble Consulting, Austin TX Experts in Analytics, Data Architecture and PostgreSQL Data in Trouble? Get it in Treble! http://BlueTreble.com 855-TREBLE2 (855-873-2532) -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] A typo in mcxt.c
On Thu, Feb 23, 2017 at 5:40 PM, Kyotaro HORIGUCHI wrote: > Hello, while I investigated a bug of async-exec patch. I found a > (maybe) typo in mcxt.c. > > | * It's not entirely clear whether 'tis better to do this before or after > | * delinking the context; but an error in a callback will likely result in > > I'm not so confident, but the "'tis" seems to me to be a typo of > "it's". That is an archaic way of contracting the same words differently: https://en.wiktionary.org/wiki/'tis It sounds amusing and a tiny bit like a famous line from Hamlet, "Whether 'tis nobler in the mind to suffer...". https://en.wikipedia.org/wiki/To_be,_or_not_to_be -- Thomas Munro http://www.enterprisedb.com -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
[HACKERS] A typo in mcxt.c
Hello, while I investigated a bug of async-exec patch. I found a (maybe) typo in mcxt.c. | * It's not entirely clear whether 'tis better to do this before or after | * delinking the context; but an error in a callback will likely result in I'm not so confident, but the "'tis" seems to me to be a typo of "it's". regards -- Kyotaro Horiguchi NTT Open Source Software Center >From 2eadf76829863e5ebbe7dbfa9b4fae2d48b5110f Mon Sep 17 00:00:00 2001 From: Kyotaro Horiguchi Date: Wed, 22 Feb 2017 09:14:15 +0900 Subject: Fix a typo of mcxt.c --- src/backend/utils/mmgr/mcxt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/backend/utils/mmgr/mcxt.c b/src/backend/utils/mmgr/mcxt.c index 6ad0bb4..2e74e29 100644 --- a/src/backend/utils/mmgr/mcxt.c +++ b/src/backend/utils/mmgr/mcxt.c @@ -208,7 +208,7 @@ MemoryContextDelete(MemoryContext context) MemoryContextDeleteChildren(context); /* - * It's not entirely clear whether 'tis better to do this before or after + * It's not entirely clear whether it's better to do this before or after * delinking the context; but an error in a callback will likely result in * leaking the whole context (if it's not a root context) if we do it * after, so let's do it before. -- 2.9.2 -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers