Mercurial > vim
view src/tee/tee.c @ 34485:157cf882799f v9.1.0150
patch 9.1.0150: Several minor 'winfixbuf' issues
Commit: https://github.com/vim/vim/commit/4bb505e28cac0389561fff78d8bbe0319c2bcf2f
Author: Sean Dewar <6256228+seandewar@users.noreply.github.com>
Date: Tue Mar 5 20:39:07 2024 +0100
patch 9.1.0150: Several minor 'winfixbuf' issues
Problem: several minor 'winfixbuf' issues exist, mostly relating to the
quickfix list
Solution: address them and adjust tests. Retab and reflow a few things too.
(Sean Dewar)
Things touched include:
- Replace the semsgs with gettext'd emsgs.
- Handle window switching in ex_listdo properly, so curbuf and curwin
are kept in-sync and trigger autocommands; handle those properly.
- Don't change the list entry index in qf_jump_edit_buffer if we fail
due to 'wfb' (achieved by returning FAIL; QF_ABORT should only be used
if the list was changed).
- Make qf_jump_edit_buffer actually switch to prevwin when using `:cXX`
commands **outside** of the list window if 'wfb' is set in curwin.
Handle autocommands properly in case they mess with the list.
NOTE: previously, it seemed to split if 'wfb' was set, but do nothing
and fail if prevwin is *valid*. This behaviour seemed strange, and maybe
unintentional? Now it aligns more with what's described for the `:cXX`
commands in the original PR description when used outside a list window,
I think.
- In both functions, only consider prevwin if 'wfb' isn't set for it;
fallback to splitting otherwise.
- Use win_split to split. Not sure if there was a specific reason for
using ex_splitview. win_split is simpler and respects modifiers like
:vertical that may have been used. Plus, its return value can be checked
for setting opened_window in qf code (technically win_split_ins autocmds
could immediately close it or change windows, in which the qf code might
close some other window on failure; it's already the case elsewhere,
though).
closes: #14142
Signed-off-by: Sean Dewar <6256228+seandewar@users.noreply.github.com>
Signed-off-by: Christian Brabandt <cb@256bit.org>
author | Christian Brabandt <cb@256bit.org> |
---|---|
date | Tue, 05 Mar 2024 20:45:04 +0100 |
parents | 1fe2d79f7309 |
children |
line wrap: on
line source
/* vim:set ts=4 sw=4: * * Copyright (c) 1996, Paul Slootman * * Author: Paul Slootman * (paul@wurtel.hobby.nl, paul@murphy.nl, paulS@toecompst.nl) * Modifications for MSVC: Yasuhiro Matsumoto * * This source code is released into the public domain. It is provided on an * as-is basis and no responsibility is accepted for its failure to perform * as expected. It is worth at least as much as you paid for it! * * tee.c - pipe fitting * * tee reads stdin, and writes what it reads to each of the specified * files. The primary reason of existence for this version is a quick * and dirty implementation to distribute with Vim, to make one of the * most useful features of Vim possible on OS/2: quickfix. * * Of course, not using tee but instead redirecting make's output directly * into a temp file and then processing that is possible, but if we have a * system capable of correctly piping (unlike DOS, for example), why not * use it as well as possible? This tee should also work on other systems, * but it's not been tested there, only on OS/2. * * tee is also available in the GNU shellutils package, which is available * precompiled for OS/2. That one probably works better. */ #ifndef _MSC_VER # include <unistd.h> #endif #include <malloc.h> #include <stdio.h> #include <stdlib.h> #include <string.h> #include <fcntl.h> #ifdef _WIN32 # define sysconf(x) -1 #endif void usage(void) { fprintf(stderr, "tee usage:\n\ \ttee [-a] file ... file_n\n\ \n\ \t-a\tappend to files instead of truncating\n\ \nTee reads its input, and writes to each of the specified files,\n\ as well as to the standard output.\n\ \n\ This version supplied with Vim 4.2 to make ':make' possible.\n\ For a more complete and stable version, consider getting\n\ [a port of] the GNU shellutils package.\n\ "); } /* * fread only returns when count is read or at EOF. * We could use fgets, but I want to be able to handle binary blubber. */ int myfread(char *buf, int elsize /*ignored*/, int max, FILE *fp) { int c; int n = 0; while ((n < max) && ((c = getchar()) != EOF)) { *(buf++) = c; n++; if (c == '\n' || c == '\r') break; } return n; } int main(int argc, char *argv[]) { int append = 0; size_t numfiles; int maxfiles; FILE **filepointers; int i; char buf[BUFSIZ]; int n; int optind = 1; for (i = 1; i < argc; i++) { if (argv[i][0] != '-') break; if (!strcmp(argv[i], "-a")) append++; else usage(); optind++; } numfiles = argc - optind; if (numfiles == 0) { fprintf(stderr, "doesn't make much sense using tee without any file name arguments...\n"); usage(); exit(2); } maxfiles = sysconf(_SC_OPEN_MAX); /* or fill in 10 or so */ if (maxfiles < 0) maxfiles = 10; if (numfiles + 3 > maxfiles) /* +3 accounts for stdin, out, err */ { fprintf(stderr, "Sorry, there is a limit of max %d files.\n", maxfiles - 3); exit(1); } filepointers = calloc(numfiles, sizeof(FILE *)); if (filepointers == NULL) { fprintf(stderr, "Error allocating memory for %ld files\n", (long)numfiles); exit(1); } for (i = 0; i < numfiles; i++) { filepointers[i] = fopen(argv[i+optind], append ? "ab" : "wb"); if (filepointers[i] == NULL) { fprintf(stderr, "Can't open \"%s\"\n", argv[i+optind]); exit(1); } } #ifdef _WIN32 setmode(fileno(stdin), O_BINARY); fflush(stdout); /* needed for _fsetmode(stdout) */ setmode(fileno(stdout), O_BINARY); #endif while ((n = myfread(buf, sizeof(char), sizeof(buf), stdin)) > 0) { fwrite(buf, sizeof(char), n, stdout); fflush(stdout); for (i = 0; i < numfiles; i++) { if (filepointers[i] && fwrite(buf, sizeof(char), n, filepointers[i]) != n) { fprintf(stderr, "Error writing to file \"%s\"\n", argv[i+optind]); fclose(filepointers[i]); filepointers[i] = NULL; } } } for (i = 0; i < numfiles; i++) { if (filepointers[i]) fclose(filepointers[i]); } exit(0); }