lib/58042: tm(3) man page has wrong type for struct tm::tm_gmtoff member

[campbell+netbsd%mumble.net@localhost]

>Number:         58042
>Category:       lib
>Synopsis:       tm(3) man page has wrong type for struct tm::tm_gmtoff member
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    lib-bug-people
>State:          open
>Class:          doc-bug
>Submitter-Id:   net
>Arrival-Date:   Fri Mar 15 23:20:00 +0000 2024
>Originator:     Taylor R Campbell
>Release:        current, 10, 9, 8, ...
>Organization:
The NetBSD Foundatime
>Environment:
>Description:
   NetBSD Extensions
     In addition, the following NetBSD-specific fields are available:

           Type             Field      Represents
           int              tm_gmtoff  Offset from UTC in seconds
           __aconst char    tm_zone    Timezone abbreviation

But time.h actually declares:

        long    tm_gmtoff;      /* offset from UTC in seconds */
        __aconst char *tm_zone; /* timezone abbreviation */

Also: documenting __aconst here is confusing.  This appears to be intended for builds with __AUDIT__ defined to catch abuse of what should properly be const, but I don't know if that is wired up anywhere so it might be dead code.  In any case, it's a confusing way to document the interface.
>How-To-Repeat:
man tm
>Fix:
Yes, please!