From mboxrd@z Thu Jan 1 00:00:00 1970 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on inbox.vuxu.org X-Spam-Level: X-Spam-Status: No, score=0.0 required=5.0 tests=none autolearn=ham autolearn_force=no version=3.4.4 Received: (qmail 31200 invoked from network); 26 Mar 2023 14:54:29 -0000 Received: from 9front.inri.net (168.235.81.73) by inbox.vuxu.org with ESMTPUTF8; 26 Mar 2023 14:54:29 -0000 Received: from duke.felloff.net ([216.126.196.34]) by 9front; Sun Mar 26 10:52:32 -0400 2023 Message-ID: <27DA3049B946F86EEA8E40F8E12644FA@felloff.net> Date: Sun, 26 Mar 2023 16:52:19 +0200 From: cinap_lenrek@felloff.net To: 9front@9front.org In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" Content-Transfer-Encoding: 7bit List-ID: <9front.9front.org> List-Help: X-Glyph: ➈ X-Bullshit: converged converged property descriptor Subject: Re: [9front] Re: removal of /lib/unicode Reply-To: 9front@9front.org Precedence: bulk IIRC, __func__ and #pragma once are for helping some ape ports only. I'd agree that these are kind of anti features... These should never be used in native plan9 code. On the other hand, if it makes the life easier for porting ape stuff and ideally keeping stuff upstreamed with the original authors i was not against it. The _Noreturn stuff is motivated by practical problems. It comes up often in the kernel which uses the waserror() and error()/nexterror() handling logic. Right now, we squelch the "used but not set" warnings that it causes by initializing stuff always. But this hides real bugs. As an experiment, i changed the error()/nexterror() into macros wrapped in a infinite loop and removed all these dummy initializers and actually found bugs in the scsi driver in the kernel. -- cinap