Hi Bram and list,

2016-1-26(Tus) 0:41:48 UTC+9 h_east:
> Hi Bram,
> 
> 2016-1-24(Sun) 23:05:57 UTC+9 Bram Moolenaar:
> > Hirohito Higashi wrote:
> > 
> > [...]
> > 
> > > > I think we need to split this up in pieces anyway, otherwise the patch
> > > > is too big to review.  I don't think there is a reason to remove __ARGS
> > > > at the same time we update the functions themselves.  And updating the
> > > > functions is a bit more tricky (e.g. adding (void) when there are no
> > > > arguments), might want to do that one file at a time.
> > > > 
> > > > A script for this would be very welcome!
> > > 
> > > Done.
> > > I attached Vim script file .
> > > All function declarations convert to ANSI C style, except for the
> > > following 9 functions.
> > > 
> > > add_menu_path() [menu.c]
> > > do_one_cmd() [ex_docmd.c]
> > > ff_check_visited() [misc2.c]
> > > ff_create_stack_element() [misc2.c]
> > > otherfile_buf() [buffer.c]
> > > parse_tag_line() [tag.c]
> > > screen_line() [screen.c]
> > > test_for_current() [tag.c]
> > > vim_vsnprintf() [message.c]
> > > 
> > > Please convert manually these functions :-)
> > > 
> > > I confirmed in Vim 7.4.1161 on fwdora Linux.
> > > 
> > > How to run the script:
> > >  1. Place the eliminate__ARGS.vim to the hierarchy on one of the vim 
> > > directory.
> > >  2. Run
> > >    $ cd vim
> > >    $ vim -Nu NONE -S ../eliminate__ARGS.vim
> > > 
> > > NOTE: 
> > > It takes considerable time to the execution of the script.
> > > It took 11 minutes and that's the desktop that I have purchased eight
> > > years ago.
> > > 
> > > Please check it.
> > 
> > Thanks!  I'll look into this over the coming week.
> 
> Script update.  (A bit refactor)
> It works fine on Vim 7.4.1176.

Update Vim script attached.
I validated all source code (maybe).

Fixes:
- It also will process if there is a preprocessor directive in K&R room.
- Add target to src/if_perl.xs.
- ...

Manual correction is necessary is the following 16 functions.

static char *e (p, i)    [src/auto/configure : 3353]
fgets_cr(s, n, stream)    [src/ex_cmds2.c : 3717]
aucmd_prepbuf(aco, buf)    [src/fileio.c : 10071]
aucmd_restbuf(aco)    [src/fileio.c : 10087]
fontset_descent(fs)    [src/gui_x11.c : 2253]
get_x11_title(test_only)    [src/os_unix.c : 2066]
get_x11_icon(test_only)    [src/os_unix.c : 2073]
mch_get_host_name(s, len)    [src/os_unix.c : 2373]
mch_copy_sec(from_file, to_file)    [src/os_unix.c : 2792]
main(argc, argv)    [src/po/sjiscorr.c : 10]
OpenPTY(ttyn)    [src/pty.c : 201]
OpenPTY(ttyn)    [src/pty.c : 226]
OpenPTY(ttyn)    [src/pty.c : 252]
OpenPTY(ttyn)    [src/pty.c : 280]
OpenPTY(ttyn)    [src/pty.c : 322]
OpenPTY(ttyn)    [src/pty.c : 369]

Thanks.
--
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.

Attachment: eliminate__ARGS.vim
Description: Binary data

Raspunde prin e-mail lui