🚀 go-pugleaf

RetroBBS NetNews Server

Inspired by RockSolid Light RIP Retro Guy

Article View: gnu.epoch.misc
Article #366

epoch.el sets global target-buffer

#366
From: Eric Eide
Date: Fri, 13 Nov 1992 16:49
29 lines
1563 bytes
What a strange subject, eh?  What I'm trying to say is that several of the
functions in Epoch 4.2's "epoch.el" set the global variable "target-buffer"
without any apparent need:

  (defun find-file-other-screen (filename)
    "Find file in other screen"
    (interactive "FFind file other screen: ")
    (setq target-buffer (find-file-noselect filename))
    (if (bufferp target-buffer) (find-buffer-other-screen target-buffer))
  )

This looks like sloppy coding (IMHO); the offending functions should just make
a binding for target-buffer as needed:

  (defun find-file-other-screen (filename)
    "Find file in other screen"
    (interactive "FFind file other screen: ")
    (let ((target-buffer (find-file-noselect filename)))
      (if (bufferp target-buffer)
          (find-buffer-other-screen target-buffer))
      ))

Perhaps there's something I'm not seeing, or perhaps this is just historical
accident.

--
-------------------------------------------------------------------------------
Eric Eide          |          University of Utah Department of Computer Science
eeide@cs.utah.edu  | Buddhist to hot dog vendor: "Make me one with everything."
Technical Details
Message-ID: <EEIDE.92Nov13164946@asylum.cs.utah.edu>
Path: arc1-ca.pugleaf.net!sparky!uunet!convex!linac!uwm.edu!caen!hellgate.utah.edu!eeide

Show Full Headers
Complete Article Headers
Newsgroups: gnu.epoch.misc
Path: arc1-ca.pugleaf.net!sparky!uunet!convex!linac!uwm.edu!caen!hellgate.utah.edu!eeide
From: eeide%asylum.cs.utah.edu@cs.utah.edu (Eric Eide)
Subject: epoch.el sets global target-buffer
Message-ID: <EEIDE.92Nov13164946@asylum.cs.utah.edu>
Organization: University of Utah Department of Computer Science
Distribution: gnu
Date: 13 Nov 92 16:49:46
Lines: 29