[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: 'struct window' cleanup #3
From: |
Eli Zaretskii |
Subject: |
Re: 'struct window' cleanup #3 |
Date: |
Wed, 27 Jun 2012 20:37:19 +0300 |
> From: Stefan Monnier <address@hidden>
> Date: Wed, 27 Jun 2012 09:32:15 -0400
> Cc: , address@hidden
>
> > /* Right after splitting windows, last_point may be nil. */
> > && INTEGERP (w->last_point);
> > My tests never shows zero here, so I'm installing assertion instead.
>
> Since positions are strictly positive, you can use -1 as meaning "nil".
> (I recommend -1 over 0, because it's "more different").
I agree, but beware of those 'if (!w->last_point)' gotchas...
- Re: 'struct window' cleanup #2, (continued)
- Re: 'struct window' cleanup #2, Eli Zaretskii, 2012/06/26
- Re: 'struct window' cleanup #2, Dmitry Antipov, 2012/06/26
- Re: 'struct window' cleanup #2, Eli Zaretskii, 2012/06/26
- Re: 'struct window' cleanup #2, Stefan Monnier, 2012/06/26
- Re: 'struct window' cleanup #2, Eli Zaretskii, 2012/06/26
- 'struct window' cleanup #3, Dmitry Antipov, 2012/06/27
- Re: 'struct window' cleanup #3, Stefan Monnier, 2012/06/27
- Re: 'struct window' cleanup #3,
Eli Zaretskii <=
- Re: 'struct window' cleanup #3, Eli Zaretskii, 2012/06/27
- Re: 'struct window' cleanup #3, Dmitry Antipov, 2012/06/27
- Re: 'struct window' cleanup #3, Eli Zaretskii, 2012/06/27
- Re: 'struct window' cleanup #3, Dmitry Antipov, 2012/06/28
- Re: 'struct window' cleanup #2, martin rudalics, 2012/06/27
- Re: 'struct window' cleanup #2, Eli Zaretskii, 2012/06/27
Re: 'struct window' cleanup #2, Eli Zaretskii, 2012/06/25