Hi,

On Tue, Sep 8, 2015 at 8:30 PM, h_east <[email protected]> wrote:
> Hi Tony and Yegappan!
>
> 2015-9-9(Wed) 11:24:19 UTC+9 Tony Mechelynck:
>> On Wednesday, September 9, 2015 at 3:48:26 AM UTC+2, [email protected] 
>> wrote:
>> > Hi,
>> >
>> > The patch to fix the build problems with the tiny build is attached.
>> >
>> > - Yegappan
>> >
>>
>> With this patch, the tiny build succeeds, with the following warning:
>>
>> gcc -c -I. -Iproto -DHAVE_CONFIG_H     -O2 -fno-strength-reduce -Wall 
>> -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1        -o objects/move.o move.c
>> move.c: In function ‘scroll_cursor_top’:
>> move.c:1775:7: warning: ‘i’ may be used uninitialized in this function 
>> [-Wmaybe-uninitialized]
>>   used += i;
>>        ^
>
> I think src/move.c may be modified as below.
> What do you think?
>

Yes. I agree. Your change is the correct fix for this build issue
with move.c.

- Yegappan

>
> ********
> diff -r f717d96a39b3 src/move.c
> --- a/src/move.c        Tue Sep 08 23:45:05 2015 +0200
> +++ b/src/move.c        Wed Sep 09 12:23:54 2015 +0900
> @@ -1769,7 +1769,11 @@
>             i = 1;
>         else
>  #endif
> +#ifdef FEAT_DIFF
>             i = plines_nofill(top);
> +#else
> +           i = plines(top);
> +#endif
>         used += i;
>         if (extra + i <= off && bot < curbuf->b_ml.ml_line_count)
>         {
>
> ********
>
>
> --
> Best regards,
> Hirohito Higashi (a.k.a h_east)
>

-- 
-- 
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

--- 
You received this message because you are subscribed to the Google Groups 
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Raspunde prin e-mail lui