Hi JamesVega, welcome to the Vim Tips wiki! Thanks for your edit to the Insert current date or time page. It's always great to see new contributors. This is an automated message; somebody may drop a message for you here in the next day or two with any comments or suggestions, but until then keep up the good work, and don't be afraid to be bold!
Hi, JamesVega. I noticed your recent edits, and just had a couple of notes.
First, I agree that the comments in Insert current date or time were unnecessary. However, we like to keep the comments section on every tip of the wiki. We find it encourages contributions, and additionally we tend to avoid the talk pages of tips in most cases (there have been a very few notable exceptions). Somewhere in our policy pages we discuss this, or you can talk with John.
Second, and this is more of a side note, I'm not sure what your edit comment that the mappings you deleted "needlessly use ':execute'" meant. I didn't see a way to make these particular mappings work without execute. It's an irrelevant point now, since they've been deleted in favor of using the expression register as in the tip proper, but I was curious what you were getting at.
Finally, welcome! I see you've gotten off to a good start. Keep being bold and we'll give you pointers along the way.
--Fritzophrenic 22:06, February 4, 2010 (UTC)
- The maps that I removed would have been better off using the expression register (or an <expr> map), like the very similar maps near the top of the tip. This is especially true for the imap. <Esc>:execute "normal i" . strftime(...)<CR> is overkill when you can do <C-r>=strftime(...)<CR>. Also, in the general sense, the use of :execute is better off being avoided when there are other alternatives. Using :execute means you have to be more careful about the expression you're evaluating and the context in which you're doing so to avoid potential side effects from improperly escaped input.
- --JamesVega 02:33, February 5, 2010 (UTC)
- Ok, that makes sense. Thanks for clarifying! --Fritzophrenic 15:00, February 5, 2010 (UTC)