Date: Fri, 12 Feb 2010 21:08:23 +0100 From: [email protected]
`dolist' needs a temporary variable to bind each list element to.
Well, sure, but `dolist' does the binding for you: [...]
Huh, I didn't know that. I've taken appropriate action and removed all such redundant `let' bindings in changeset [82].
;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;; ;;;;;;;;;;;;; VISUAL MODE BINDINGS ;;;;;;;;;;;;;;;; ;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;; (define-key viper-vi-basic-map "v" 'vimpulse-visual-mode) (define-key viper-vi-basic-map "V" 'vimpulse-visual-mode-linewise)
My point is that the above lines do not belong *anywhere*, they're just redundant; [...]
Yes, I removed them in changeset [81]. Sorry for being unclear. http://code.assembla.com/vimpulse/subversion/changesets/81
Here's a renaming proposal:
vimpulse-dependencies.el => vimpulse-depend.el vimpulse-misc-keybindings.el => vimpulse-keys.el vimpulse-paren-matching.el => vimpulse-paren.el vimpulse-text-object-system.el => vimpulse-text.el vimpulse-viper-function-redefinitions.el => vimpulse-viper.el vimpulse-visual-mode.el => vimpulse-visual.el
I don't think that is really necessary -- for one thing, the current file names describe their contents better than the shorter alternatives you propose, and besides that, it's all just kinda behind-the-scenes -- most users don't even know about the single files, they're using the concatenated vimpulse.el.
Yes, it doesn't make much difference as of currently, since Vimpulse is released as a single file on EmacsWiki. In the future, however, it may bundle some sort of Info manual. Maybe we will have two downloads then: a single big file on EmacsWiki, and a compressed archive containing the .el files (separately?), the manual, and some installation scripts. In that case, conventionally short names would be ... well, it would be conventional. :)
It's a PITA for me to look at ten files instead of one, esp. when some of them really only contain a few lines.
Yeah. It is sometimes useful for debugging, though. Vegard _________________________________________________________________ Windows Live™ Mail. Flere kontoer på ett sted. http://download.live.com/wlmail