Module Name: src Committed By: riastradh Date: Mon Sep 25 17:09:27 UTC 2023
Modified Files: src/sys/sys: lwp.h Log Message: sys/lwp.h: Revert unnecessary style- and build-breaking change. Broke the clang builds because of duplicate syncobj_t definitions. Just use `struct syncobj'; the `syncobj_t' alias causes more trouble than it's worth -- KNF now advises against it and I see no compelling reason to make an exception here. To generate a diff of this commit: cvs rdiff -u -r1.222 -r1.223 src/sys/sys/lwp.h Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.
Modified files: Index: src/sys/sys/lwp.h diff -u src/sys/sys/lwp.h:1.222 src/sys/sys/lwp.h:1.223 --- src/sys/sys/lwp.h:1.222 Sat Sep 23 20:23:07 2023 +++ src/sys/sys/lwp.h Mon Sep 25 17:09:27 2023 @@ -1,4 +1,4 @@ -/* $NetBSD: lwp.h,v 1.222 2023/09/23 20:23:07 ad Exp $ */ +/* $NetBSD: lwp.h,v 1.223 2023/09/25 17:09:27 riastradh Exp $ */ /* * Copyright (c) 2001, 2006, 2007, 2008, 2009, 2010, 2019, 2020, 2023 @@ -81,7 +81,6 @@ static __inline struct cpu_info *lwp_get #include <sys/pcu.h> -typedef struct syncobj const syncobj_t; struct lockdebug; struct sysent; @@ -131,7 +130,7 @@ struct lwp { kcpuset_t *l_affinity; /* l: CPU set for affinity */ /* Synchronisation. */ - syncobj_t *l_syncobj; /* l: sync object operations set */ + struct syncobj *l_syncobj; /* l: sync object operations set */ LIST_ENTRY(lwp) l_sleepchain; /* l: sleep queue */ wchan_t l_wchan; /* l: sleep address */ const char *l_wmesg; /* l: reason for sleep */