[LTP] [PATCH v1 1/1] doc/c-test-api.txt: fix broken example code

[LTP] [PATCH v1 1/1] doc/c-test-api.txt: fix broken example code

[Souta Kawahara]

There is double definition of 'test'.
Fixed it.

Signed-off-by: Souta Kawahara <souta.kawahara@miraclelinux.com>
---
 doc/c-test-api.txt | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/doc/c-test-api.txt b/doc/c-test-api.txt
index e4e88edec..560a1a203 100644
--- a/doc/c-test-api.txt
+++ b/doc/c-test-api.txt
@@ -36,7 +36,7 @@ static void setup(void)
 		tst_brk(TBROK | TERRNO, "setenv() failed");
 }
 
-static void test(void)
+static void run(void)
 {
 	char *ret;
 
@@ -64,7 +64,7 @@ next:
 }
 
 static struct tst_test test = {
-	.test_all = test,
+	.test_all = run,
 	.setup = setup,
 };
 -------------------------------------------------------------------------------
-- 
2.25.1


-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

[LTP] [PATCH v2 1/1] fix broken example code

[Souta Kawahara]

There is double definition of 'test'.
Fixed it.
Also Fixed example code for the same function described after that.

Signed-off-by: Souta Kawahara <souta.kawahara@miraclelinux.com>
---
 doc/c-test-api.txt | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/doc/c-test-api.txt b/doc/c-test-api.txt
index e4e88edec..2ad841038 100644
--- a/doc/c-test-api.txt
+++ b/doc/c-test-api.txt
@@ -36,7 +36,7 @@ static void setup(void)
 		tst_brk(TBROK | TERRNO, "setenv() failed");
 }
 
-static void test(void)
+static void run(void)
 {
 	char *ret;
 
@@ -64,7 +64,7 @@ next:
 }
 
 static struct tst_test test = {
-	.test_all = test,
+	.test_all = run,
 	.setup = setup,
 };
 -------------------------------------------------------------------------------
@@ -257,7 +257,7 @@ value and errno and reports the test result.
 
 [source,c]
 -------------------------------------------------------------------------------
-static void test(void)
+static void run(void)
 {
 	...
 	TST_EXP_PASS(stat(fname, &statbuf), "stat(%s, ...)", fname);
@@ -278,7 +278,7 @@ begin with a format string and then appropriate values to be formatted.
 
 [source,c]
 -------------------------------------------------------------------------------
-static void test(void)
+static void run(void)
 {
 	...
 	TST_EXP_FD(open(fname, O_RDONLY), "open(%s, O_RDONLY)", fname);
@@ -294,7 +294,7 @@ positive integer is returned.
 
 [source,c]
 -------------------------------------------------------------------------------
-static void test(void)
+static void run(void)
 {
 	...
 	TST_EXP_FAIL(stat(fname, &statbuf), ENOENT, "stat(%s, ...)", fname);
@@ -308,7 +308,7 @@ passed as the second argument.
 
 [source,c]
 -------------------------------------------------------------------------------
-static void test(void)
+static void run(void)
 {
 	...
 	TST_EXP_FAIL2(msgget(key, flags), EINVAL, "msgget(%i, %i)", key, flags);
-- 
2.25.1


-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

Re: [LTP] [PATCH v2 1/1] fix broken example code

[Avinesh Kumar]

Hi Souta,

Reviewed-by: Avinesh Kumar <akumar@suse.de>

On Thursday, August 24, 2023 11:42:18 AM IST Souta Kawahara wrote:
> There is double definition of 'test'.
> Fixed it.
> Also Fixed example code for the same function described after that.
> 
> Signed-off-by: Souta Kawahara <souta.kawahara@miraclelinux.com>
> ---
>  doc/c-test-api.txt | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/doc/c-test-api.txt b/doc/c-test-api.txt
> index e4e88edec..2ad841038 100644
> --- a/doc/c-test-api.txt
> +++ b/doc/c-test-api.txt
> @@ -36,7 +36,7 @@ static void setup(void)
>  		tst_brk(TBROK | TERRNO, "setenv() failed");
>  }
> 
> -static void test(void)
> +static void run(void)
>  {
>  	char *ret;
> 
> @@ -64,7 +64,7 @@ next:
>  }
> 
>  static struct tst_test test = {
> -	.test_all = test,
> +	.test_all = run,
>  	.setup = setup,
>  };
>  ---------------------------------------------------------------------------
> ---- @@ -257,7 +257,7 @@ value and errno and reports the test result.
> 
>  [source,c]
>  ---------------------------------------------------------------------------
> ---- -static void test(void)
> +static void run(void)
>  {
>  	...
>  	TST_EXP_PASS(stat(fname, &statbuf), "stat(%s, ...)", fname);
> @@ -278,7 +278,7 @@ begin with a format string and then appropriate values
> to be formatted.
> 
>  [source,c]
>  ---------------------------------------------------------------------------
> ---- -static void test(void)
> +static void run(void)
>  {
>  	...
>  	TST_EXP_FD(open(fname, O_RDONLY), "open(%s, O_RDONLY)", fname);
> @@ -294,7 +294,7 @@ positive integer is returned.
> 
>  [source,c]
>  ---------------------------------------------------------------------------
> ---- -static void test(void)
> +static void run(void)
>  {
>  	...
>  	TST_EXP_FAIL(stat(fname, &statbuf), ENOENT, "stat(%s, ...)", 
fname);
> @@ -308,7 +308,7 @@ passed as the second argument.
> 
>  [source,c]
>  ---------------------------------------------------------------------------
> ---- -static void test(void)
> +static void run(void)
>  {
>  	...
>  	TST_EXP_FAIL2(msgget(key, flags), EINVAL, "msgget(%i, %i)", key, 
flags);

--
Regards,
Avinesh



-- 
Mailing list info: https://lists.linux.it/listinfo/ltp