[InterNetNews/inn] 5f260c: Include portable/system.h as first portable header

Julien ÉLIE noreply at github.com
Fri Aug 4 20:32:44 UTC 2023


  Branch: refs/heads/main
  Home:   https://github.com/InterNetNews/inn
  Commit: 5f260cbb794c847c74739f0197344ef3f42a2367
      https://github.com/InterNetNews/inn/commit/5f260cbb794c847c74739f0197344ef3f42a2367
  Author: Julien ÉLIE <Julien-Elie at users.noreply.github.com>
  Date:   2023-08-04 (Fri, 04 Aug 2023)

  Changed paths:
    M .clang-format
    M lib/Makefile
    M lib/asprintf.c
    M lib/getaddrinfo.c
    M lib/getnameinfo.c
    M lib/inet_aton.c
    M lib/inet_ntoa.c
    M lib/inet_ntop.c
    M lib/network.c
    M tests/innd/fakeinnd.c
    M tests/lib/asprintf-t.c
    M tests/lib/fdflag-t.c
    M tests/lib/getaddrinfo-t.c
    M tests/lib/getnameinfo-t.c
    M tests/lib/inet_aton-t.c
    M tests/lib/inet_ntoa-t.c
    M tests/lib/inet_ntop-t.c
    M tests/lib/network/addr-ipv4-t.c
    M tests/lib/network/addr-ipv6-t.c
    M tests/lib/network/client-t.c
    M tests/lib/network/server-t.c

  Log Message:
  -----------
  Include portable/system.h as first portable header

In a block of included headers, sort them with config.h first, followed
with portable/system.h and then the other headers in alphabetical
order.

Otherwise some defines in portable/system.h which could be used by
other headers may not be defined.  It notably affects the forced
FD_SETSIZE redefinition which was not taken into account when
portable/socket.h was included before portable/system.h.  This is now
fixed.

Thanks to Jesse Rehmer for the bug report.

see #273




More information about the inn-committers mailing list