{"id":17217,"url":"https://patchwork.libcamera.org/api/1.1/covers/17217/?format=json","web_url":"https://patchwork.libcamera.org/cover/17217/","project":{"id":1,"url":"https://patchwork.libcamera.org/api/1.1/projects/1/?format=json","name":"libcamera","link_name":"libcamera","list_id":"libcamera_core","list_email":"libcamera-devel@lists.libcamera.org","web_url":"","scm_url":"","webscm_url":""},"msgid":"<20220829084445.61585-1-tomi.valkeinen@ideasonboard.com>","date":"2022-08-29T08:44:43","name":"[libcamera-devel,v2,0/2] LogCategory fixing","submitter":{"id":109,"url":"https://patchwork.libcamera.org/api/1.1/people/109/?format=json","name":"Tomi Valkeinen","email":"tomi.valkeinen@ideasonboard.com"},"mbox":"https://patchwork.libcamera.org/cover/17217/mbox/","series":[{"id":3442,"url":"https://patchwork.libcamera.org/api/1.1/series/3442/?format=json","web_url":"https://patchwork.libcamera.org/project/libcamera/list/?series=3442","date":"2022-08-29T08:44:43","name":"LogCategory fixing","version":2,"mbox":"https://patchwork.libcamera.org/series/3442/mbox/"}],"comments":"https://patchwork.libcamera.org/api/covers/17217/comments/","headers":{"Return-Path":"<libcamera-devel-bounces@lists.libcamera.org>","X-Original-To":"parsemail@patchwork.libcamera.org","Delivered-To":"parsemail@patchwork.libcamera.org","Received":["from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id 23673C0DA4\n\tfor <parsemail@patchwork.libcamera.org>;\n\tMon, 29 Aug 2022 08:45:00 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 64FFA61FC4;\n\tMon, 29 Aug 2022 10:44:59 +0200 (CEST)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[IPv6:2001:4b98:dc2:55:216:3eff:fef7:d647])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 41A1161F9F\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tMon, 29 Aug 2022 10:44:57 +0200 (CEST)","from deskari.lan (91-158-154-79.elisa-laajakaista.fi\n\t[91.158.154.79])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 7A3AC481;\n\tMon, 29 Aug 2022 10:44:56 +0200 (CEST)"],"DKIM-Signature":["v=1; a=rsa-sha256; c=relaxed/simple; d=libcamera.org;\n\ts=mail; t=1661762699;\n\tbh=88lhA+whrF8HVN3kom5ZzkHmIsiQcJP24gVN4mEHkTg=;\n\th=To:Date:Subject:List-Id:List-Unsubscribe:List-Archive:List-Post:\n\tList-Help:List-Subscribe:From:Reply-To:From;\n\tb=J+QFKrbfWiyCS/jl+O0ceqaN5UAghMb99JsBRD01lBHb8KHdlj1Tyb9he4MIKWPeV\n\tct3t5sAMM6rcDFpF9VLZFJKRC98IbXjcrJGO5A0KcEMvRay+MWN0Caaxmy/02OMqwA\n\t4WSeOinZmc/zz2HFuDpatO7xRS1Vft11eIfXBqI4mUQVW7+XS6EGP4b/kuoPkAxaWl\n\t6aMZBdC5Z3u1M2bxnbiLT13ieqJk9E9D3WiFi4QO/5nitvNlsfTRIeGJH/9KjtgNzc\n\tnAdEIYgAt5h3zwd7pPFWkKgGrZfMnP0tRFgpbQu4YKEGbwwasFpwMe6cKM8lRDVhyP\n\tEB1zz1GVy/ogw==","v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1661762696;\n\tbh=88lhA+whrF8HVN3kom5ZzkHmIsiQcJP24gVN4mEHkTg=;\n\th=From:To:Cc:Subject:Date:From;\n\tb=T7ERtCejhTlXOwY9KMEXDe+FQ8PrSFBE+uWOfPVdD4LvC/UC/UH7uK6m0H2M6WHGN\n\ttWQZw3JsmQCidZKx0tL4flX+Vawasb4EKljZSVLiPjqQpMMaIlAKN7gFOZXmDCJCOK\n\tc2tvyqSH0thNE43J0tv/5dFe/n592Q3wl4pPu6qI="],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key; \n\tunprotected) header.d=ideasonboard.com\n\theader.i=@ideasonboard.com\n\theader.b=\"T7ERtCej\"; dkim-atps=neutral","To":"libcamera-devel <libcamera-devel@lists.libcamera.org>,\n\tLaurent Pinchart <laurent.pinchart@ideasonboard.com>","Date":"Mon, 29 Aug 2022 11:44:43 +0300","Message-Id":"<20220829084445.61585-1-tomi.valkeinen@ideasonboard.com>","X-Mailer":"git-send-email 2.34.1","MIME-Version":"1.0","Content-Transfer-Encoding":"8bit","Subject":"[libcamera-devel] [PATCH v2 0/2] LogCategory fixing","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"<libcamera-devel.lists.libcamera.org>","List-Unsubscribe":"<https://lists.libcamera.org/options/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=unsubscribe>","List-Archive":"<https://lists.libcamera.org/pipermail/libcamera-devel/>","List-Post":"<mailto:libcamera-devel@lists.libcamera.org>","List-Help":"<mailto:libcamera-devel-request@lists.libcamera.org?subject=help>","List-Subscribe":"<https://lists.libcamera.org/listinfo/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=subscribe>","From":"Tomi Valkeinen via libcamera-devel\n\t<libcamera-devel@lists.libcamera.org>","Reply-To":"Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>"},"content":"Hi,\n\nChanges to v1:\n\n- Use char * instead of string, as that is the style used in log.cpp\n- Fix review comments wrt. doc strings\n- Drop the ASSERT()\n\nI decided to drop the string uses and keep using char *, as that seems\nto be the overall style in log.cpp. I do like strings better, or\nstring_views where applicable, but perhaps converting only small parts\nof the file to use those classes in the context of this bug fix is\nunnecessary.\n\n Tomi\n\nTomi Valkeinen (2):\n  libcamera: base: log: Fix use of freed name\n  libcamera: base: log: Fix LogCategory creation issues\n\n include/libcamera/base/log.h | 10 ++++---\n src/libcamera/base/log.cpp   | 51 +++++++++++++++++++++++++++++++-----\n 2 files changed, 50 insertions(+), 11 deletions(-)"}