Re: Internal visibility

[Ludovic Courtès]

Hi,

address@hidden (Ludovic Courtès) writes:

> Your enthusiasm gave me an incentive to go ahead and try it out (see
> attached patch against `master').  This brings the number of exported
> `scm_i_' symbols (as seen with "objdump -T") down from 195 to 68 here,
> which should slightly reduce load time since it gives less work to the
> loader.  The patch also marks the `scm_init' functions as internal.  It
> uses "internal" visibility, not just "hidden" (see "Function Attributes"
> node of the GCC manual).
>
> A few functions had to be left public: those used in public macros or
> inlines, some functions of `numbers.c' that are used in `srfi-60', and
> `scm_i_string_{writable_,}chars ()' functions which have been used
> outside (e.g., in G-Wrap, IIRC) as they're too convenient...
>
> I know Guile-VM uses a few internal functions/macros, so Andy will have
> to check whether this breaks anything (and propose a new public API if
> it does :-)).  It would be great to double-check with other code that
> uses Guile.  Then if that looks acceptable, we can commit it, including
> to the 1.8 branch.

I'd like to push the patch within the next few days if nobody complains,
alright?

Thanks,
Ludovic.