mbedtls icon indicating copy to clipboard operation
mbedtls copied to clipboard

Make local functions and objects static

Open casaroli opened this issue 1 year ago • 1 comments

Description

In NuttX flat build, all objects are linked together, so that they end up in a flat namespace.

If ssl_server2 and ssl_client2 are linked together, we get some multiple definitions of these functions.

It should not hurt the current use cases if set these to static.

Please accept this PR as part of a (hopefully short) series to make mbedtls more cleanly integrated in NuttX.

PR checklist

Please tick as appropriate and edit the reasons (e.g.: "backport: not needed because this is a new feature")

  • [x] changelog provided, or not required
  • [x] 3.6 backport done, or not required
  • [x] 2.28 backport done, or not required
  • [x] tests provided, or not required

Notes for the submitter

Please refer to the contributing guidelines, especially the checklist for PR contributors.

Help make review efficient:

  • Multiple simple commits
    • please structure your PR into a series of small commits, each of which does one thing
  • Avoid force-push
    • please do not force-push to update your PR - just add new commit(s)
  • See our Guidelines for Contributors for more details about the review process.

casaroli avatar Jun 02 '24 14:06 casaroli

Hey @casaroli,

just yesterday I pushed some commits to the PR #9189, walking over the same code files and making a lot of stuff static. Could you please checkout and build the code and see if this fixes your issues as well?

misch7 avatar Jun 02 '24 18:06 misch7