Page MenuHomeFreeBSD

libc/iconv: don't export nonexistant symbols
ClosedPublic

Authored by brooks on Mar 1 2024, 11:55 PM.
Tags
None
Referenced Files
F102576196: D44183.diff
Thu, Nov 14, 6:43 AM
Unknown Object (File)
Sat, Nov 9, 6:11 PM
Unknown Object (File)
Sat, Nov 9, 6:10 PM
Unknown Object (File)
Sat, Nov 9, 6:10 PM
Unknown Object (File)
Wed, Nov 6, 8:26 AM
Unknown Object (File)
Wed, Oct 16, 3:09 PM
Unknown Object (File)
Oct 14 2024, 6:17 PM
Unknown Object (File)
Oct 13 2024, 12:59 AM
Subscribers

Details

Summary

It's unclear to me that any of these symbols ever existed. The ones
I've spot checked are only mentioned in the initial Citrus iconv import
(commit ad30f8e79bd1) and this code hasn't changed much over time.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 56392
Build 53280: arc lint + arc unit

Event Timeline

lib/libc/iconv/Symbol.map
40

At least this one...

62

... and this set, or so, should be used by various lib/libiconv_modules through a lot of macro shenanigans ("indirection")... do you still end up with a functional libiconv?

ah, I see; some of these were inlined in a header, some of them left un-exported in the first place... someone seemingly applied a relatively large hammer.

This revision is now accepted and ready to land.Mar 7 2024, 2:39 AM
This revision was automatically updated to reflect the committed changes.