[PATCH] including string.h is not needed

Gedare Bloom gedare at rtems.org
Fri Mar 25 13:25:22 UTC 2016


This looks fine to me if someone cares to commit it.

On Wed, Feb 17, 2016 at 5:05 AM, Zhu Zhongjie <zhongjiezhu at yahoo.com> wrote:
>
> ---
>  cpukit/libcsupport/src/assoclocalbyname.c           | 2 --
>  cpukit/libcsupport/src/assoclocalbyremote.c         | 2 --
>  cpukit/libcsupport/src/assoclocalbyremotebitfield.c | 3 ---
>  cpukit/libcsupport/src/assocnamebad.c               | 1 -
>  cpukit/libcsupport/src/assocnamebylocal.c           | 2 --
>  cpukit/libcsupport/src/assocnamebyremote.c          | 2 --
>  cpukit/libcsupport/src/assocptrbylocal.c            | 2 --
>  cpukit/libcsupport/src/assocptrbyremote.c           | 2 --
>  cpukit/libcsupport/src/assocremotebylocal.c         | 2 --
>  cpukit/libcsupport/src/assocremotebylocalbitfield.c | 3 ---
>  cpukit/libcsupport/src/assocremotebyname.c          | 2 --
>  11 files changed, 23 deletions(-)
>
> diff --git a/cpukit/libcsupport/src/assoclocalbyname.c
> b/cpukit/libcsupport/src/assoclocalbyname.c
> index 3904500..2be2f65 100644
> --- a/cpukit/libcsupport/src/assoclocalbyname.c
> +++ b/cpukit/libcsupport/src/assoclocalbyname.c
> @@ -23,8 +23,6 @@
>  #include <rtems.h>
>  #include <rtems/assoc.h>
>
> -#include <string.h>             /* strcat, strcmp */
> -
>  uint32_t rtems_assoc_local_by_name(
>    const rtems_assoc_t *ap,
>    const char          *name
> diff --git a/cpukit/libcsupport/src/assoclocalbyremote.c
> b/cpukit/libcsupport/src/assoclocalbyremote.c
> index 24a86d8..0d06118 100644
> --- a/cpukit/libcsupport/src/assoclocalbyremote.c
> +++ b/cpukit/libcsupport/src/assoclocalbyremote.c
> @@ -14,8 +14,6 @@
>  #include <rtems.h>
>  #include <rtems/assoc.h>
>
> -#include <string.h>             /* strcat, strcmp */
> -
>  uint32_t rtems_assoc_local_by_remote(
>    const rtems_assoc_t *ap,
>    uint32_t             remote_value
> diff --git a/cpukit/libcsupport/src/assoclocalbyremotebitfield.c
> b/cpukit/libcsupport/src/assoclocalbyremotebitfield.c
> index 23b4c72..8fe24cb 100644
> --- a/cpukit/libcsupport/src/assoclocalbyremotebitfield.c
> +++ b/cpukit/libcsupport/src/assoclocalbyremotebitfield.c
> @@ -12,9 +12,6 @@
>  #include <rtems.h>
>  #include <rtems/assoc.h>
>
> -#include <string.h>             /* strcat, strcmp */
> -
> -
>  uint32_t   rtems_assoc_local_by_remote_bitfield(
>      const rtems_assoc_t *ap,
>      uint32_t       remote_value
> diff --git a/cpukit/libcsupport/src/assocnamebad.c
> b/cpukit/libcsupport/src/assocnamebad.c
> index 3369141..f3084d5 100644
> --- a/cpukit/libcsupport/src/assocnamebad.c
> +++ b/cpukit/libcsupport/src/assocnamebad.c
> @@ -14,7 +14,6 @@
>  #include <rtems.h>
>  #include <rtems/assoc.h>
>
> -#include <inttypes.h>
>  #include <stdio.h>              /* sprintf */
>
>  const char *
> diff --git a/cpukit/libcsupport/src/assocnamebylocal.c
> b/cpukit/libcsupport/src/assocnamebylocal.c
> index 0dd47b8..1f752a1 100644
> --- a/cpukit/libcsupport/src/assocnamebylocal.c
> +++ b/cpukit/libcsupport/src/assocnamebylocal.c
> @@ -14,8 +14,6 @@
>  #include <rtems.h>
>  #include <rtems/assoc.h>
>
> -#include <string.h>             /* strcat, strcmp */
> -
>  const char *rtems_assoc_name_by_local(
>    const rtems_assoc_t *ap,
>    uint32_t             local_value
> diff --git a/cpukit/libcsupport/src/assocnamebyremote.c
> b/cpukit/libcsupport/src/assocnamebyremote.c
> index 376ba47..70c6c38 100644
> --- a/cpukit/libcsupport/src/assocnamebyremote.c
> +++ b/cpukit/libcsupport/src/assocnamebyremote.c
> @@ -14,8 +14,6 @@
>  #include <rtems.h>
>  #include <rtems/assoc.h>
>
> -#include <string.h>             /* strcat, strcmp */
> -
>
>  const char *rtems_assoc_name_by_remote(
>    const rtems_assoc_t *ap,
> diff --git a/cpukit/libcsupport/src/assocptrbylocal.c
> b/cpukit/libcsupport/src/assocptrbylocal.c
> index a0e7c3c..bacc3db 100644
> --- a/cpukit/libcsupport/src/assocptrbylocal.c
> +++ b/cpukit/libcsupport/src/assocptrbylocal.c
> @@ -14,8 +14,6 @@
>  #include <rtems.h>
>  #include <rtems/assoc.h>
>
> -#include <string.h>             /* strcat, strcmp */
> -
>  const rtems_assoc_t *rtems_assoc_ptr_by_local(
>    const rtems_assoc_t *ap,
>    uint32_t             local_value
> diff --git a/cpukit/libcsupport/src/assocptrbyremote.c
> b/cpukit/libcsupport/src/assocptrbyremote.c
> index 4651581..efb6356 100644
> --- a/cpukit/libcsupport/src/assocptrbyremote.c
> +++ b/cpukit/libcsupport/src/assocptrbyremote.c
> @@ -14,8 +14,6 @@
>  #include <rtems.h>
>  #include <rtems/assoc.h>
>
> -#include <string.h>             /* strcat, strcmp */
> -
>  const rtems_assoc_t *rtems_assoc_ptr_by_remote(
>    const rtems_assoc_t *ap,
>    uint32_t       remote_value
> diff --git a/cpukit/libcsupport/src/assocremotebylocal.c
> b/cpukit/libcsupport/src/assocremotebylocal.c
> index cd5fa0c..2f8d384 100644
> --- a/cpukit/libcsupport/src/assocremotebylocal.c
> +++ b/cpukit/libcsupport/src/assocremotebylocal.c
> @@ -12,8 +12,6 @@
>  #include <rtems.h>
>  #include <rtems/assoc.h>
>
> -#include <string.h>             /* strcat, strcmp */
> -
>  uint32_t rtems_assoc_remote_by_local(
>    const rtems_assoc_t *ap,
>    uint32_t       local_value
> diff --git a/cpukit/libcsupport/src/assocremotebylocalbitfield.c
> b/cpukit/libcsupport/src/assocremotebylocalbitfield.c
> index b26c98c..4da16bd 100644
> --- a/cpukit/libcsupport/src/assocremotebylocalbitfield.c
> +++ b/cpukit/libcsupport/src/assocremotebylocalbitfield.c
> @@ -23,9 +23,6 @@
>  #include <rtems.h>
>  #include <rtems/assoc.h>
>
> -#include <string.h>             /* strcat, strcmp */
> -
> -
>  uint32_t   rtems_assoc_remote_by_local_bitfield(
>      const rtems_assoc_t *ap,
>      uint32_t             local_value
> diff --git a/cpukit/libcsupport/src/assocremotebyname.c
> b/cpukit/libcsupport/src/assocremotebyname.c
> index 8a894fc..bd9c2ba 100644
> --- a/cpukit/libcsupport/src/assocremotebyname.c
> +++ b/cpukit/libcsupport/src/assocremotebyname.c
> @@ -14,8 +14,6 @@
>  #include <rtems.h>
>  #include <rtems/assoc.h>
>
> -#include <string.h>             /* strcat, strcmp */
> -
>  uint32_t rtems_assoc_remote_by_name(
>    const rtems_assoc_t *ap,
>    const char          *name
> --
> 1.9.1
>
>
> _______________________________________________
> devel mailing list
> devel at rtems.org
> http://lists.rtems.org/mailman/listinfo/devel


More information about the devel mailing list