From f1d3b75561ad4275155327d13696fb5978f47df0 Mon Sep 17 00:00:00 2001 From: "Eric S. Raymond" Date: Tue, 20 Nov 2018 06:54:24 -0500 Subject: [PATCH] Remove comments that are obsolete now that the code is goto-less. --- TODO | 2 -- main.c | 13 ------------- notes.adoc | 6 +----- 3 files changed, 1 insertion(+), 20 deletions(-) diff --git a/TODO b/TODO index 5819776..4290528 100644 --- a/TODO +++ b/TODO @@ -3,8 +3,6 @@ The FORTRANish mess that once was is now mostly idiomatic C. Some issues remain to be cleaned up: -* Remaining unstructured gotos in do_command(). - * The program is still pretty much typeless. Some attempt has been made to introduce semantic types, but the job is barely started. diff --git a/main.c b/main.c index f9fb116..dc879a1 100644 --- a/main.c +++ b/main.c @@ -1,17 +1,4 @@ /* - * There used to be a note that said this: - * - * The author - Don Woods - apologises for the style of the code; it - * is a result of running the original Fortran IV source through a - * home-brew Fortran-to-C converter. - * - * Now that the code has been restructured into something much closer - * to idiomatic C, the following is more appropriate: - * - * ESR apologizes for the remaing gotos (now confined to one function - * in this file - there used to be over 350 of them, *everywhere*). - * Applying the Structured Program Theorem can be hard. - * * Copyright (c) 1977, 2005 by Will Crowther and Don Woods * Copyright (c) 2017 by Eric S. Raymond * SPDX-License-Identifier: BSD-2-clause diff --git a/notes.adoc b/notes.adoc index 538de28..8838393 100644 --- a/notes.adoc +++ b/notes.adoc @@ -13,7 +13,7 @@ the game; Jason signed on early in the process to help. The assistance of Peje Nilsson in restructuring some particularly grotty gotos is gratefully acknowledged. Petr Voropaev contributed fuzz testing and code cleanups. Aaron Traas did a lot of painstaking work to improve -test coverage. +test coverage, and factored out the last handful of gotos. == Nomenclature == @@ -155,10 +155,6 @@ ways: and the choice to refrain will make forward translation into future languages easier. -* There are a few gotos left that resist restructuring; all are in the - principal command interpreter function implementing its state - machine. - * Linked lists (for objects at a location) are implemented using an array of link indices. This is a surviving FORTRANism that is quite unlike normal practice in C or any more modern language. We have not tried -- 2.31.1