Hi Bram and list!

2016-1-23(Sat) 2:33:18 UTC+9 Bram Moolenaar:
> Hirohito Higashi wrote:
> 
> > Hi Bram!
> > 
> > 2016-1-22(Fri) 4:24:57 UTC+9 Bram Moolenaar:
> > > Hirohito Higashi wrote:
> > > 
> > > > 2016-1-21(Thu) 6:48:27 UTC+9 Bram Moolenaar:
> > > > > Hirohito Higashi wrote:
> > > > > 
> > > > > > Hi Bram and Vimmers!
> > > > > > 
> > > > > > 2016-1-20(Wed) 3:00:59 UTC+9 Bram Moolenaar:
> > > > > > > Hirohito Higashi wrote:
> > > > > > > 
> > > > > > > > > Sorry for the huge patch!
> > > > > > > > > 
> > > > > > > > > Let me know if there is someone with an old compiler that 
> > > > > > > > > can't handle
> > > > > > > > > this ANSI style code.  If not then I'll continue moving 
> > > > > > > > > forward with
> > > > > > > > > this, making C89 the minimal required C standard.
> > > > > > > > 
> > > > > > > > I've eliminated the __ARGS.
> > > > > > > 
> > > > > > > Thanks.  I'll wait a bit to make sure removing __ARGS doesn't 
> > > > > > > cause
> > > > > > > problems for anyone.
> > > > > > > 
> > > > > > > I suppose you used a Vim command or script?
> > > > > > 
> > > > > > I mainly used complex repeats (:help complex-repeat) and :cfdo
> > > > > > It can be processed as 15 minutes.
> > > > > 
> > > > > Perhaps you can give me this script, instead of updating the big patch
> > > > > every time a file changes around __ARGS?
> > > > 
> > > > I attached a Vim script.
> > > > Usage:
> > > > - Place the eliminate__ARGS.vim to the hierarchy on one of the vim 
> > > > directory.
> > > > - execute following command.
> > > >   $ cd vim
> > > >   $ vim -Nu NONE -S ../eliminate__ARGS.vim
> > > > 
> > > > > One thing I would change: No space between the function name and the
> > > > > following (.
> > > > 
> > > > Yeah, I think so.  Attached script support it.
> > > > 
> > > > > 
> > > > > > > > Unfortunately, I can not fix src/osdef.sh
> > > > > > > > Please fix this file.
> > > > > > > > 
> > > > > > > > Otherwise, I have succeeded to build and test ALL DONE with the
> > > > > > > > auto/osdef.h before the modification on Linux.  
> > > > > > > > (src/auto/osdef.h is
> > > > > > > > created by src/osdef.sh)
> > > > > > > > 
> > > > > > > > I need a help!
> > > > > > > 
> > > > > > > Changing those sed commands is tricky...
> > > > > > 
> > > > > > Update a patch.
> > > > > > I got it.  (I used a work-around :-)
> > > > > > Please see following files.
> > > > > >   src/osdef.sh
> > > > > >   src/osdef1.h.in
> > > > > >   src/osdef2.h.in
> > > > > > 
> > > > > > 
> > > > > > I confirmed in fedora Linux.
> > > > > > 
> > > > > > Someone confirm in the other OS please.
> > > > > 
> > > > > I see there is still __ARGS in the input file.  It would work, but 
> > > > > it's
> > > > > nicer if we can drop it.
> > > > 
> > > > Maybe done :-)
> > > 
> > > Thanks. Unless someone complains about the removal of __ARGS soon I'll
> > > probably make the change this weekend.
> > 
> > Please 48 hours from now.
> > I try to make ANSI C conversion script of the function declaration.
> 
> 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.

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