[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 4/7] emacs: Add 'guix-devel-use-current-module'.
From: |
Alex Kost |
Subject: |
Re: [PATCH 4/7] emacs: Add 'guix-devel-use-current-module'. |
Date: |
Sun, 26 Jul 2015 21:11:54 +0300 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Mathieu Lirzin (2015-07-26 17:00 +0300) wrote:
> Alex Kost <address@hidden> writes:
>
>> Mathieu Lirzin (2015-07-25 21:51 +0300) wrote:
>>> What do you think about integrating it in section "Contributing" instead?
>>
>> Maybe, I don't have a preference here.
>
> Since it's not really related to package management, I think it would be
> preferable to move this in the node "The perfect Setup".
OK, you convinced me :-) I'll put the docs there.
>>> I remember reading somewhere in the Emacs manual that "C-c <Letter>" is
>>> meant for user customization in the global key map (can't remember where
>>> ;)). So what suggesting "C-c C-u" which is a mode specific key binding?
>>
>> I don't know whether there is such (or any other) suggestion anywhere.
>> But since that was just an example, not a recommendation (not even a
>> suggestion), I think it doesn't matter what key binding to use there.
>> Do you have any preference? What about "C-H-M-s-u"? :-)
>
> I agree it's not really important for advanced Emacs users who will have
> no problem choosing something adapted to their setup. Nonetheless IMHO
> consistency and simplicity helps newcomers to understand, So I will be
> in favour of anything that follows this convention. :)
>
> (global-set-key (kbd "C-c <letter>") foo-function-callable-anywhere)
>
> (define-key bar-mode-map
> (kbd "C-c C-<letter>")
> 'foo-function-for-bar-mode)
According to (info "(elisp) Key Binding Conventions"), 'C-c LETTER'
bindings are reserved for users, so I think using "C-c u" in that
example is absolutely fine, don't you agree?
--
Alex
[PATCH 3/7] emacs: Add code to call guile procedures., Alex Kost, 2015/07/25
[PATCH 5/7] ui: Add 'show-derivation-outputs'., Alex Kost, 2015/07/25
[PATCH 6/7] guix build: Add 'build-package'., Alex Kost, 2015/07/25
[PATCH 7/7] emacs: Add 'guix-devel-use-current-module'., Alex Kost, 2015/07/25