public inbox for libc-help@sourceware.org
 help / color / mirror / Atom feed
* include/stdio.h
@ 2021-11-08 21:45 Jonny Grant
  2021-11-08 22:27 ` include/stdio.h Florian Weimer
  0 siblings, 1 reply; 4+ messages in thread
From: Jonny Grant @ 2021-11-08 21:45 UTC (permalink / raw)
  To: Szabolcs Nagy via Libc-help

Hello

May I check, I'd not expected /usr/include/stdio.h to state this, is it to state stdio is built on top of C++ iostreams nowadays?

In the repo it is /glibc/libio/stdio.h

"Define ISO C stdio on top of C++ iostreams"

Jonny

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: include/stdio.h
  2021-11-08 21:45 include/stdio.h Jonny Grant
@ 2021-11-08 22:27 ` Florian Weimer
  2021-11-09 10:22   ` include/stdio.h Jonny Grant
  2023-10-28 22:43   ` include/stdio.h Jonny Grant
  0 siblings, 2 replies; 4+ messages in thread
From: Florian Weimer @ 2021-11-08 22:27 UTC (permalink / raw)
  To: Jonny Grant; +Cc: Szabolcs Nagy via Libc-help

* Jonny Grant:

> May I check, I'd not expected /usr/include/stdio.h to state this, is it to state stdio is built on top of C++ iostreams nowadays?
>
> In the repo it is /glibc/libio/stdio.h
>
> "Define ISO C stdio on top of C++ iostreams"

Internally, it's still built this way.  But the C++ meant here is the
C++ ABI implemented by GCC 2.95, so that aspect is now entirely useless.
We should probably remove that C++ reference from the public header.

Thanks,
Florian


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: include/stdio.h
  2021-11-08 22:27 ` include/stdio.h Florian Weimer
@ 2021-11-09 10:22   ` Jonny Grant
  2023-10-28 22:43   ` include/stdio.h Jonny Grant
  1 sibling, 0 replies; 4+ messages in thread
From: Jonny Grant @ 2021-11-09 10:22 UTC (permalink / raw)
  To: Florian Weimer; +Cc: libc-help



On 08/11/2021 22:27, Florian Weimer wrote:
> * Jonny Grant:
> 
>> May I check, I'd not expected /usr/include/stdio.h to state this, is it to state stdio is built on top of C++ iostreams nowadays?
>>
>> In the repo it is /glibc/libio/stdio.h
>>
>> "Define ISO C stdio on top of C++ iostreams"
> 
> Internally, it's still built this way.  But the C++ meant here is the
> C++ ABI implemented by GCC 2.95, so that aspect is now entirely useless.
> We should probably remove that C++ reference from the public header.
> 
> Thanks,
> Florian
> 

Hi Florian

Maybe just remove the line and follow "string.h" text?

/*
 *	ISO C99 Standard: Standard Input Output <stdio.h>
 */

Cheers
Jonny

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: include/stdio.h
  2021-11-08 22:27 ` include/stdio.h Florian Weimer
  2021-11-09 10:22   ` include/stdio.h Jonny Grant
@ 2023-10-28 22:43   ` Jonny Grant
  1 sibling, 0 replies; 4+ messages in thread
From: Jonny Grant @ 2023-10-28 22:43 UTC (permalink / raw)
  To: Florian Weimer; +Cc: Szabolcs Nagy via Libc-help



On 08/11/2021 22:27, Florian Weimer wrote:
> * Jonny Grant:
> 
>> May I check, I'd not expected /usr/include/stdio.h to state this, is it to state stdio is built on top of C++ iostreams nowadays?
>>
>> In the repo it is /glibc/libio/stdio.h
>>
>> "Define ISO C stdio on top of C++ iostreams"
> 
> Internally, it's still built this way.  But the C++ meant here is the
> C++ ABI implemented by GCC 2.95, so that aspect is now entirely useless.
> We should probably remove that C++ reference from the public header.
> 
> Thanks,
> Florian
> 

I emailed a patch to libc-alpha to take out that C++ reference from the header just now.
Kind regards, Jonny

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2023-10-28 22:43 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-08 21:45 include/stdio.h Jonny Grant
2021-11-08 22:27 ` include/stdio.h Florian Weimer
2021-11-09 10:22   ` include/stdio.h Jonny Grant
2023-10-28 22:43   ` include/stdio.h Jonny Grant

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).