Hi, 2016/1/23 Sat 2:33:18 UTC+9 Bram Moolenaar wrote: > 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!
If we assume C89, we can also assume that we can use stdarg.h, I think. How about removing '#ifdef HAVE_STDARG_H' checks? Regards, Ken Takata -- -- 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.
