[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <5d83f043-ff60-94f3-b3a7-e72876ce8dd2@gmail.com>
Date: Fri, 2 Oct 2020 13:44:59 +0200
From: "Michael Kerrisk (man-pages)" <mtk.manpages@...il.com>
To: Alejandro Colomar <colomar.6.4.3@...il.com>
Cc: mtk.manpages@...il.com, Paul Eggert <eggert@...ucla.edu>,
linux-man@...r.kernel.org, libc-alpha@...rceware.org,
gcc@....gnu.org,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2 1/2] system_data_types.7: Add 'void *'
Hi Alex,
On 10/2/20 10:48 AM, Alejandro Colomar wrote:
> Hi Michael,
>
> On 2020-10-02 10:24, Alejandro Colomar wrote:
>> On 2020-10-01 19:32, Paul Eggert wrote:
>> > For 'void *' you should also mention that one cannot use arithmetic on
>> > void * pointers, so they're special in that way too.
>>
>> Good suggestion!
>>
>> > Also, you should
>> > warn that because one can convert from any pointer type to void * and
>> > then to any other pointer type, it's a deliberate hole in C's
>> > type-checking.
>>
>> Also good. I'll talk about generic function parameters for this.
> I think the patch as is now is complete enough to be added.
>
> So I won't rewrite it for now.
> Please review the patch as is,
> and I'll add more info to this type in the future.
Actually, I would rather prefer one patch series, rather than
patches on patches please. It also makes review of the overall
'void *' text easier if it's all one patch. So, If you could
squash the patches together and resubmit, that would be helful.
Thanks,
Michael
--
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
Powered by blists - more mailing lists