All of lore.kernel.org
 help / color / mirror / Atom feed
* [LTP] [PATCH] Replace deprecated <strings.h> with <string.h>
@ 2020-03-25 10:01 Jozef Pupava
  2020-03-25 10:12 ` Cyril Hrubis
  0 siblings, 1 reply; 2+ messages in thread
From: Jozef Pupava @ 2020-03-25 10:01 UTC (permalink / raw)
  To: ltp

I guess this won't really solve anything, just the issue.

Closes: #521

Signed-off-by: Jozef Pupava <jpupava@suse.com>
---
 testcases/kernel/containers/share/ns_create.c                           | 2 +-
 testcases/kernel/fs/fsx-linux/fsx-linux.c                               | 2 +-
 .../open_posix_testsuite/conformance/interfaces/sem_timedwait/3-1.c     | 2 +-
 .../open_posix_testsuite/conformance/interfaces/sem_timedwait/7-1.c     | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/testcases/kernel/containers/share/ns_create.c b/testcases/kernel/containers/share/ns_create.c
index 1d0339e71..4a4007c3a 100644
--- a/testcases/kernel/containers/share/ns_create.c
+++ b/testcases/kernel/containers/share/ns_create.c
@@ -30,7 +30,7 @@
 #include <sys/stat.h>
 #include <unistd.h>
 #include <string.h>
-#include <strings.h>
+#include <string.h>
 #include <errno.h>
 #include "test.h"
 #include "lapi/namespaces_constants.h"
diff --git a/testcases/kernel/fs/fsx-linux/fsx-linux.c b/testcases/kernel/fs/fsx-linux/fsx-linux.c
index 02f3eb827..4a20f4da6 100644
--- a/testcases/kernel/fs/fsx-linux/fsx-linux.c
+++ b/testcases/kernel/fs/fsx-linux/fsx-linux.c
@@ -43,7 +43,7 @@
 #include <sys/param.h>
 #include <limits.h>
 #include <time.h>
-#include <strings.h>
+#include <string.h>
 #include <sys/time.h>
 #endif
 #include <fcntl.h>
diff --git a/testcases/open_posix_testsuite/conformance/interfaces/sem_timedwait/3-1.c b/testcases/open_posix_testsuite/conformance/interfaces/sem_timedwait/3-1.c
index dd42bbd91..97bdbff5d 100644
--- a/testcases/open_posix_testsuite/conformance/interfaces/sem_timedwait/3-1.c
+++ b/testcases/open_posix_testsuite/conformance/interfaces/sem_timedwait/3-1.c
@@ -20,7 +20,7 @@
 #endif
 #include <semaphore.h>
 #include <stdio.h>
-#include <strings.h>
+#include <string.h>
 #include <time.h>
 #include <unistd.h>
 #include "posixtest.h"
diff --git a/testcases/open_posix_testsuite/conformance/interfaces/sem_timedwait/7-1.c b/testcases/open_posix_testsuite/conformance/interfaces/sem_timedwait/7-1.c
index 79de0af7f..da2cd3084 100644
--- a/testcases/open_posix_testsuite/conformance/interfaces/sem_timedwait/7-1.c
+++ b/testcases/open_posix_testsuite/conformance/interfaces/sem_timedwait/7-1.c
@@ -21,7 +21,7 @@
 #endif
 #include <semaphore.h>
 #include <stdio.h>
-#include <strings.h>
+#include <string.h>
 #include <time.h>
 #include <unistd.h>
 #include "posixtest.h"
-- 
2.16.4

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [LTP] [PATCH] Replace deprecated <strings.h> with <string.h>
  2020-03-25 10:01 [LTP] [PATCH] Replace deprecated <strings.h> with <string.h> Jozef Pupava
@ 2020-03-25 10:12 ` Cyril Hrubis
  0 siblings, 0 replies; 2+ messages in thread
From: Cyril Hrubis @ 2020-03-25 10:12 UTC (permalink / raw)
  To: ltp

Hi!
> Closes: #521
> 
> Signed-off-by: Jozef Pupava <jpupava@suse.com>
> ---
>  testcases/kernel/containers/share/ns_create.c                           | 2 +-
>  testcases/kernel/fs/fsx-linux/fsx-linux.c                               | 2 +-
>  .../open_posix_testsuite/conformance/interfaces/sem_timedwait/3-1.c     | 2 +-
>  .../open_posix_testsuite/conformance/interfaces/sem_timedwait/7-1.c     | 2 +-
>  4 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/testcases/kernel/containers/share/ns_create.c b/testcases/kernel/containers/share/ns_create.c
> index 1d0339e71..4a4007c3a 100644
> --- a/testcases/kernel/containers/share/ns_create.c
> +++ b/testcases/kernel/containers/share/ns_create.c
> @@ -30,7 +30,7 @@
>  #include <sys/stat.h>
>  #include <unistd.h>
>  #include <string.h>
> -#include <strings.h>
> +#include <string.h>

The string.h include is already there, so we just need to delete it
here.

>  #include <errno.h>
>  #include "test.h"
>  #include "lapi/namespaces_constants.h"
> diff --git a/testcases/kernel/fs/fsx-linux/fsx-linux.c b/testcases/kernel/fs/fsx-linux/fsx-linux.c
> index 02f3eb827..4a20f4da6 100644
> --- a/testcases/kernel/fs/fsx-linux/fsx-linux.c
> +++ b/testcases/kernel/fs/fsx-linux/fsx-linux.c
> @@ -43,7 +43,7 @@
>  #include <sys/param.h>
>  #include <limits.h>
>  #include <time.h>
> -#include <strings.h>
> +#include <string.h>
>  #include <sys/time.h>
>  #endif
>  #include <fcntl.h>
> diff --git a/testcases/open_posix_testsuite/conformance/interfaces/sem_timedwait/3-1.c b/testcases/open_posix_testsuite/conformance/interfaces/sem_timedwait/3-1.c
> index dd42bbd91..97bdbff5d 100644
> --- a/testcases/open_posix_testsuite/conformance/interfaces/sem_timedwait/3-1.c
> +++ b/testcases/open_posix_testsuite/conformance/interfaces/sem_timedwait/3-1.c
> @@ -20,7 +20,7 @@
>  #endif
>  #include <semaphore.h>
>  #include <stdio.h>
> -#include <strings.h>
> +#include <string.h>
>  #include <time.h>
>  #include <unistd.h>
>  #include "posixtest.h"
> diff --git a/testcases/open_posix_testsuite/conformance/interfaces/sem_timedwait/7-1.c b/testcases/open_posix_testsuite/conformance/interfaces/sem_timedwait/7-1.c
> index 79de0af7f..da2cd3084 100644
> --- a/testcases/open_posix_testsuite/conformance/interfaces/sem_timedwait/7-1.c
> +++ b/testcases/open_posix_testsuite/conformance/interfaces/sem_timedwait/7-1.c
> @@ -21,7 +21,7 @@
>  #endif
>  #include <semaphore.h>
>  #include <stdio.h>
> -#include <strings.h>
> +#include <string.h>
>  #include <time.h>
>  #include <unistd.h>
>  #include "posixtest.h"

As far as I can tell there are no string functions used in the
sem_timedwait() tests, so we can as well just remove the include in
these two.

-- 
Cyril Hrubis
chrubis@suse.cz

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2020-03-25 10:12 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-25 10:01 [LTP] [PATCH] Replace deprecated <strings.h> with <string.h> Jozef Pupava
2020-03-25 10:12 ` Cyril Hrubis

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.