From: Eric McDonald <mcdonald@phy.cmich.edu>
To: Hans Ronne <hronne@comhem.se>
Cc: xconq7@sources.redhat.com
Subject: Re: time.g weirdness
Date: Thu, 12 Aug 2004 04:03:00 -0000 [thread overview]
Message-ID: <411ACE32.40605@phy.cmich.edu> (raw)
In-Reply-To: <l03130301bd3ffec2487f@[212.181.162.155]>
Hans Ronne wrote:
>>On Wed, 11 Aug 2004, Hans Ronne wrote:
> Well, the problem in that case was actually the opposite: it was impossible
> to move into the unknown because the early version of Peter's pathfinder
> always returned false.
Oh, right. I did flip things around.
>I think the problems we see now are due to
> subsequent changes in the command pre-flight code which were in prompted by
> and in part depended on Peter's code. Some of these changes were kept when
> the pathfinding was removed.
>
> I have just checked out the November 12 sources. My intention is to try to
> restore command processing as closely as possible to how it worked back
> then. With no path-finding algorithm, some of these pre-flight checks make
> little sense.
Well, just for the record, removing the pathfinding code was not an easy
exercise once things had started to become entangled as they had.
However, it was not a blind one either; the code surgery would have been
completed much sooner than it was, if I hadn't given each reverse patch
a fair amount of consideration and determined what needed to be cut out
of the reverse patch (in order to keep it in the code repository). It is
certainly possible that with so many different little pieces of code
being touched over a period spanning half a year, and with the
application of an occasional patch hunk failing (and thus needing
further editing), that something slipped through the cracks. It could
also be that I misjudged how a piece of machinery would behave after the
alteration. I did try to pay attention to such things, but, like I said,
it was not an easy task by any means. I also think that, when I
completed the operation, I admonished ourselves to be on the lookout for
breakage, knowing that it would almost certainly be found.
Eric
next prev parent reply other threads:[~2004-08-12 1:56 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-08-10 17:51 Robert Goulding
2004-08-10 18:11 ` Hans Ronne
2004-08-11 1:23 ` Hans Ronne
2004-08-11 16:34 ` Jim Kingdon
2004-08-11 16:34 ` Hans Ronne
2004-08-11 16:47 ` Eric McDonald
2004-08-11 17:09 ` Hans Ronne
2004-08-12 4:03 ` Eric McDonald [this message]
2004-08-12 11:16 ` Hans Ronne
2004-08-11 16:41 ` Eric McDonald
2004-08-12 8:42 ` Jim Kingdon
2004-08-12 8:56 ` Hans Ronne
2004-08-12 15:42 ` Jim Kingdon
2004-08-12 16:58 ` Eric McDonald
2004-08-12 17:35 ` Hans Ronne
2004-08-12 16:55 ` Eric McDonald
2004-08-12 15:39 ` Hans Ronne
2004-08-12 16:53 ` Jim Kingdon
2004-08-12 16:59 ` Hans Ronne
2004-08-12 17:08 ` Eric McDonald
2004-08-12 17:19 ` Hans Ronne
2004-08-12 17:24 ` Eric McDonald
2004-08-12 18:15 ` Hans Ronne
2004-08-13 23:46 ` Eric McDonald
2004-08-14 3:50 ` Wrecked-Type Doesn't Apply to Detonating Units Elijah Meeks
2004-08-14 9:19 ` Eric McDonald
2004-08-14 15:36 ` Hans Ronne
2004-08-14 16:00 ` Eric McDonald
2004-08-14 17:42 ` Eric McDonald
2004-08-16 0:27 ` Elijah Meeks
2004-08-16 18:24 ` Eric McDonald
2004-08-16 21:34 ` Elijah Meeks
2004-08-17 0:50 ` Eric McDonald
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=411ACE32.40605@phy.cmich.edu \
--to=mcdonald@phy.cmich.edu \
--cc=hronne@comhem.se \
--cc=xconq7@sources.redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).