From 55d18c302edd5ddae5cdab06f01d9469c7857af2 Mon Sep 17 00:00:00 2001 From: Joel Sherrill Date: Wed, 28 Jan 2026 10:48:42 -0600 Subject: [PATCH] fstests: Address -Wsign-compare warnings This warning occurs when comparing a signed variable to an unsigned one. This is frequently an int or ssize_t variable compared to a uint32_t or size_t. Sometimes the size_t is from a sizeof() use. --- testsuites/fstests/fsfatfssync01/init.c | 2 +- testsuites/fstests/fsrdwr/init.c | 4 ++-- testsuites/fstests/fssymlink/test.c | 6 +++--- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/testsuites/fstests/fsfatfssync01/init.c b/testsuites/fstests/fsfatfssync01/init.c index 245024450b..9d989202e9 100644 --- a/testsuites/fstests/fsfatfssync01/init.c +++ b/testsuites/fstests/fsfatfssync01/init.c @@ -107,7 +107,7 @@ static void write_to_file( const char *file, bool sync ) rtems_test_assert( n == (ssize_t) sizeof( buf ) ); pos_after = lseek( fd, 0, SEEK_END ); - rtems_test_assert( pos_after == pos_before + sizeof( buf ) ); + rtems_test_assert( pos_after == pos_before + (ssize_t) sizeof( buf ) ); if ( sync ) { rv = fsync( fd ); diff --git a/testsuites/fstests/fsrdwr/init.c b/testsuites/fstests/fsrdwr/init.c index bdd0a4a590..d633ae0b23 100644 --- a/testsuites/fstests/fsrdwr/init.c +++ b/testsuites/fstests/fsrdwr/init.c @@ -612,7 +612,7 @@ block_rw_lseek (int fd, size_t pos) off_t actual; actual = lseek (fd, pos, SEEK_SET); - rtems_test_assert (actual == pos); + rtems_test_assert (actual == (off_t) pos); } static void @@ -648,7 +648,7 @@ block_rw_check (int fd, const char *out, char *in, size_t size) off_t file_size; file_size = lseek (fd, 0, SEEK_END); - rtems_test_assert (file_size == size); + rtems_test_assert (file_size == (off_t) size); block_rw_lseek (fd, 0); diff --git a/testsuites/fstests/fssymlink/test.c b/testsuites/fstests/fssymlink/test.c index 9fc8c9bd2d..c0b9036337 100644 --- a/testsuites/fstests/fssymlink/test.c +++ b/testsuites/fstests/fssymlink/test.c @@ -81,7 +81,7 @@ static void symlink_test01(void ) status=lstat(symlink_file01,&statbuf); rtems_test_assert(status==0); rtems_test_assert(S_ISLNK(statbuf.st_mode)); - rtems_test_assert(len==statbuf.st_size); + rtems_test_assert((off_t)len==statbuf.st_size); puts("call readlink "); @@ -99,7 +99,7 @@ static void symlink_test01(void ) status=lstat(symlink_file01,&statbuf); rtems_test_assert(status==0); rtems_test_assert(S_ISLNK(statbuf.st_mode)); - rtems_test_assert(len==statbuf.st_size); + rtems_test_assert((off_t)len==statbuf.st_size); puts("call readlink "); name_len=readlink(symlink_file01,name,sizeof(name)-1); @@ -119,7 +119,7 @@ static void symlink_test01(void ) status=lstat(symlink_file01,&statbuf); rtems_test_assert(status==0); rtems_test_assert(S_ISLNK(statbuf.st_mode)); - rtems_test_assert(len==statbuf.st_size); + rtems_test_assert((off_t)len==statbuf.st_size); puts("call readlink ");