[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: skeleton.el _ versus @, a new patch
From: |
Joe Kelsey |
Subject: |
Re: skeleton.el _ versus @, a new patch |
Date: |
22 Apr 2003 17:27:58 -0700 |
On Tue, 2003-04-22 at 06:31, Stefan Monnier wrote:
> > > > Here is a patch to add - as an alternate skeleton character. - operates
> > > > exactly as _ in setting skeleton-point, but does not interact with the
> > > > region wrapping effects. This means that @ is completely unaffected and
> > > > remains as just the method of setting the skeleton-positions.
> > > >
> > > > --- skeleton.el.orig Sat Jul 14 04:21:08 2001
> > > > +++ skeleton.el Sun Apr 20 15:44:44 2003
> > > > @@ -453,7 +453,8 @@
> > > > ((eq element '|)
> > > > (unless skeleton-modified (pop skeleton)))
> > > > ((eq element '@)
> > > > - (push (point) skeleton-positions)
> > > > + (push (point) skeleton-positions))
> > > > + ((eq element '-)
> > > > (unless skeleton-point (setq skeleton-point (point))))
> > > > ((eq 'quote (car-safe element))
> > > > (eval (nth 1 element)))
> > >
> > > I think in that case, we should set skeleton-point unconditionally.
> >
> > You *really* only want to set it on the first occurrence of the
> > appropriate character, not the last!
>
> I don't see why anyone would want to put more than 1 such operator
> in a skeleton anyway. So reason why I want to make it unconditional
> is so that you can do (nil "do {" \n _ \n "} while (" - ")") and be
> sure that point will always end up between the parens (I wouldn't
> want this behavior, personally, but given such a skeleton why would
> the author have put the `-' if it's not to specify where to place
> point).
I now understand.
- Re: skeleton.el _ versus @, (continued)
- Re: skeleton.el _ versus @, Richard Stallman, 2003/04/13
- Re: skeleton.el _ versus @, Stefan Monnier, 2003/04/13
- Re: skeleton.el _ versus @, Kai Großjohann, 2003/04/13
- Re: skeleton.el _ versus @, Joe Kelsey, 2003/04/13
- Re: skeleton.el _ versus @, a new patch, Joe Kelsey, 2003/04/20
- Re: skeleton.el _ versus @, a new patch, Stefan Monnier, 2003/04/21
- Re: skeleton.el _ versus @, a new patch, Joe Kelsey, 2003/04/21
- Re: skeleton.el _ versus @, a new patch, Stefan Monnier, 2003/04/22
- Re: skeleton.el _ versus @, a new patch,
Joe Kelsey <=
- Re: skeleton.el _ versus @, a new patch, Richard Stallman, 2003/04/21
- Re: skeleton.el _ versus @, a new patch, Joe Kelsey, 2003/04/21
- Re: skeleton.el _ versus @, a new patch, Richard Stallman, 2003/04/23
- Re: skeleton.el _ versus @, a new patch, Joe Kelsey, 2003/04/24
- Re: skeleton.el _ versus @, a new patch, Richard Stallman, 2003/04/25
- Re: skeleton.el _ versus @, a new patch, Stefan Monnier, 2003/04/28
- Re: skeleton.el _ versus @, a new patch, Richard Stallman, 2003/04/29
Re: skeleton.el _ versus @, Richard Stallman, 2003/04/02