From 628082158aecf3d5aeb8732d7ddf5dacc45e4d18 Mon Sep 17 00:00:00 2001 From: Stephan Bergmann Date: Fri, 3 Jun 2022 14:01:31 +0200 Subject: [PATCH] Revert "Better use size_t for malloc_size" This reverts commit 652e4ee372e8939f48f69411b6326b237c6d9e4b. Turns out on Windows read (aka _read) actually has a parameter of type unsigned, not size_t, ( "read"), so this started to cause > C:/cygwin/home/tdf/jenkins/workspace/lo_tb_master_win64_dbg/soltools/mkdepend/main.c(504): error C2220: the following warning is treated as an error > C:/cygwin/home/tdf/jenkins/workspace/lo_tb_master_win64_dbg/soltools/mkdepend/main.c(504): warning C4267: 'function': conversion from 'size_t' to 'unsigned int', possible loss of data () for MSVC builds targeting x64. Change-Id: Ica821c8b32e225b7ebbfcbd33d8b7d55515ab3e0 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135270 Tested-by: Jenkins Reviewed-by: Stephan Bergmann --- soltools/mkdepend/main.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/soltools/mkdepend/main.c b/soltools/mkdepend/main.c index 58633c830115..b8b84b453476 100644 --- a/soltools/mkdepend/main.c +++ b/soltools/mkdepend/main.c @@ -473,7 +473,7 @@ struct filepointer *getfile(char *file) struct stat st; off_t size_backup; ssize_t bytes_read; - size_t malloc_size; + unsigned malloc_size; content = (struct filepointer *)malloc(sizeof(struct filepointer)); if ((fd = open(file, O_RDONLY)) < 0) { @@ -486,7 +486,7 @@ struct filepointer *getfile(char *file) size_backup = st.st_size; malloc_size = size_backup; - /* Since off_t usually is larger than size_t, need to test for + /* Since off_t usually is larger than unsigned, need to test for * truncation. */ if ( (off_t)malloc_size != size_backup )