To: vim_dev@googlegroups.com Subject: Patch 7.4.1764 Fcc: outbox From: Bram Moolenaar Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ------------ Patch 7.4.1764 Problem: C++ style comment. (Ken Takata) Solution: Finish the work started here: don't call perror() when stderr isn't working. Files: src/os_unix.c *** ../vim-7.4.1763/src/os_unix.c 2016-04-20 20:18:16.957270339 +0200 --- src/os_unix.c 2016-04-21 12:48:06.922603703 +0200 *************** *** 5138,5144 **** if (pid == 0) { ! int null_fd = -1; /* child */ reset_signals(); /* handle signals normally */ --- 5138,5145 ---- if (pid == 0) { ! int null_fd = -1; ! int stderr_works = TRUE; /* child */ reset_signals(); /* handle signals normally */ *************** *** 5175,5180 **** --- 5176,5182 ---- { close(2); ignored = dup(null_fd); + stderr_works = FALSE; } else if (use_out_for_err) { *************** *** 5210,5216 **** /* See above for type of argv. */ execvp(argv[0], argv); ! // perror("executing job failed"); _exit(EXEC_FAILED); /* exec failed, return failure code */ } --- 5212,5219 ---- /* See above for type of argv. */ execvp(argv[0], argv); ! if (stderr_works) ! perror("executing job failed"); _exit(EXEC_FAILED); /* exec failed, return failure code */ } *** ../vim-7.4.1763/src/version.c 2016-04-21 09:20:17.787279918 +0200 --- src/version.c 2016-04-21 12:52:14.728034360 +0200 *************** *** 750,751 **** --- 750,753 ---- { /* Add new patch number below this line */ + /**/ + 1764, /**/ -- Q: Is selling software the same as selling hardware? A: No, good hardware is sold new, good software has already been used by many. /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net \\\ /// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\ \\\ an exciting new programming language -- http://www.Zimbu.org /// \\\ help me help AIDS victims -- http://ICCF-Holland.org ///