PR/59395 CVS commit: src/bin/sh

["Robert Elz" <kre%netbsd.org@localhost>]

The following reply was made to PR bin/59395; it has been noted by GNATS.

From: "Robert Elz" <kre%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/59395 CVS commit: src/bin/sh
Date: Tue, 6 May 2025 13:09:26 +0000

 Module Name:	src
 Committed By:	kre
 Date:		Tue May  6 13:09:26 UTC 2025
 
 Modified Files:
 	src/bin/sh: memalloc.c
 
 Log Message:
 PR bin/59395  Remove some dead code.
 
 Back in Nov 2021, the way here docs were implemented was changed
 to fix problem report 53550 and this code can no longer ever
 be executed (it is impossible without reintroducing the 53550
 problems), so time for it to go.
 
 This will make no functional difference to sh in HEAD,
 and wouldn't in -10, if this were worth pulling up there (it isn't).
 
 However, in -9 this code still runs (in the relatively rare cases
 that it applies), but is broken, leading to the problem in 59395.
 Removing this code, which is just an optimisation, will correct
 that problem (for current releases only -9 is affected).
 
 XXX pullup -9 (will fix PR bin/59395)
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.39 -r1.40 src/bin/sh/memalloc.c
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.