[PATCH] move cyg_posix_function_start() and _finish() from misc.cxx to pthread.cxx
Andrew Lunn
andrew@lunn.ch
Thu Oct 27 18:59:00 GMT 2005
On Wed, Oct 26, 2005 at 09:22:09PM +0200, Alexander Neundorf wrote:
> Hi,
>
> the functions cyg_posix_function_start() and _finish() from
> compat/posix/current/src/misc.cxx deal with ASR handling and nothing else.
> From checking the eCos code it seems the only two files which use ASRs are
> pthread.cxx and signal.cxx in the POSIX package.
You missed that the fileio package makes use of them.
> So IMO cyg_posix_function_start() and _finish() should be moved to
> pthread.cxx, so that they are only compiled and used if they are actually
> required.
They are currently only used if actually required. The linker will
only pull them into the image if something calls them. So i don't
really see the point of the patch. It does not make the image smaller
and it does not aid maintainability.
Andrew
More information about the Ecos-patches
mailing list