Documentation for varbit is missing size parameter

2018-03-10 Thread PG Doc comments form
The following documentation comment has been logged on the website: Page: https://www.postgresql.org/docs/10/static/datatype.html Description: The documentation for the varbit data type is missing the size parameter "[ (n) ]".

Re: Documentation for varbit is missing size parameter

2018-03-15 Thread Euler Taveira
2018-03-11 1:43 GMT-03:00 PG Doc comments form : > The documentation for the varbit data type is missing the size parameter "[ > (n) ]". > Good catch! It seems to be an oversight in commit 768b647ead78d0d63915c1708cad13c2468f9440. The attached patch adds it. -- Euler Taveira

Re: Documentation for varbit is missing size parameter

2018-03-30 Thread Bruce Momjian
On Fri, Mar 16, 2018 at 01:17:04AM -0300, Euler Taveira wrote: > 2018-03-11 1:43 GMT-03:00 PG Doc comments form : > > The documentation for the varbit data type is missing the size parameter "[ > > (n) ]". > > > Good catch! It seems to be an oversight in commit > 768b647ead78d0d63915c1708cad13c2468

Re: Documentation for varbit is missing size parameter

2018-03-30 Thread Magnus Hagander
On Fri, Mar 30, 2018 at 5:18 PM, Bruce Momjian wrote: > On Fri, Mar 16, 2018 at 01:17:04AM -0300, Euler Taveira wrote: > > 2018-03-11 1:43 GMT-03:00 PG Doc comments form : > > > The documentation for the varbit data type is missing the size > parameter "[ > > > (n) ]". > > > > > Good catch! It se

Re: Documentation for varbit is missing size parameter

2018-03-30 Thread Bruce Momjian
On Fri, Mar 30, 2018 at 05:39:46PM +0200, Magnus Hagander wrote: > On Fri, Mar 30, 2018 at 5:18 PM, Bruce Momjian wrote: > > On Fri, Mar 16, 2018 at 01:17:04AM -0300, Euler Taveira wrote: > > 2018-03-11 1:43 GMT-03:00 PG Doc comments form : > > > The documentation for the varbit data

Re: Documentation for varbit is missing size parameter

2018-03-30 Thread Magnus Hagander
On Fri, Mar 30, 2018 at 6:26 PM, Bruce Momjian wrote: > On Fri, Mar 30, 2018 at 05:39:46PM +0200, Magnus Hagander wrote: > > On Fri, Mar 30, 2018 at 5:18 PM, Bruce Momjian wrote: > > > > On Fri, Mar 16, 2018 at 01:17:04AM -0300, Euler Taveira wrote: > > > 2018-03-11 1:43 GMT-03:00 PG Doc

Re: Documentation for varbit is missing size parameter

2018-03-30 Thread Bruce Momjian
On Fri, Mar 30, 2018 at 06:28:15PM +0200, Magnus Hagander wrote: > On Fri, Mar 30, 2018 at 6:26 PM, Bruce Momjian wrote: > Uh, I am always debating how important it is to backpatck vs the churn > we require of translations of our docs.  In this case, it didn't seem > worthwhile to have

Re: Documentation for varbit is missing size parameter

2018-03-30 Thread David G. Johnston
On Friday, March 30, 2018, Bruce Momjian wrote: > > I looked at this patch and thought it was more a stylistic item rather > than a correction. Is that right? > Looks like a syntax bug fix to me - especially since it is a reference table. Those should be back-patched. David J.

Re: Documentation for varbit is missing size parameter

2018-03-30 Thread Bruce Momjian
On Fri, Mar 30, 2018 at 10:31:07AM -0700, David G. Johnston wrote: > On Friday, March 30, 2018, Bruce Momjian wrote: > > I looked at this patch and thought it was more a stylistic item rather > than a correction.  Is that right? > > > Looks like a syntax bug fix to me - especially since

Re: Documentation for varbit is missing size parameter

2018-04-01 Thread Magnus Hagander
On Fri, Mar 30, 2018 at 7:34 PM, Bruce Momjian wrote: > On Fri, Mar 30, 2018 at 10:31:07AM -0700, David G. Johnston wrote: > > On Friday, March 30, 2018, Bruce Momjian wrote: > > > > I looked at this patch and thought it was more a stylistic item > rather > > than a correction. Is that