Re: Emacs-diffs Digest, Vol 94, Issue 89

[Eli Zaretskii]

> === modified file 'src/scroll.c'
> --- a/src/scroll.c    2010-07-12 17:47:17 +0000
> +++ b/src/scroll.c    2010-09-24 15:20:58 +0000
> @@ -94,7 +94,7 @@
>                    int free_at_end)
>  {
>    register int i, j;
> -  int frame_lines = FRAME_LINES (frame);
> +  EMACS_INT frame_lines = FRAME_LINES (frame);

Is this really a good idea?  There's no chance the number of lines in
a frame will overflow a 32-bit int any time soon.  SO I think an
explicit cast to int is a better solution here.

(There are other similar places in this revision.)