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 */

Reply via email to