Looks pretty good thanks but i probably won't be able to try it until tomorrow.
On Mon, Feb 15, 2010 at 12:13:15PM -0800, Micah Cowan wrote:
> Nicholas Marriott wrote:
> > This looks fine apart from a couple of things:
> >
> > - Why is the count a u_long? I don't see a need for it here, let's jus
Nicholas Marriott wrote:
> This looks fine apart from a couple of things:
>
> - Why is the count a u_long? I don't see a need for it here, let's just use a
> u_int. Types that commonly change size depending on arch are stupid. In
> fact,
> I might go to far as to say we should put in a hard l
Either loop round the entire switch if possible or duplicate the loop in each
one (I would move the ?: out into a temporary variable in that case).
On Mon, Feb 15, 2010 at 02:06:58AM -0800, Micah Cowan wrote:
> Nicholas Marriott wrote:
> > I'll look at all your diffs in detail later tonight but I
This looks fine apart from a couple of things:
- Why is the count a u_long? I don't see a need for it here, let's just use a
u_int. Types that commonly change size depending on arch are stupid. In fact,
I might go to far as to say we should put in a hard limit of 1000 repeats or
something.
Nicholas Marriott wrote:
> I'll look at all your diffs in detail later tonight but I can tell you now
> there is no way a macro like this is going in :-).
Better suggestion, then? ...I could dupe the for-loop a buncha times,
but the macro looks cleaner to my eyes.
Or did you just mean the first,
I'll look at all your diffs in detail later tonight but I can tell you now
there is no way a macro like this is going in :-).
On Mon, Feb 15, 2010 at 12:32:24AM -0800, Micah Cowan wrote:
> > +/* Loop while there's a prefix (or 1, if there isn't one).
> > + * Requires the variable "data", pointing
> +/* Loop while there's a prefix (or 1, if there isn't one).
> + * Requires the variable "data", pointing at an instance of
> + * struct window_copy_mode_data. */
> +#define REPEAT_NUMPREFIX_TIMES \
> + if (data->numprefix == 0) data->numprefix++; \
> + while (data->numprefix--)
Muc
This patch allows you to supply repeat prefixes to commands in
copy-mode. For instance, to move forward 10 characters, you'd type ‘10l’
("ten ell") in vim, and ‘M-1 0 C-f´ in emacs (of course, you could use
the right arrow-key instead of ell or C-f).
The prefix keys are actually bound commands (to
