* bug#2270: 23.0.90; find-library: (1) doc string, (2) other-window version @ 2009-02-11 2:27 Chong Yidong 2012-10-24 17:41 ` bug#2270: [PATCH] bug#2270, RE: 23.0.90; find-library:... " Drew Adams 0 siblings, 1 reply; 13+ messages in thread From: Chong Yidong @ 2009-02-11 2:27 UTC (permalink / raw) To: Drew Adams; +Cc: 2270 > 1. The doc string of find-library says only this: > "Find the elisp source of LIBRARY." > > Hardly very helpful. I've clarified it. > 2. Please add this command (or equivalent), after adjusting its doc > string also to explain what LIBRARY is: After the release, maybe. ^ permalink raw reply [flat|nested] 13+ messages in thread
* bug#2270: [PATCH] bug#2270, RE: 23.0.90; find-library:... (2) other-window version 2009-02-11 2:27 bug#2270: 23.0.90; find-library: (1) doc string, (2) other-window version Chong Yidong @ 2012-10-24 17:41 ` Drew Adams [not found] ` <jwv3913g142.fsf-monnier+emacs@gnu.org> 0 siblings, 1 reply; 13+ messages in thread From: Drew Adams @ 2012-10-24 17:41 UTC (permalink / raw) To: 'Chong Yidong'; +Cc: 2270, 2270 > Sent: Tuesday, February 10, 2009 6:27 PM > > > 2. Please add this command (or equivalent), after adjusting its doc > > string also to explain what LIBRARY is: > > After the release, maybe. Clearly, that upcoming release was many moon ago. I repeat my suggestion to add command `find-library-other-window'. I also suggest that it be bound by default to `C-x 4 l'. I've used this command & key forever, and wouldn't be without it. (And I rarely have a need anymore for same-window `find-library'.) --- (defun find-library-other-window (library) "Find the Emacs-Lisp source of LIBRARY in another window." (interactive (progn (require 'find-func) (let* ((path (cons (or find-function-source-path load-path) (find-library-suffixes))) (def (if (eq (function-called-at-point) 'require) (save-excursion (backward-up-list) (forward-char) (backward-sexp -2) (thing-at-point 'symbol)) (thing-at-point 'symbol)))) (when def (setq def (and (locate-file-completion def path 'test) def))) (list (completing-read "Library name: " 'locate-file-completion path nil nil nil def))))) (let ((buf (find-file-noselect (find-library-name library)))) (pop-to-buffer buf 'other-window))) ^ permalink raw reply [flat|nested] 13+ messages in thread
[parent not found: <jwv3913g142.fsf-monnier+emacs@gnu.org>]
* bug#2270: [PATCH] bug#2270, RE: 23.0.90; find-library:... (2) other-window version [not found] ` <jwv3913g142.fsf-monnier+emacs@gnu.org> @ 2012-10-24 19:38 ` Drew Adams 2012-10-24 20:42 ` Stefan Monnier 0 siblings, 1 reply; 13+ messages in thread From: Drew Adams @ 2012-10-24 19:38 UTC (permalink / raw) To: 'Stefan Monnier'; +Cc: 2270, 2270 > > I repeat my suggestion to add command `find-library-other-window'. > > As mentioned in other threads, I'd rather we add a "use other > window for next command" prefix command (which we could bind > to C-x 4), I could live with adding yet another such foo-other-window > command since we don't yet have such a prefix command. Yes, please add it while waiting for your ideal to bless the planet. ;-) > > I also suggest that it be bound by default to `C-x 4 l'. > > Since find-library is not bound to any key, I don't think > find-library-other-window should be bound either. "Since"? The one doesn't follow from the other. There is no logical reason why we cannot have one command bound and not the other. It depends on what we expect the most common uses to be. I proposed a key for the other-window version because I don't think the same-window version is very useful. But that's me. In any case, FWIW, I disagree that this should not be bound by default. But if you are coming from the point of view that things must follow your proposed new _implementation_, then such a hard-and-fast rule pretty much follows, I'm guessing. In spite of the fact that it might not be useful/needed in all cases. Hammer therefore nail? > > I've used this command & key forever, and wouldn't be without it. > > (And I rarely have a need anymore for same-window `find-library'.) > > In my experience, people rarely need both the "same-window" and the > "other-window" forms of a command, indeed. Disagree, as one of the "people". In this case, yes (this person rarely uses the same-window version of this particular command). But not in general. I use BOTH versions of most same- and other-window commands. You keep repeating that supposition, BTW, but so far haven't provided any data backing it up. I, for one, would find it hard to believe that many people use only `C-x C-f' or `C-x 4 f', but not both. Perhaps this too is hammer therefore nail? Does your proposed implementation perhaps require a user to opt for one or the other for all uses? Whatever. I have the command/key for my own use. ^ permalink raw reply [flat|nested] 13+ messages in thread
* bug#2270: [PATCH] bug#2270, RE: 23.0.90; find-library:... (2) other-window version 2012-10-24 19:38 ` Drew Adams @ 2012-10-24 20:42 ` Stefan Monnier 2012-10-24 21:20 ` Stefan Monnier 2012-10-24 21:25 ` Drew Adams 0 siblings, 2 replies; 13+ messages in thread From: Stefan Monnier @ 2012-10-24 20:42 UTC (permalink / raw) To: Drew Adams; +Cc: 2270, 2270 > Yes, please add it while waiting for your ideal to bless the planet. ;-) Please provide a proper patch (i.e. not one that doesn't simply duplicate find-library's code.) > only `C-x C-f' or `C-x 4 f', but not both. Actually, adding a C-x 4 prefix that works for "all" commands would make sure that both options are always available. Stefan ^ permalink raw reply [flat|nested] 13+ messages in thread
* bug#2270: [PATCH] bug#2270, RE: 23.0.90; find-library:... (2) other-window version 2012-10-24 20:42 ` Stefan Monnier @ 2012-10-24 21:20 ` Stefan Monnier 2012-10-24 21:25 ` Drew Adams 1 sibling, 0 replies; 13+ messages in thread From: Stefan Monnier @ 2012-10-24 21:20 UTC (permalink / raw) To: Drew Adams; +Cc: 2270, 2270 > Please provide a proper patch (i.e. not one that doesn't simply ^^^ > duplicate find-library's code.) Sorry for the extra `not'. Stefan ^ permalink raw reply [flat|nested] 13+ messages in thread
* bug#2270: [PATCH] bug#2270, RE: 23.0.90; find-library:... (2) other-window version 2012-10-24 20:42 ` Stefan Monnier 2012-10-24 21:20 ` Stefan Monnier @ 2012-10-24 21:25 ` Drew Adams 2016-04-27 17:26 ` Lars Ingebrigtsen 1 sibling, 1 reply; 13+ messages in thread From: Drew Adams @ 2012-10-24 21:25 UTC (permalink / raw) To: 'Stefan Monnier'; +Cc: 2270, 2270 > Please provide a proper patch (i.e. not one that doesn't simply > duplicate find-library's code.) Dunno what you mean. Do you just mean a proper patch (e.g. `diff' output) or are you saying that you want code that is different from what I sent? > > only `C-x C-f' or `C-x 4 f', but not both. > > Actually, adding a C-x 4 prefix that works for "all" commands > would make sure that both options are always available. That's fine. I already spoke up in favor of your idea (in the abstract). I assume that it will be just as easy as now for a user to bind a different function to one or the other key sequence, without affecting the other? IOW, today a user can easily bind `C-x 4 f' to `butterfly' and leave `C-x C-f' bound to `find-file'. I assume that will be just as easy after your implementation. Likewise, for unbinding one of the two but not the other. ^ permalink raw reply [flat|nested] 13+ messages in thread
* bug#2270: [PATCH] bug#2270, RE: 23.0.90; find-library:... (2) other-window version 2012-10-24 21:25 ` Drew Adams @ 2016-04-27 17:26 ` Lars Ingebrigtsen 2016-04-27 17:34 ` Drew Adams 0 siblings, 1 reply; 13+ messages in thread From: Lars Ingebrigtsen @ 2016-04-27 17:26 UTC (permalink / raw) To: Drew Adams; +Cc: 2270, 'Chong Yidong', 'Stefan Monnier', 2270 I've now added a prefix argument to pop to a different window. I've also cleaned up the switch-to-buffer error handling code, which I think isn't necessary any more after Martin's window fixes... -- (domestic pets only, the antidote for overdose, milk.) bloggy blog: http://lars.ingebrigtsen.no ^ permalink raw reply [flat|nested] 13+ messages in thread
* bug#2270: [PATCH] bug#2270, RE: 23.0.90; find-library:... (2) other-window version 2016-04-27 17:26 ` Lars Ingebrigtsen @ 2016-04-27 17:34 ` Drew Adams 2016-04-27 17:38 ` Lars Ingebrigtsen 2016-04-27 18:46 ` Stefan Monnier 0 siblings, 2 replies; 13+ messages in thread From: Drew Adams @ 2016-04-27 17:34 UTC (permalink / raw) To: Lars Ingebrigtsen; +Cc: 2270, Chong Yidong, Stefan Monnier, 2270 > I've now added a prefix argument to pop to a different window. I've > also cleaned up the switch-to-buffer error handling code, which I think > isn't necessary any more after Martin's window fixes... 1. I would still prefer to have the other-window version bound to `C-x 4 l'. 2. I think that the other-window version is far more useful than the same-window version. So if other-window behavior is available only via a prefix arg I'd prefer that it be the other way around: a prefix arg uses the same window; no prefix arg uses another window. (Yes, that would be a change in the default behavior.) (Anyway, thanks for adding the other-window behavior.) ^ permalink raw reply [flat|nested] 13+ messages in thread
* bug#2270: [PATCH] bug#2270, RE: 23.0.90; find-library:... (2) other-window version 2016-04-27 17:34 ` Drew Adams @ 2016-04-27 17:38 ` Lars Ingebrigtsen 2016-04-27 17:43 ` Dmitry Gutov 2016-04-27 18:46 ` Stefan Monnier 1 sibling, 1 reply; 13+ messages in thread From: Lars Ingebrigtsen @ 2016-04-27 17:38 UTC (permalink / raw) To: Drew Adams; +Cc: 2270, Chong Yidong, Stefan Monnier, 2270 Drew Adams <drew.adams@oracle.com> writes: >> I've now added a prefix argument to pop to a different window. I've >> also cleaned up the switch-to-buffer error handling code, which I think >> isn't necessary any more after Martin's window fixes... > > 1. I would still prefer to have the other-window version bound to > `C-x 4 l'. `find-library' isn't bound to any key, but I think it probably should be. `C-x 4 l' seems like as good a keystroke as any. Any thought from other people? I use the command quite a lot... > 2. I think that the other-window version is far more useful than > the same-window version. So if other-window behavior is available > only via a prefix arg I'd prefer that it be the other way around: > a prefix arg uses the same window; no prefix arg uses another > window. (Yes, that would be a change in the default behavior.) The command now uses pop-to-buffer-same-window, which is quite customisable, as I understand things. (I haven't looked into the new-fangled window popping controlling functions.) So I think this should be customisable ... somehow ... by the users now. -- (domestic pets only, the antidote for overdose, milk.) bloggy blog: http://lars.ingebrigtsen.no ^ permalink raw reply [flat|nested] 13+ messages in thread
* bug#2270: [PATCH] bug#2270, RE: 23.0.90; find-library:... (2) other-window version 2016-04-27 17:38 ` Lars Ingebrigtsen @ 2016-04-27 17:43 ` Dmitry Gutov 2016-04-27 17:56 ` Lars Ingebrigtsen 0 siblings, 1 reply; 13+ messages in thread From: Dmitry Gutov @ 2016-04-27 17:43 UTC (permalink / raw) To: Lars Ingebrigtsen, Drew Adams; +Cc: 2270, Chong Yidong, Stefan Monnier, 2270 On 04/27/2016 08:38 PM, Lars Ingebrigtsen wrote: > `find-library' isn't bound to any key, but I think it probably should > be. `C-x 4 l' seems like as good a keystroke as any. Any thought from > other people? I use the command quite a lot... I think the "other window" version should simply be a separate command, like find-function-other-window, xref-find-definitions-other-window, etc. Having a default binding for it would be a bit odd, considering find-library doesn't have one. >> 2. I think that the other-window version is far more useful than >> the same-window version. So if other-window behavior is available >> only via a prefix arg I'd prefer that it be the other way around: >> a prefix arg uses the same window; no prefix arg uses another >> window. (Yes, that would be a change in the default behavior.) -1 from me. > The command now uses pop-to-buffer-same-window, which is quite > customisable, as I understand things. (I haven't looked into the > new-fangled window popping controlling functions.) So I think this > should be customisable ... somehow ... by the users now. It's... not as easy as one might hope. And if you customize `find-library' to use the other window this way, there will be no way to have it use the current window just this one time. And vice versa. ^ permalink raw reply [flat|nested] 13+ messages in thread
* bug#2270: [PATCH] bug#2270, RE: 23.0.90; find-library:... (2) other-window version 2016-04-27 17:43 ` Dmitry Gutov @ 2016-04-27 17:56 ` Lars Ingebrigtsen 2016-04-27 18:57 ` Dmitry Gutov 0 siblings, 1 reply; 13+ messages in thread From: Lars Ingebrigtsen @ 2016-04-27 17:56 UTC (permalink / raw) To: Dmitry Gutov; +Cc: 2270, Chong Yidong, Stefan Monnier, 2270 Dmitry Gutov <dgutov@yandex.ru> writes: > I think the "other window" version should simply be a separate > command, like find-function-other-window, > xref-find-definitions-other-window, etc. Oh, I had gotten the impression that we're moving away from that? I though that was part of the reason for those... popping customisation thingies that I haven't paid all that much attention to. :-) I can add an -other-window version -- it would just call the current version with the new optional parameter... -- (domestic pets only, the antidote for overdose, milk.) bloggy blog: http://lars.ingebrigtsen.no ^ permalink raw reply [flat|nested] 13+ messages in thread
* bug#2270: [PATCH] bug#2270, RE: 23.0.90; find-library:... (2) other-window version 2016-04-27 17:56 ` Lars Ingebrigtsen @ 2016-04-27 18:57 ` Dmitry Gutov 0 siblings, 0 replies; 13+ messages in thread From: Dmitry Gutov @ 2016-04-27 18:57 UTC (permalink / raw) To: Lars Ingebrigtsen; +Cc: 2270, Chong Yidong, Stefan Monnier, 2270 On 04/27/2016 08:56 PM, Lars Ingebrigtsen wrote: > Oh, I had gotten the impression that we're moving away from that? I > though that was part of the reason for those... popping customisation > thingies that I haven't paid all that much attention to. :-) IIUC we were moving toward prefix commands, but not the way you've used current-prefix-arg here. ^ permalink raw reply [flat|nested] 13+ messages in thread
* bug#2270: [PATCH] bug#2270, RE: 23.0.90; find-library:... (2) other-window version 2016-04-27 17:34 ` Drew Adams 2016-04-27 17:38 ` Lars Ingebrigtsen @ 2016-04-27 18:46 ` Stefan Monnier 1 sibling, 0 replies; 13+ messages in thread From: Stefan Monnier @ 2016-04-27 18:46 UTC (permalink / raw) To: Drew Adams; +Cc: Lars Ingebrigtsen, 2270 > 1. I would still prefer to have the other-window version bound to > `C-x 4 l'. Actually, using the `other-frame-window' package from GNU ELPA, you could have exactly that without any extra code. > 2. I think that the other-window version is far more useful than > the same-window version. So if other-window behavior is available > only via a prefix arg I'd prefer that it be the other way around: > a prefix arg uses the same window; no prefix arg uses another > window. (Yes, that would be a change in the default behavior.) Indeed, the `other-frame-window' also needs to be expanded to provide a new prefix which means "use same frame" or "use same window". Stefan ^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2016-04-27 18:57 UTC | newest]
Thread overview: 13+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-02-11 2:27 bug#2270: 23.0.90; find-library: (1) doc string, (2) other-window version Chong Yidong
2012-10-24 17:41 ` bug#2270: [PATCH] bug#2270, RE: 23.0.90; find-library:... " Drew Adams
[not found] ` <jwv3913g142.fsf-monnier+emacs@gnu.org>
2012-10-24 19:38 ` Drew Adams
2012-10-24 20:42 ` Stefan Monnier
2012-10-24 21:20 ` Stefan Monnier
2012-10-24 21:25 ` Drew Adams
2016-04-27 17:26 ` Lars Ingebrigtsen
2016-04-27 17:34 ` Drew Adams
2016-04-27 17:38 ` Lars Ingebrigtsen
2016-04-27 17:43 ` Dmitry Gutov
2016-04-27 17:56 ` Lars Ingebrigtsen
2016-04-27 18:57 ` Dmitry Gutov
2016-04-27 18:46 ` Stefan Monnier
Code repositories for project(s) associated with this public inbox https://git.savannah.gnu.org/cgit/emacs.git 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).