]> git.ipfire.org Git - thirdparty/libvirt.git/commitdiff
vircgroupmock: Make global variables static
authorMichal Privoznik <mprivozn@redhat.com>
Thu, 3 Mar 2022 15:23:48 +0000 (16:23 +0100)
committerMichal Privoznik <mprivozn@redhat.com>
Fri, 4 Mar 2022 14:08:59 +0000 (15:08 +0100)
Apparently clang was fixed as it no longer considers having
global variables static a problem. Make the variables static to
be sure they aren't used outside of the source file.

This effectively reverts v1.0.6-rc1~198 which started the trend.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>
tests/vircgroupmock.c

index cbf5269df6fed5b8038f1c94dcd03949c0302abc..a213c8e68642a0ec31d6f3742b35893569e620df 100644 (file)
@@ -36,14 +36,10 @@ static FILE *(*real_fopen)(const char *path, const char *mode);
 static int (*real_access)(const char *path, int mode);
 static int (*real_mkdir)(const char *path, mode_t mode);
 
-/* Don't make static, since it causes problems with clang
- * when passed as an arg to asprintf()
- * vircgroupmock.c:462:22: error: static variable 'fakesysfsdir' is used in an inline function with external linkage [-Werror,-Wstatic-in-inline]
- */
-char *fakerootdir;
-char *fakesysfscgroupdir;
-const char *fakedevicedir0 = FAKEDEVDIR0;
-const char *fakedevicedir1 = FAKEDEVDIR1;
+static char *fakerootdir;
+static char *fakesysfscgroupdir;
+static const char *fakedevicedir0 = FAKEDEVDIR0;
+static const char *fakedevicedir1 = FAKEDEVDIR1;
 
 
 # define SYSFS_CGROUP_PREFIX "/not/really/sys/fs/cgroup"