Re: [documentation patch] add detail to const and pure attributes

2018-04-03 Thread Jason Merrill
On Mon, Apr 2, 2018 at 6:34 PM, Martin Sebor wrote: > Attached is a changed patch that uses "cannot have side effects" > instead of "can have no side effects." > +effects it does not make sense for such a function to return @code{void}. > +Declaring such functions is diagnosed. Let's make the la

Re: [documentation patch] add detail to const and pure attributes

2018-04-03 Thread Martin Sebor
On 04/03/2018 03:27 AM, Pedro Alves wrote: On 04/02/2018 11:34 PM, Martin Sebor wrote: On 04/02/2018 12:09 AM, Sandra Loosemore wrote: On 03/27/2018 03:21 PM, Pedro Alves wrote: On 03/27/2018 09:19 PM, Martin Sebor wrote: On 03/27/2018 01:38 PM, Pedro Alves wrote: On 03/27/2018 07:18 PM, Mar

Re: [documentation patch] add detail to const and pure attributes

2018-04-03 Thread Pedro Alves
On 04/02/2018 11:34 PM, Martin Sebor wrote: > On 04/02/2018 12:09 AM, Sandra Loosemore wrote: >> On 03/27/2018 03:21 PM, Pedro Alves wrote: >>> On 03/27/2018 09:19 PM, Martin Sebor wrote: On 03/27/2018 01:38 PM, Pedro Alves wrote: > On 03/27/2018 07:18 PM, Martin Sebor wrote: >> +Becau

Re: [documentation patch] add detail to const and pure attributes

2018-04-02 Thread Martin Sebor
On 04/02/2018 12:09 AM, Sandra Loosemore wrote: On 03/27/2018 03:21 PM, Pedro Alves wrote: On 03/27/2018 09:19 PM, Martin Sebor wrote: On 03/27/2018 01:38 PM, Pedro Alves wrote: On 03/27/2018 07:18 PM, Martin Sebor wrote: +Because a @code{pure} function can have no side-effects it does not

Re: [documentation patch] add detail to const and pure attributes

2018-04-01 Thread Sandra Loosemore
On 03/27/2018 03:21 PM, Pedro Alves wrote: On 03/27/2018 09:19 PM, Martin Sebor wrote: On 03/27/2018 01:38 PM, Pedro Alves wrote: On 03/27/2018 07:18 PM, Martin Sebor wrote: +Because a @code{pure} function can have no side-effects it does not FWIW, I'd suggest rephrasing as:  Because a @co

Re: [documentation patch] add detail to const and pure attributes

2018-03-27 Thread Pedro Alves
On 03/27/2018 09:19 PM, Martin Sebor wrote: > On 03/27/2018 01:38 PM, Pedro Alves wrote: >> On 03/27/2018 07:18 PM, Martin Sebor wrote: >>> +Because a @code{pure} function can have no side-effects it does not >> >> FWIW, I'd suggest rephrasing as: >> >>  Because a @code{pure} function cannot have s

Re: [documentation patch] add detail to const and pure attributes

2018-03-27 Thread Martin Sebor
On 03/27/2018 01:38 PM, Pedro Alves wrote: On 03/27/2018 07:18 PM, Martin Sebor wrote: +Because a @code{pure} function can have no side-effects it does not FWIW, I'd suggest rephrasing as: Because a @code{pure} function cannot have side effects because "can have no side-effects" can be read

Re: [documentation patch] add detail to const and pure attributes

2018-03-27 Thread Pedro Alves
On 03/27/2018 07:18 PM, Martin Sebor wrote: > +Because a @code{pure} function can have no side-effects it does not FWIW, I'd suggest rephrasing as: Because a @code{pure} function cannot have side effects because "can have no side-effects" can be read as "is allowed to have no side effects", whi