From patchwork Sun Mar 29 15:19:29 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Carl Eugen Hoyos X-Patchwork-Id: 18491 Return-Path: X-Original-To: patchwork@ffaux-bg.ffmpeg.org Delivered-To: patchwork@ffaux-bg.ffmpeg.org Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by ffaux.localdomain (Postfix) with ESMTP id 03F8844A452 for ; Sun, 29 Mar 2020 18:19:48 +0300 (EEST) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id DA13868B67B; Sun, 29 Mar 2020 18:19:47 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from mail-il1-f172.google.com (mail-il1-f172.google.com [209.85.166.172]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 92A6868B5CC for ; Sun, 29 Mar 2020 18:19:41 +0300 (EEST) Received: by mail-il1-f172.google.com with SMTP id r5so13355036ilq.6 for ; Sun, 29 Mar 2020 08:19:41 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:from:date:message-id:subject:to; bh=aIqCKhhiyk0YSc521Y61FuDjLRsKpQJVFigj/NBMI/U=; b=bYYuWN00UWnb3cKbLkU3PYXh8syWzoArwFWzBXv64NASZ32E04PV1lqiOcovEHHyR3 6znJiXIO6xE95RpA7dZbFAgKVq1PXMeOBdbUWip16/HILaX5awcvr0oNTfjzKZFz0M4P I1UeTCorPIZ6VEF7cNCMtMmCKiYFtjbcXhk/CQ9OFdZd6SRvy6I+jp11AM2Zs6TNuMM0 6W+zI5Xssas8KLzcvzel0f61EhunIiq63fYN4Ysi8q68di4oPg+COYsvwE4SdVToQeMr 4bNBT/A+q+T38Ue1O+P7dkzgT9su6flpyF1/7az2zSeCoEDYjD6wzuwDoOq533JpYnB3 J7sw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:from:date:message-id:subject:to; bh=aIqCKhhiyk0YSc521Y61FuDjLRsKpQJVFigj/NBMI/U=; b=ZfVsNvwCDLhgKKqcbmkdgdeTfOhGCRr56Pb9JAXdyKj19FccHYCkzY0luKQQiwMeEC fR5fjmLjVrcW1vVsE8MA1KHCKOG8TL2y05SqRL8GR8dmWN1REAaK4A7HdhKtJ4f31Gus 3wR4QzXZljgvm5xgsBd+gJwqO+u+yZ0nWfkfIr1DcIUk1XPDUAxC5uE0wBapxFgV0MWB +s+4GuVkp0UHFHQMXgyLa6TgiYn1MoNW7FmGeNhGZDPqXfnoxYpeErQ2+0KpBoZzruHu K0Qqa2BLA+pRRkfagTLPulHbCyZEMseG4wZzBQCeW2DQWvcTs9jOIPO6l7a8cz7y5MPW Uc0A== X-Gm-Message-State: ANhLgQ2N0OYDmAadGaHAhUkN0moTfJyStEBZS3GIrjJ5QQqbUqfNgbjp 2Ze/ZNz728Omql9Hwz0ADivoZZvoi5LT+EZMyM2X1A== X-Google-Smtp-Source: ADFU+vu0P3Tv8J5cfogOGw25+hr5kXHMKqXkly9zwchUQqFzDpjO7NXNd+JcBbOvCbN15xALbVa9Q1NODzbhBduAJe4= X-Received: by 2002:a92:890b:: with SMTP id n11mr7428632ild.54.1585495180131; Sun, 29 Mar 2020 08:19:40 -0700 (PDT) MIME-Version: 1.0 From: Carl Eugen Hoyos Date: Sun, 29 Mar 2020 17:19:29 +0200 Message-ID: To: FFmpeg development discussions and patches Subject: [FFmpeg-devel] [RFC, PATCH]lavu/log: Do not change the terminal background colour X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Hi! Attached patch fixes ticket #8587, not sure if we want this or not, but I do not consider the missing encircled / blinking effect (that is not used by our command line tools) an issue. Please comment, Carl Eugen From 64eeec19197fe7726f3bb103417ae3277b1cd8f9 Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Sun, 29 Mar 2020 14:32:38 +0200 Subject: [PATCH] lavu/log: Always use the default background color. Fixes ticket #8587. --- libavutil/log.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/libavutil/log.c b/libavutil/log.c index 8d4945249e..352a2d9d2a 100644 --- a/libavutil/log.c +++ b/libavutil/log.c @@ -90,7 +90,7 @@ static HANDLE con; #else static const uint32_t color[16 + AV_CLASS_CATEGORY_NB] = { - [AV_LOG_PANIC /8] = 52 << 16 | 196 << 8 | 0x41, + [AV_LOG_PANIC /8] = 196 << 8 | 0x41, [AV_LOG_FATAL /8] = 208 << 8 | 0x41, [AV_LOG_ERROR /8] = 196 << 8 | 0x11, [AV_LOG_WARNING/8] = 226 << 8 | 0x03, @@ -196,8 +196,7 @@ static void ansi_fputs(int level, int tint, const char *str, int local_use_color str); } else if (local_use_color == 256) { fprintf(stderr, - "\033[48;5;%"PRIu32"m\033[38;5;%"PRIu32"m%s\033[0m", - (color[level] >> 16) & 0xff, + "\033[49m\033[38;5;%"PRIu32"m%s\033[0m", (color[level] >> 8) & 0xff, str); } else -- 2.24.1