Retire snprintf.c

Issue #1712 closed
Erik Schnetter created an issue

C99 provides snprintf, and we assume at least C99 support in Cactus. Cactus does not need to provide snprintf any more.


Comments (11)

  1. Erik Schnetter reporter
    • removed comment

    Regarding strdup: Ouch. Hello 1972, nice to meet you.

    Isn't there a gcc warning that requires prototypes for all functions? I think we should be able to enable this safely.

  2. Roland Haas
    • removed comment

    There is a gcc warning that one can use for prototypes, however it warns about many benign uses of say printf without prototype. It is also only a warning and I would not want to have to rely on it (or only on its error-ized version from -Werror) to prevent subtle errors (since the wrong prototyped one will work as long as the returned address is less than 2GB) since we have very many other warnings that are triggered when compiling the full set of thorns.

  3. Log in to comment