This is the mail archive of the newlib@sourceware.org mailing list for the newlib project.
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |
Other format: | [Raw text] |
Here is a patch that does what Sebastian requested. I did not include the regenerated Makefile.in, leaving it to Jeff to do that if it makes the cut to be put in. (I did note the regeneration in the ChangeLog.) Craig -----Original Message----- From: newlib-owner@sourceware.org [mailto:newlib-owner@sourceware.org] On Behalf Of Joel Sherrill Sent: Thursday, February 12, 2009 10:00 AM To: Sebastian Huber Cc: newlib@sources.redhat.com Subject: Re: Split ttyname.c to avoid unnecessary dependencies Sebastian Huber wrote: > Joel Sherrill wrote: > >> Sebastian Huber wrote: >> >>> Hi, >>> >>> the two functions ttyname() and ttyname_r() are currently defined in >>> one file >>> >>> libc/unix/ttyname.c >>> >>> The ttyname() function needs 1030 bytes of data (with common _PATH_DEV >>> and >>> MAXNAMLEN values). This is quite much if you have a system with a >>> small memory >>> space. The file should be split up into ttyname.c and ttyname_r.c to >>> avoid >>> this memory requirement if you only use ttyname_r(). >>> >>> >>> >> I don't disagree that this is a good idea but neither method >> shows up in the RTEMS minimum, hello, or ticker sample. >> What in RTEMS is pulling it into your application? >> > > It is the telnet demon. > > OK. That is pretty far above a minimum configuration but unused code is unused code. It should be split. Can you submit a patch? -- Joel Sherrill, Ph.D. Director of Research & Development joel.sherrill@OARcorp.com On-Line Applications Research Ask me about RTEMS: a free RTOS Huntsville AL 35805 Support Available (256) 722-9985
Attachment:
ttyname.patch
Description: ttyname.patch
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |