malloc,realloc andfree are referred in many different files. Maybe add them innitpick_ignore? Andcalloc for complete.

malloc,realloc andfree are referred in many different files. Maybe add them innitpick_ignore? Andcalloc for complete.

Yes, I noticed. I'll add them, and I'll remove the_weakref since it does not belong to this PR really.

The most referred functions:

      3 :c:func:`!addstr`      3 :c:func:`calloc`      3 :c:func:`exec`      3 :c:func:`function__entry`      3 :c:func:`printf`      3 :c:func:`sigaction`      3 :c:func:`signal`      3 :c:func:`sprintf`      3 :c:func:`strerror`      4 :c:func:`devpoll`      4 :c:func:`lockf`      4 :c:func:`main`      4 :c:func:`snprintf`      4 :c:func:`stat`      4 :c:func:`system`      4 :c:func:`vsnprintf`      5 :c:func:`dlopen`      5 :c:func:`fcntl`      5 :c:func:`flock`      5 :c:func:`!getopt`      5 :c:func:`!scanf`      6 :c:func:`realloc`      8 :c:func:`select`      9 :c:func:`poll`     11 :c:func:`gmtime`     11 :c:func:`localtime`     18 :c:func:`fork`     21 :c:func:`free`     34 :c:func:`malloc`

Some of them, likegetopt are only referred in one file.

1. I am not sure that all functions referred at least 3 times should be added innitpick_ignore. Many of them are only referred in one file (excluding NEWS entries). Memory management functionsmalloc,realloc andfree are obvious candidates for inclusion.calloc can also be included besides it is only referred 3 times.gmtime andlocaltime are referred several times in two related modules, they also can be included.fork is referred in multiple files.poll is only referred in select.rst, but multiple times, it is questionable. Etc, etc. On other side,addstr is only mentioned in curses.rst, and in all three cases already with!. To me, it is an obvious candidate for not inclusion.function__entry is a strange thing, is it even a standard name?

2. I did not differentiate between C, Posix and Windows types, Posix and Windows environment variables. It is tiresome to find where exactly the name is defined (and some are defined on multiple platforms), and it does not really matter. I only grouped by the role.

It is my thoughts. If you disagree, go ahead, but may be some of them can be useful to you.

Thanks for your input. I think it makes sense to remove functions which are only mentioned a few times, and instead use! to mark them up. I'll return to this later today.

Thanks for the review and insight, Serhiy. Highly appreciated!

Thanks@erlend-aasland for the PR 🌮🎉.. I'm working now to backport this PR to: 3.11, 3.12.
🐍🍒⛏🤖

Sorry,@erlend-aasland, I could not cleanly backport this to3.11 due to a conflict.
Please backport usingcherry_picker on command line.
cherry_picker b447e19e720e6781025432a40eb72b1cc93ac944 3.11

GH-107154 is a backport of this pull request to the3.12 branch.

GH-107157 is a backport of this pull request to the3.11 branch.