diff options
author | Michał Cichoń <michcic@gmail.com> | 2015-12-10 01:02:02 +0100 |
---|---|---|
committer | Michał Cichoń <michcic@gmail.com> | 2015-12-10 01:02:02 +0100 |
commit | 20c5fc1767436ee106156a9aabf6545e72a4b519 (patch) | |
tree | e47817b21f864dd68009c9fb812037ae8ac6d51a | |
parent | 81ef4f32ebb07b768a7cc4a8937a624cdcdc6f80 (diff) | |
download | pianobar-windows-20c5fc1767436ee106156a9aabf6545e72a4b519.tar.gz pianobar-windows-20c5fc1767436ee106156a9aabf6545e72a4b519.tar.bz2 pianobar-windows-20c5fc1767436ee106156a9aabf6545e72a4b519.zip |
Use custom function to feed console instead of trying to be clever and capturing stdout.
This fixes IO on Windows 8 and Windows 10.
-rw-r--r-- | src/console.c | 613 | ||||
-rw-r--r-- | src/console.h | 9 | ||||
-rw-r--r-- | src/main.c | 2 | ||||
-rw-r--r-- | src/ui.c | 14 | ||||
-rw-r--r-- | src/ui_act.c | 3 | ||||
-rw-r--r-- | src/ui_readline.c | 8 |
6 files changed, 330 insertions, 319 deletions
diff --git a/src/console.c b/src/console.c index b05f0a3..6fa0c79 100644 --- a/src/console.c +++ b/src/console.c @@ -1,6 +1,6 @@ /* Copyright (c) 2015 - Micha³ Cichoñ <thedmd@interia.pl> + Micha³ Cichoñ <thedmd@interia.pl> Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated documentation files (the "Software"), to deal @@ -29,392 +29,395 @@ THE SOFTWARE. #include <process.h> #include "vtparse/vtparse.h" -enum { READ = 0, WRITE }; - -# define BAR_IN_CAPACITY 1024 -# define BAR_OUT_CAPACITY 1024 +# define BAR_BUFFER_CAPACITY 1024 static const int BarTerminalToAttibColor[8] = { - 0 /* black */, 4 /* red */, 2 /* green */, 6 /* yellow */, - 1 /* blue */, 5 /* magenta */, 3 /* cyan */, 7 /* white */ + 0 /* black */, 4 /* red */, 2 /* green */, 6 /* yellow */, + 1 /* blue */, 5 /* magenta */, 3 /* cyan */, 7 /* white */ }; -static struct BarConsoleState { - HANDLE StdIn; - HANDLE StdOut; - - int Pipes[2]; - int OriginalStdOut; - int OriginalStdErr; - - WORD DefaultAttributes; - WORD CurrentAttributes; +static struct BarConsoleState +{ + HANDLE StdIn; + HANDLE StdOut; - HANDLE ConsoleThread; - bool Terminate; + WORD DefaultAttributes; + WORD CurrentAttributes; - vtparse_t Parser; + HANDLE ConsoleThread; + bool Terminate; - char InBuffer[BAR_IN_CAPACITY]; - size_t InBufferSize; + vtparse_t Parser; - char OutBuffer[BAR_OUT_CAPACITY]; - size_t OutBufferSize; + char Buffer[BAR_BUFFER_CAPACITY]; + size_t BufferSize; } g_BarConsole; -static inline void BarOutSetAttributes(WORD attributes) { - g_BarConsole.CurrentAttributes = attributes; - SetConsoleTextAttribute(CONSOLE_REAL_OUTPUT_HANDLE, g_BarConsole.CurrentAttributes); +static inline void BarOutSetAttributes(WORD attributes) +{ + g_BarConsole.CurrentAttributes = attributes; + SetConsoleTextAttribute(BarConsoleGetStdOut(), g_BarConsole.CurrentAttributes); } -static inline void BarOutFlush() { - if (g_BarConsole.OutBufferSize == 0) - return; +static void BarParseCallback(struct vtparse* parser, vtparse_action_t action, unsigned char ch) +{ + if (action == VTPARSE_ACTION_PRINT || action == VTPARSE_ACTION_EXECUTE) + { + putc(ch, stdout); + if (action == VTPARSE_ACTION_EXECUTE) + fflush(stdout); + } + + if (action == VTPARSE_ACTION_CSI_DISPATCH) + { + WORD attribute = g_BarConsole.CurrentAttributes; + int i; + + switch (ch) + { + case 'K': + BarConsoleEraseLine(parser->num_params > 0 ? parser->params[0] : 0); + break; + + case 'm': + for (i = 0; i < parser->num_params; ++i) + { + int p = parser->params[i]; + if (p == 0) + attribute = g_BarConsole.DefaultAttributes; + //else if (p == 1) + // attribute |= FOREGROUND_INTENSITY; + else if (p == 4) + attribute |= COMMON_LVB_UNDERSCORE; + else if (p == 7) + attribute |= COMMON_LVB_REVERSE_VIDEO; + //else if (p == 21) + // attribute &= ~FOREGROUND_INTENSITY; + else if (p == 24) + attribute &= ~COMMON_LVB_UNDERSCORE; + else if (p == 27) + attribute &= ~COMMON_LVB_REVERSE_VIDEO; + else if (p >= 30 && p <= 37) + attribute = (attribute & ~0x07) | BarTerminalToAttibColor[p - 30]; + else if (p >= 40 && p <= 47) + attribute = (attribute & ~0x70) | (BarTerminalToAttibColor[p - 40] << 4); + else if (p >= 90 && p <= 97) + attribute = (attribute & ~0x07) | BarTerminalToAttibColor[p - 90] | FOREGROUND_INTENSITY; + else if (p >= 100 && p <= 107) + attribute = ((attribute & ~0x70) | (BarTerminalToAttibColor[p - 100] << 4)) | BACKGROUND_INTENSITY; + } + fflush(stdout); + BarOutSetAttributes(attribute); + break; + } + } +} - _write(g_BarConsole.OriginalStdOut, g_BarConsole.OutBuffer, g_BarConsole.OutBufferSize); +void BarConsoleInit() +{ + unsigned threadId = 0; + CONSOLE_SCREEN_BUFFER_INFO csbi; + memset(&g_BarConsole, 0, sizeof(g_BarConsole)); - g_BarConsole.OutBufferSize = 0; -} + SetConsoleCP(CP_UTF8); + SetConsoleOutputCP(CP_UTF8); -static inline void BarOutPut(char c) { - if (g_BarConsole.OutBufferSize >= BAR_OUT_CAPACITY) - BarOutFlush(); + g_BarConsole.StdIn = GetStdHandle(STD_INPUT_HANDLE); + g_BarConsole.StdOut = GetStdHandle(STD_OUTPUT_HANDLE); - g_BarConsole.OutBuffer[g_BarConsole.OutBufferSize++] = c; -} + GetConsoleScreenBufferInfo(g_BarConsole.StdOut, &csbi); -static inline void BarOutPuts(const char* c) { - size_t length = strlen(c); - size_t i; + g_BarConsole.DefaultAttributes = csbi.wAttributes; + g_BarConsole.CurrentAttributes = csbi.wAttributes; - for (i = 0; i < length; ++i) - BarOutPut(c[i]); + vtparse_init(&g_BarConsole.Parser, BarParseCallback); } -static void BarParseCallback(struct vtparse* parser, vtparse_action_t action, unsigned char ch) { - if (action == VTPARSE_ACTION_PRINT || action == VTPARSE_ACTION_EXECUTE) - { - BarOutPut(ch); - if (action == VTPARSE_ACTION_EXECUTE) - BarOutFlush(); - } - - if (action == VTPARSE_ACTION_CSI_DISPATCH) - { - WORD attribute = g_BarConsole.CurrentAttributes; - int i; - - switch (ch) - { - case 'K': - BarConsoleEraseLine(parser->num_params > 0 ? parser->params[0] : 0); - break; - - case 'm': - for (i = 0; i < parser->num_params; ++i) { - int p = parser->params[i]; - if (p == 0) - attribute = g_BarConsole.DefaultAttributes; - //else if (p == 1) - // attribute |= FOREGROUND_INTENSITY; - else if (p == 4) - attribute |= COMMON_LVB_UNDERSCORE; - else if (p == 7) - attribute |= COMMON_LVB_REVERSE_VIDEO; - //else if (p == 21) - // attribute &= ~FOREGROUND_INTENSITY; - else if (p == 24) - attribute &= ~COMMON_LVB_UNDERSCORE; - else if (p == 27) - attribute &= ~COMMON_LVB_REVERSE_VIDEO; - else if (p >= 30 && p <= 37) - attribute = (attribute & ~0x07) | BarTerminalToAttibColor[p - 30]; - else if (p >= 40 && p <= 47) - attribute = (attribute & ~0x70) | (BarTerminalToAttibColor[p - 40] << 4); - else if (p >= 90 && p <= 97) - attribute = (attribute & ~0x07) | BarTerminalToAttibColor[p - 90] | FOREGROUND_INTENSITY; - else if (p >= 100 && p <= 107) - attribute = ((attribute & ~0x70) | (BarTerminalToAttibColor[p - 100] << 4)) | BACKGROUND_INTENSITY; - } - BarOutFlush(); - BarOutSetAttributes(attribute); - break; - } - } +void BarConsoleDestroy() +{ } -static unsigned __stdcall BarConsoleThread(void* args) { - - while (!g_BarConsole.Terminate) { +HANDLE BarConsoleGetStdIn() +{ + return g_BarConsole.StdIn; +} - int bytes = _read(g_BarConsole.Pipes[READ], g_BarConsole.InBuffer, BAR_IN_CAPACITY); +HANDLE BarConsoleGetStdOut() +{ + return g_BarConsole.StdOut; +} - if (bytes > 0) - vtparse(&g_BarConsole.Parser, g_BarConsole.InBuffer, bytes); +void BarConsoleSetTitle(const char* title) +{ + size_t len = MultiByteToWideChar(CP_UTF8, 0, title, -1, NULL, 0); - BarOutFlush(); - } + TCHAR* wTitle = malloc((len + 1) * sizeof(TCHAR)); + if (NULL != wTitle) + { + MultiByteToWideChar(CP_UTF8, 0, title, -1, wTitle, len); + SetConsoleTitleW(wTitle); - return 0; + free(wTitle); + } + else + SetConsoleTitleA(title); } -void BarConsoleInit() { - unsigned threadId = 0; - CONSOLE_SCREEN_BUFFER_INFO csbi; - memset(&g_BarConsole, 0, sizeof(g_BarConsole)); +void BarConsoleSetSize(int width, int height) +{ + HANDLE handle; + SMALL_RECT r; + COORD c, s; + CONSOLE_SCREEN_BUFFER_INFO csbi; - SetConsoleCP(CP_UTF8); - SetConsoleOutputCP(CP_UTF8); + handle = BarConsoleGetStdOut(); - g_BarConsole.StdIn = GetStdHandle(STD_INPUT_HANDLE); - g_BarConsole.StdOut = GetStdHandle(STD_OUTPUT_HANDLE); + if (!GetConsoleScreenBufferInfo(handle, &csbi)) + return; - GetConsoleScreenBufferInfo(g_BarConsole.StdOut, &csbi); + s.X = csbi.srWindow.Right - csbi.srWindow.Left + 1; + s.Y = csbi.srWindow.Bottom - csbi.srWindow.Top + 1; - g_BarConsole.DefaultAttributes = csbi.wAttributes; - g_BarConsole.CurrentAttributes = csbi.wAttributes; + if (s.X > width && s.Y > height) + return; - if (_pipe(g_BarConsole.Pipes, 65536, O_BINARY) != -1) { - g_BarConsole.OriginalStdOut = _dup(_fileno(stdout)); - g_BarConsole.OriginalStdErr = _dup(_fileno(stderr)); + c.X = width; + c.Y = height; - fflush(stdout); - fflush(stderr); + if (s.X > c.X) + c.X = s.X; + if (s.Y > c.Y) + c.Y = s.Y; - dup2(g_BarConsole.Pipes[WRITE], fileno(stdout)); - dup2(g_BarConsole.Pipes[WRITE], fileno(stderr)); + SetConsoleScreenBufferSize(handle, c); - g_BarConsole.ConsoleThread = (HANDLE)_beginthreadex(NULL, 0, BarConsoleThread, NULL, 0, &threadId); - if (!g_BarConsole.ConsoleThread) - BarConsoleDestroy(); - } - - vtparse_init(&g_BarConsole.Parser, BarParseCallback); + r.Left = 0; + r.Top = 0; + r.Right = c.X - 1; + r.Bottom = c.Y - 1; + SetConsoleWindowInfo(handle, TRUE, &r); } -void BarConsoleDestroy() { - if (g_BarConsole.ConsoleThread) { - g_BarConsole.Terminate = true; - fputs(" ", stderr); - fputs(" ", stdout); - - fflush(stdout); - fflush(stderr); - - WaitForSingleObject(g_BarConsole.ConsoleThread, INFINITE); - } - - if (g_BarConsole.OriginalStdErr > 0) { - fflush(stderr); - dup2(g_BarConsole.OriginalStdErr, fileno(stderr)); - _close(g_BarConsole.OriginalStdErr); - g_BarConsole.OriginalStdErr = 0; - } - if (g_BarConsole.OriginalStdOut > 0) { - fflush(stdout); - dup2(g_BarConsole.OriginalStdOut, fileno(stdout)); - _close(g_BarConsole.OriginalStdOut); - g_BarConsole.OriginalStdOut = 0; - } - if (g_BarConsole.Pipes[0] > 0) { - _close(g_BarConsole.Pipes[0]); - g_BarConsole.Pipes[0] = 0; - } - if (g_BarConsole.Pipes[1] > 0) { - _close(g_BarConsole.Pipes[1]); - g_BarConsole.Pipes[1] = 0; - } +void BarConsoleSetCursorPosition(COORD position) +{ + SetConsoleCursorPosition(BarConsoleGetStdOut(), position); } -HANDLE BarConsoleGetStdIn () { - return g_BarConsole.StdIn; -} +COORD BarConsoleGetCursorPosition() +{ + CONSOLE_SCREEN_BUFFER_INFO consoleInfo; + COORD result = { -1, -1 }; + + if (GetConsoleScreenBufferInfo(BarConsoleGetStdOut(), &consoleInfo)) + result = consoleInfo.dwCursorPosition; -HANDLE BarConsoleGetStdOut () { - return GetStdHandle(STD_OUTPUT_HANDLE);//g_BarConsole.StdOut; + return result; } -void BarConsoleSetTitle (const char* title) { - size_t len = MultiByteToWideChar (CP_UTF8, 0, title, -1, NULL, 0); +COORD BarConsoleMoveCursor(int xoffset) +{ + COORD position; - TCHAR* wTitle = malloc((len + 1) * sizeof(TCHAR)); - if (NULL != wTitle) - { - MultiByteToWideChar (CP_UTF8, 0, title, -1, wTitle, len); - SetConsoleTitleW (wTitle); + position = BarConsoleGetCursorPosition(); + position.X += xoffset; + BarConsoleSetCursorPosition(position); - free(wTitle); - } - else - SetConsoleTitleA (title); + return position; } -void BarConsoleSetSize (int width, int height) { - HANDLE handle; - SMALL_RECT r; - COORD c, s; - CONSOLE_SCREEN_BUFFER_INFO csbi; +void BarConsoleEraseCharacter() +{ + TCHAR buffer[256]; + WORD buffer2[256]; + CONSOLE_SCREEN_BUFFER_INFO csbiInfo; + COORD coords; + HANDLE handle = BarConsoleGetStdOut(); + int length, read, write; + + if (!GetConsoleScreenBufferInfo(handle, &csbiInfo)) + return; + + length = csbiInfo.dwSize.X - csbiInfo.dwCursorPosition.X - 1; + read = csbiInfo.dwCursorPosition.X + 1; + write = csbiInfo.dwCursorPosition.X; + + while (length >= 0) + { + int size = min(length, 256); + DWORD chRead = 0, chWritten = 0; + coords = csbiInfo.dwCursorPosition; + coords.X = read; + ReadConsoleOutputAttribute(handle, buffer2, size, coords, &chRead); + ReadConsoleOutputCharacter(handle, buffer, size, coords, &chRead); + + if (chRead == 0) + break; + + coords.X = write; + WriteConsoleOutputAttribute(handle, buffer2, chRead, coords, &chWritten); + WriteConsoleOutputCharacter(handle, buffer, chRead, coords, &chWritten); + + read += chRead; + write += chRead; + length -= chRead; + } +} - handle = CONSOLE_REAL_OUTPUT_HANDLE; +void BarConsoleEraseLine(int mode) +{ + CONSOLE_SCREEN_BUFFER_INFO csbiInfo; + DWORD writen, length; + COORD coords; - if (!GetConsoleScreenBufferInfo(handle, &csbi)) - return; + HANDLE handle = BarConsoleGetStdOut(); - s.X = csbi.srWindow.Right - csbi.srWindow.Left + 1; - s.Y = csbi.srWindow.Bottom - csbi.srWindow.Top + 1; + if (!GetConsoleScreenBufferInfo(handle, &csbiInfo)) + return; - if (s.X > width && s.Y > height) - return; + writen = 0; + coords.X = 0; + coords.Y = csbiInfo.dwCursorPosition.Y; - c.X = width; - c.Y = height; + switch (mode) + { + default: + case 0: /* from cursor */ + coords.X = BarConsoleGetCursorPosition().X; + length = csbiInfo.dwSize.X - coords.X; + break; - if (s.X > c.X) - c.X = s.X; - if (s.Y > c.Y) - c.Y = s.Y; + case 1: /* to cursor */ + coords.X = 0; + length = BarConsoleGetCursorPosition().X; + break; - SetConsoleScreenBufferSize(handle, c); + case 2: /* whole line */ + coords.X = 0; + length = csbiInfo.dwSize.X; + break; + } - r.Left = 0; - r.Top = 0; - r.Right = c.X - 1; - r.Bottom = c.Y - 1; - SetConsoleWindowInfo(handle, TRUE, &r); -} + FillConsoleOutputCharacter(handle, ' ', length, coords, &writen); + FillConsoleOutputAttribute(handle, csbiInfo.wAttributes, csbiInfo.dwSize.X, coords, &writen); -void BarConsoleSetCursorPosition (COORD position) { - SetConsoleCursorPosition(CONSOLE_REAL_OUTPUT_HANDLE, position); + SetConsoleCursorPosition(handle, coords); } -COORD BarConsoleGetCursorPosition () { - CONSOLE_SCREEN_BUFFER_INFO consoleInfo; - COORD result = { -1, -1 }; +void BarConsoleSetClipboard(const char* text) +{ + WCHAR* wideString; + HANDLE stringHandle; + size_t wideSize, wideBytes; - if (GetConsoleScreenBufferInfo(CONSOLE_REAL_OUTPUT_HANDLE, &consoleInfo)) - result = consoleInfo.dwCursorPosition; + wideSize = MultiByteToWideChar(CP_UTF8, 0, text, -1, NULL, 0); + wideBytes = wideSize * sizeof(WCHAR); + wideString = malloc(wideBytes); + if (!wideString) + return; - return result; -} + MultiByteToWideChar(CP_UTF8, 0, text, -1, wideString, wideSize); -COORD BarConsoleMoveCursor (int xoffset) { - COORD position; + stringHandle = GlobalAlloc(GMEM_MOVEABLE, wideBytes); + if (!stringHandle) + { + free(wideString); + return; + } - position = BarConsoleGetCursorPosition(); - position.X += xoffset; - BarConsoleSetCursorPosition(position); + memcpy(GlobalLock(stringHandle), wideString, wideBytes); - return position; -} + GlobalUnlock(stringHandle); -void BarConsoleEraseCharacter () { - TCHAR buffer[256]; - WORD buffer2[256]; - CONSOLE_SCREEN_BUFFER_INFO csbiInfo; - COORD coords; - HANDLE handle = CONSOLE_REAL_OUTPUT_HANDLE; - int length, read, write; - - if (!GetConsoleScreenBufferInfo(handle, &csbiInfo)) - return; - - length = csbiInfo.dwSize.X - csbiInfo.dwCursorPosition.X - 1; - read = csbiInfo.dwCursorPosition.X + 1; - write = csbiInfo.dwCursorPosition.X; - - while (length >= 0) { - int size = min(length, 256); - DWORD chRead = 0, chWritten = 0; - coords = csbiInfo.dwCursorPosition; - coords.X = read; - ReadConsoleOutputAttribute(handle, buffer2, size, coords, &chRead); - ReadConsoleOutputCharacter(handle, buffer, size, coords, &chRead); - - if (chRead == 0) - break; - - coords.X = write; - WriteConsoleOutputAttribute(handle, buffer2, chRead, coords, &chWritten); - WriteConsoleOutputCharacter(handle, buffer, chRead, coords, &chWritten); - - read += chRead; - write += chRead; - length -= chRead; - } -} + if (!OpenClipboard(NULL)) + { + GlobalFree(stringHandle); + free(wideString); + return; + } -void BarConsoleEraseLine (int mode) { - CONSOLE_SCREEN_BUFFER_INFO csbiInfo; - DWORD writen, length; - COORD coords; + EmptyClipboard(); - HANDLE handle = CONSOLE_REAL_OUTPUT_HANDLE; + SetClipboardData(CF_UNICODETEXT, stringHandle); - if (!GetConsoleScreenBufferInfo(handle, &csbiInfo)) - return; + CloseClipboard(); - writen = 0; - coords.X = 0; - coords.Y = csbiInfo.dwCursorPosition.Y; + free(wideString); +} - switch (mode) { - default: - case 0: /* from cursor */ - coords.X = BarConsoleGetCursorPosition().X; - length = csbiInfo.dwSize.X - coords.X; - break; +void BarConsoleFlush() +{ + char buffer[BAR_BUFFER_CAPACITY]; + size_t bufferSize = 0; - case 1: /* to cursor */ - coords.X = 0; - length = BarConsoleGetCursorPosition().X; - break; + if (g_BarConsole.BufferSize == 0) + return; - case 2: /* whole line */ - coords.X = 0; - length = csbiInfo.dwSize.X; - break; - } + bufferSize = g_BarConsole.BufferSize; + memcpy(buffer, g_BarConsole.Buffer, bufferSize); - FillConsoleOutputCharacter(handle, ' ', length, coords, &writen); - FillConsoleOutputAttribute(handle, csbiInfo.wAttributes, csbiInfo.dwSize.X, coords, &writen); + g_BarConsole.BufferSize = 0; - SetConsoleCursorPosition(handle, coords); + vtparse(&g_BarConsole.Parser, buffer, bufferSize); } -void BarConsoleSetClipboard(const char* text) { - WCHAR* wideString; - HANDLE stringHandle; - size_t wideSize, wideBytes; +void BarConsolePutc(char c) +{ + if (g_BarConsole.BufferSize >= BAR_BUFFER_CAPACITY) + BarConsoleFlush(); - wideSize = MultiByteToWideChar(CP_UTF8, 0, text, -1, NULL, 0); - wideBytes = wideSize * sizeof(WCHAR); - wideString = malloc(wideBytes); - if (!wideString) - return; - - MultiByteToWideChar(CP_UTF8, 0, text, -1, wideString, wideSize); + g_BarConsole.Buffer[g_BarConsole.BufferSize++] = c; +} - stringHandle = GlobalAlloc(GMEM_MOVEABLE, wideBytes); - if (!stringHandle) { - free(wideString); - return; - } +void BarConsolePuts(const char* c) +{ + size_t length = strlen(c); + size_t i; - memcpy(GlobalLock(stringHandle), wideString, wideBytes); + for (i = 0; i < length; ++i) + BarConsolePutc(c[i]); +} - GlobalUnlock(stringHandle); +static char* BarConsoleFormat(char* buffer, size_t buffer_size, const char* format, va_list args) +{ + bool allocated = false; + + int chars_writen; + while ((chars_writen = _vsnprintf(buffer, buffer_size - 1, format, args)) < 0) + { + size_t new_buffer_size = buffer_size * 3 / 2; + if (new_buffer_size < buffer_size) + { // handle overflow + chars_writen = buffer_size; + break; + } + + if (allocated) + buffer = realloc(buffer, new_buffer_size); + else + buffer = malloc(new_buffer_size); + buffer_size = new_buffer_size; + } + + return buffer; +} - if (!OpenClipboard(NULL)) { - GlobalFree(stringHandle); - free(wideString); - return; - } +void BarConsolePrint(const char* format, ...) +{ + va_list args; + va_start(args, format); + BarConsolePrintV(format, args); + va_end(args); +} - EmptyClipboard(); +void BarConsolePrintV(const char* format, va_list args) +{ + char localBuffer[BAR_BUFFER_CAPACITY]; + size_t bufferSize = 0, i = 0; - SetClipboardData(CF_UNICODETEXT, stringHandle); + char* buffer = BarConsoleFormat(localBuffer, bufferSize, format, args); - CloseClipboard(); + BarConsolePuts(buffer); - free(wideString); + if (buffer != localBuffer) + free(buffer); }
\ No newline at end of file diff --git a/src/console.h b/src/console.h index 4b3a073..de5d951 100644 --- a/src/console.h +++ b/src/console.h @@ -25,7 +25,8 @@ THE SOFTWARE. #define SRC_CONSOLE_H_WY8F3MNH #include "config.h" - +#include <stdio.h> +#include <stdarg.h> #include <windows.h> void BarConsoleInit (); @@ -42,4 +43,10 @@ void BarConsoleEraseLine (int mode); // 0 - from cursor, 1 - to cursor, 2 - enti void BarConsoleSetClipboard (const char*); +void BarConsoleFlush(); +void BarConsolePutc(char c); +void BarConsolePuts(const char* c); +void BarConsolePrint(const char* format, ...); +void BarConsolePrintV(const char* format, va_list args); + #endif /* SRC_CONSOLE_H_WY8F3MNH */ @@ -77,7 +77,7 @@ static bool BarMainGetLoginCredentials (BarSettings_t *settings, BarUiMsg (settings, MSG_QUESTION, "Password: "); BarReadlineStr (passBuf, sizeof (passBuf), rl, BAR_RL_NOECHO); /* write missing newline */ - puts (""); + BarConsolePuts(""); settings->password = strdup (passBuf); } else { //pid_t chld; @@ -75,7 +75,7 @@ static const char *BarStrCaseStr (const char *haystack, const char *needle) { return NULL; } -char* BatStrFormat (const char* format, va_list args) { +char* BarStrFormat (const char* format, va_list args) { static const size_t c_initial_buffer_size = 256; char* buffer = malloc(c_initial_buffer_size); @@ -117,7 +117,7 @@ void BarUiMsg (const BarSettings_t *settings, const BarUiMsg_t type, case MSG_DEBUG: /* print ANSI clear line */ - fputs ("\033[2K", stdout); + BarConsolePuts("\033[2K"); break; default: @@ -125,14 +125,14 @@ void BarUiMsg (const BarSettings_t *settings, const BarUiMsg_t type, } if (settings->msgFormat[type].prefix != NULL) { - fputs (settings->msgFormat[type].prefix, stdout); + BarConsolePuts(settings->msgFormat[type].prefix); } va_start (fmtargs, format); - vprintf (format, fmtargs); + BarConsolePrintV (format, fmtargs); if (type == MSG_DEBUG) { - char* msg = BatStrFormat (format, fmtargs); + char* msg = BarStrFormat (format, fmtargs); if (msg != NULL) { BarConsoleSetClipboard (msg); free (msg); @@ -142,10 +142,10 @@ void BarUiMsg (const BarSettings_t *settings, const BarUiMsg_t type, va_end (fmtargs); if (settings->msgFormat[type].postfix != NULL) { - fputs (settings->msgFormat[type].postfix, stdout); + BarConsolePuts(settings->msgFormat[type].postfix); } - fflush (stdout); + BarConsoleFlush(); } /* piano wrapper: prepare/execute http request and pass result back to diff --git a/src/ui_act.c b/src/ui_act.c index 54167f8..4206a33 100644 --- a/src/ui_act.c +++ b/src/ui_act.c @@ -32,6 +32,7 @@ THE SOFTWARE. #include "ui.h" #include "ui_readline.h" #include "ui_dispatch.h" +#include "console.h" /* standard eventcmd call */ @@ -691,7 +692,7 @@ BarUiActCallback(BarUiActSettings) { modified = true; } /* write missing newline */ - puts (""); + BarConsolePuts(""); break; } diff --git a/src/ui_readline.c b/src/ui_readline.c index ebb21ef..5d6b7a7 100644 --- a/src/ui_readline.c +++ b/src/ui_readline.c @@ -241,7 +241,7 @@ size_t BarReadline (char *buf, const size_t bufSize, const char *mask, case VK_RETURN: if (echo) - fputc('\n', stdout); + BarConsolePutc('\n'); return bufLen; default: { @@ -274,8 +274,8 @@ size_t BarReadline (char *buf, const size_t bufSize, const char *mask, strncpy(bufOut, encodedCodePoint, encodedCodePointLength); if (echo) { - fputs(encodedCodePoint, stdout); - fflush(stdout); + BarConsolePuts(encodedCodePoint); + BarConsoleFlush(); } bufOut += encodedCodePointLength; @@ -285,7 +285,7 @@ size_t BarReadline (char *buf, const size_t bufSize, const char *mask, if ((bufLen >= (int)(bufSize - 1)) && overflow) { if (echo) - fputc('\n', stdout); + BarConsolePutc('\n'); return bufLen; } } |