From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: <ffmpeg-devel-bounces@ffmpeg.org> Received: from ffbox0-bg.ffmpeg.org (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTPS id 11B394BB1A for <ffmpegdev@gitmailbox.com>; Wed, 21 May 2025 20:05:19 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTP id B1F1768DC69; Wed, 21 May 2025 23:05:15 +0300 (EEST) Received: from NAM12-DM6-obe.outbound.protection.outlook.com (mail-dm6nam12olkn2098.outbound.protection.outlook.com [40.92.22.98]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTPS id 2B2D968DC5D for <ffmpeg-devel@ffmpeg.org>; Wed, 21 May 2025 23:05:13 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=TnAT+gxHNhTPJ34omgKFovmYe3xE5o+FLsb4nWRfdjPDwbsYieNWEcJsTfeqAfbrNIXQ7/X8EUCZ/+qQOnwMoY5IRj580R9qjyL0BxdteHts/A0rCS/0Nq3kFHqRyd0NDmI2JkOgL7X33/+npiAaP3pvUB6pZCjP61U0mlROQF3BaTOxXyPBVuGj+ZRagZ7HZitS31JyLf9XFnVh98uZwTDowqYKQI3NiaBcJV8OEUlu8UYi+WWEiY/+mDAjOsO9+slwLfjKfqLipPLSKThhnBCMFkTOis+kqVVHwVedryItcJkBqj+8Ep24Y/J8EG2CnAeHrBvtu+te47Sftp1aOQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=isxJ6erujwtaomYMclHOqLhXq/YnzytIR74HamC8N0o=; b=Ynhb33G78092OXVvQ77qSNtdqgngp6tXxPqN2L3iHy+YQqf6bHo2YKWajXmijfok96Dq+ysXv3sERsHIObPv4cAFydON1xhLLW73cqzaYtwXGbMODW4Pcv6G+ABERR4uYpZieZyuO41Wx0tXonlKQJBrT8MzG2++qvGXGfdW+lMhRaYaN0LWhrA6f+g6O3CQxJcfkWWQnvf60L/mwFckGfG3mTx0tvFWrGUKXlTlmAS+Du47yY+1aFBdmZeftinXtNeFsjruU/hs+PiJghf+L23XN4R34J+Mc1RoFHAaoXLiyn/8IbDqv0EUSniKeanOwaSEwONfTFZ7ybFJ13rVWg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=hotmail.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=isxJ6erujwtaomYMclHOqLhXq/YnzytIR74HamC8N0o=; b=SQGCYzLVVKTjogK4xTK4y8m3/6eZuTZ220VUA3c90i2XBfrGovx3s+CUofXkn7bxJE0bXmVgq9igC+RCjohYiKj3VtU8TqRIxqYaWG6721eY7h9TXiPtbpO7+UgrzXuxE0R2WV92V0j968jaEszjUAWvW0OCsvaITfKxqCe83VREUNv/ejH1VHQSGX/nSo2tmJoWfFfyv/Hvl5fumKZf5jML3Yld96pJ6HqTGS8maCVFSnz5duQFj0WKupZEwoBahpOBI+28N9+SXLVkp+2OUGQME31P3ytnd8Sx5deEnh9nqz4+r87SrlH5huscK0+VJFuXMEfmkIrTZJjNjtzeFQ== Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM (2603:10b6:8:b::20) by DS7P223MB0455.NAMP223.PROD.OUTLOOK.COM (2603:10b6:8:9d::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8722.33; Wed, 21 May 2025 20:05:10 +0000 Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::bf09:8e9:b07f:98a7]) by DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::bf09:8e9:b07f:98a7%4]) with mapi id 15.20.8769.019; Wed, 21 May 2025 20:05:10 +0000 From: "softworkz ." <softworkz-at-hotmail.com@ffmpeg.org> To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org> Thread-Topic: [FFmpeg-devel] Graphprint Patches Overview Thread-Index: AdvJ5+bnifF8gBLfTUqEJSqZYoRXbwAV5GKAABHry9A= Date: Wed, 21 May 2025 20:05:10 +0000 Message-ID: <DM8P223MB036509CA77E9690642E130EDBA9EA@DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM> References: <DM8P223MB036557FE6BF9356FA16030F3BA9EA@DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM> <CABGuwEkXQCKSKV9OPxCG0CoCHT2ALNpY+QBY=windK1_ZNtgvg@mail.gmail.com> In-Reply-To: <CABGuwEkXQCKSKV9OPxCG0CoCHT2ALNpY+QBY=windK1_ZNtgvg@mail.gmail.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ms-publictraffictype: Email x-ms-traffictypediagnostic: DM8P223MB0365:EE_|DS7P223MB0455:EE_ x-ms-office365-filtering-correlation-id: ae869593-a3e2-40f6-f9ef-08dd98a2ca46 x-ms-exchange-slblob-mailprops: obhAqMD0nT+76ZruCrigLPUIQMEMZUn+3yjFTDr9KpcDwdZcFM3gyWmcS65iWCYGGzFBopHTAmdyWSnxbcLmagKV8IS2qqWcNJ4FgnHUunbjY1vJXBALSuYDN3kX+mQvU4n+jhR43AsNx1WFEipwYVVsUTOaHhOS3z/JkNYinC3XgVQvoFk3mo3fc0BKJ1B2Lp+sjK8qHlRWf3MSLctE5WLiWISu4MmyaxjlsGfgQqniNQh+rGFmdHO/s1JedObApK6BrdE6JUTUQBdYfNHIADrDwrbcz7idDtHZYdVTUCVV8KfFloVXl5OVZvpqNWX6nVofnJdirxI3/8DVPE1YXKS0PP0oIbjyGhqkNMbVUCKSSZspgSwcIElwfgmwwzryvnwW//fAiEXNhdDMxNAQVelz52Dpj8P91uROV0bRrTaLMgvQC8MMzJM2B1eQksN6dE/Z598mKwnyyJCOFI74akP7A0gywcLkaCIaFsILDuS5K64VUqAvxgkBUzqBknkRP18BK6ArrDmLzS4JM9qYPRefX2BjWzxg6gByeR+ggI2agPPZ0cq3jfqgLh14abEh5ZV+Th6FI7a/l4oybtPpnxaI9TYib0iSjY3IpzSMrPrYvh11I0AhGitccUHh+PhittGF6Puf5p45QDi78nVV2QloeMS58ptiuwChFKT4i4YPwZWzaYCs43JAd4Hp4u+gL16EJdP+QfFGLnokmtE/24mWFdSeFcNx8663Ao2rqqwbBAyjWEswPhRozuK4o0qNJacDua9Fm9eYpvFu4AUQmKtWTr/zO4H4smsKrhM9upbW6aKhXNZevw9pmURg4mDo x-microsoft-antispam: BCL:0; ARA:14566002|461199028|7092599006|8060799009|19110799006|8062599006|15080799009|10035399007|1602099012|4302099013|3412199025|440099028|102099032|12091999003; x-microsoft-antispam-message-info: =?us-ascii?Q?EqJdsQIXKYjI+tsvuwKm6EwqPRZM/ki4f53jx0hwueF/Br7SR1PS9WapEMT+?= =?us-ascii?Q?Gx7d3irB/yn/gzKKHEysL44GiwNmd34O+OfMI1logQccntqF/1fnB7soveVa?= =?us-ascii?Q?J+3PBimTmPv9uxNWqUTiaGBQ1R1zMcdmMKEkDHdGi4xkVxJaUMCCkCAi+ZDA?= =?us-ascii?Q?2E6JPKW+BKe713q5yMO/SjahkFZVv2yOppRgo1NswudbXED7k/a7SmZXjyYG?= =?us-ascii?Q?k67/K7o+2A/xrtYONG5R44A/g+QbPazDq/brGsB4cteZHrpIDprbIOmuSw03?= =?us-ascii?Q?Gi3imZ2ZFDXDScQogRXzP0a6kksM5kGq/CHcDRNKNCNVmpOkkKWuRsnpmqk6?= =?us-ascii?Q?HnS+NfUH0fXkKLOMrA6kvDgueIzVrsWZVQgVveSR4OAIFWxqjl86oTgHR8v5?= =?us-ascii?Q?8UAo+htQ3uLrsJcjj9a2H1JyD5AIYZDojbIZoO+ygR/TotkuQ+Oi7OwaJXvh?= =?us-ascii?Q?nUobHD3+h9BpCzZLod0lR/1rxmED4LpeEExIb2E5yX7UDmsGp83ZuPHQokOc?= =?us-ascii?Q?5n31Q4PB1VKwLhRixz+iaw4kEXH2ZTuhVWnFVKSH4z7Wy4+sdSX7Kf3ElwWj?= =?us-ascii?Q?2X3LK4xlRe/nNJ8yJcSYFi5VaYk8tNuvoNUowm9GmFIcMZGa3SoAfCR6BXG0?= =?us-ascii?Q?MqL6zFG0drlHZM6KSKOnXH6ioE8rxgPaJdAm54W837bk1I2LVgWefyZEs0ox?= =?us-ascii?Q?zXsIJwWSVveXp5gKBWFpjcU9WIO2vA+AjrDdkEM07UJmUdd18xhVvw0dHIn8?= =?us-ascii?Q?DesBKPfZHjU38gu0Hd6MXmfc1PPnpUVg6Txu9VB0wKdXaOtM5TCuoZKUT2Op?= =?us-ascii?Q?NMMGDqFNusBEo4k8HVJcXkA2TwiPUUttuw+8XEYTTUvcUHbvM8AkutjFQadq?= =?us-ascii?Q?aNWIFGyLy9O4D6pwvH8Rs1L1NXtdRUfL6Cqa45iatASOl1EY4xFG3/lQ7WN+?= =?us-ascii?Q?oAYim0XwXd5u4+iycaCgj6wG42jDZE3H8l4FCOfimwugerGYrjmbWoh9MDXG?= =?us-ascii?Q?Aaxs20xF7jNgG8YJUYOq4ArNx0TYchnDro1vvEwciCvW/9VYGowaxTBXozhy?= =?us-ascii?Q?7xXGdlwEbIKryw2RHBN1+4QDxz7jrS9dKDcs8HuxpHi8IOnzJhUVDGmoqIuf?= =?us-ascii?Q?I91UINeHyeG1SfXx1UDUvZqoVifNk4dbZC3UJrU9kvasATpHeLP9Do2YTp3i?= =?us-ascii?Q?CFNzHhijK2VoiP5SQB8hZnRIvIqkRJijSrj+sYGTeFK/nadWwkAV5Azwpw3/?= =?us-ascii?Q?o+9AdjunwK/TgyCFHOh/VmSYV4Zl2nkWq3Z06tF7h06xqvWjLMgcxCuD9xyS?= =?us-ascii?Q?xRonUiIWGbmi2P2amvTY4FZw?= x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?jZQ4tUVXAOUI1wRFBIZdzsl3l2gzo7eAn6rHNVTgKe0wFu4qz7FkjPmJm3JE?= =?us-ascii?Q?yQ6ssoVNY3WG0+Q+h8H5dbf2Sflybm/ooJMuJlnSWsVI+2rY2HMczZhn1BsN?= =?us-ascii?Q?yp8g0DlMIUcjLnB+eZ4kx/0Vs0GHmJqjB9Jx5DhINDcDjq4iaSKP/Dqx0rIh?= =?us-ascii?Q?qS9Wysf2mceECpNlbsnxJeO2hqm356tTwiX2NXkVNpatiInu7NHFyFolZijE?= =?us-ascii?Q?zhpkvnchtOr7UxyLSDekDi7URBKw96SSp8+jfMua1g+DzjwbnidtWiQfw9Af?= =?us-ascii?Q?eAZthcHVfXYNCWHK4aaNq5qNoLEK6qCBDwkerKJtMJ59ltswKVVW6ehRWhiV?= =?us-ascii?Q?Dx3EgGKiwdC8Dji/6GMmbSzaDgYEjR7j++CErd0Hi3o3Np9UyT0lNrweo/YI?= =?us-ascii?Q?50AxsvMQnqNjA61rZB6vO7LA2Z8DtS7GX3p6DqyJglFrIdZqfXK1QuDkyd5Y?= =?us-ascii?Q?OFS1BDZYvs1oGLeVPbqlGEDAjJsPLCjAhFjLks/PvHkcwPQKFbd5A4rj5ath?= =?us-ascii?Q?eiuTo40WgY29YEPxFgebZGHwNZjbwPYRIvqdjGAcXja4jhnCsnoPDum5RTqP?= =?us-ascii?Q?LVhbteOPztbgi5w6tYm29vafzkWh1Zpw6ka+wRkedNVNlxCj5Xa7zi9aMyNy?= =?us-ascii?Q?mVJNlI8Os16VDN4x8Ee4EqQYOYAzD0zGDchti4sY8ufl1hH22qaEzcPoSl7l?= =?us-ascii?Q?Bt7H+IT6ZduAhY/R/yzTqcDUmOzaTIvNsFfsFTfsOJWAQ3J6cgUXP04dLlqu?= =?us-ascii?Q?FSpRpmp7dSGtKofTcGlvRVuxEMVK0cguxKitmUBVYaQLzlbuQvJHmx+XnuIC?= =?us-ascii?Q?/WlK0VjdRRvtb+KiRKCSZD0dOBiN2bgCfoDS5SQhgsasl3bthP3urX1dcHtq?= =?us-ascii?Q?4Tb5R5cLsRQMLX0e1jtIvY5IfS3+7C/NT+zx5kuATLE/vgv6T0m5EBWF9JaT?= =?us-ascii?Q?vcoXEsOoE2EJjxlCZo8FG3ClucrEBMDXEuTkSBhOU85hXHtb6V1VEfs48CFI?= =?us-ascii?Q?CqRIZNII2Vd5zISe9LEPyN6wvw8TYxV6GgIfg3JMz+oMvoAu32Ij5g8zbamr?= =?us-ascii?Q?Z6c3bp7x2keOZZJycZOK2qbTVCOaCXrYuKr8lPS3AfJPGdz8PD6B84fx/cg8?= =?us-ascii?Q?/lbc6KnwYnMBAdMj6WrQa3djuTgOLRr57ZTtrKR0DDx2EpuZCdzIkKnJUa8y?= =?us-ascii?Q?2wOsadu0Mnq6UYGD7vgnoW2eu1RGI70Jsd33cTB12u1wx8ql/j/8EFv2d7o?= =?us-ascii?Q?=3D?= MIME-Version: 1.0 X-OriginatorOrg: sct-15-20-8534-20-msonline-outlook-c7cf3.templateTenant X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-CrossTenant-Network-Message-Id: ae869593-a3e2-40f6-f9ef-08dd98a2ca46 X-MS-Exchange-CrossTenant-originalarrivaltime: 21 May 2025 20:05:10.2199 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa X-MS-Exchange-CrossTenant-rms-persistedconsumerorg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS7P223MB0455 Subject: Re: [FFmpeg-devel] Graphprint Patches Overview X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches <ffmpeg-devel.ffmpeg.org> List-Unsubscribe: <https://ffmpeg.org/mailman/options/ffmpeg-devel>, <mailto:ffmpeg-devel-request@ffmpeg.org?subject=unsubscribe> List-Archive: <https://ffmpeg.org/pipermail/ffmpeg-devel> List-Post: <mailto:ffmpeg-devel@ffmpeg.org> List-Help: <mailto:ffmpeg-devel-request@ffmpeg.org?subject=help> List-Subscribe: <https://ffmpeg.org/mailman/listinfo/ffmpeg-devel>, <mailto:ffmpeg-devel-request@ffmpeg.org?subject=subscribe> Reply-To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org> Cc: Kieran Kunhya <kieran618@googlemail.com> Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" <ffmpeg-devel-bounces@ffmpeg.org> Archived-At: <https://master.gitmailbox.com/ffmpegdev/DM8P223MB036509CA77E9690642E130EDBA9EA@DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM/> List-Archive: <https://master.gitmailbox.com/ffmpegdev/> List-Post: <mailto:ffmpegdev@gitmailbox.com> > -----Original Message----- > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of Kieran > Kunhya via ffmpeg-devel > Sent: Mittwoch, 21. Mai 2025 13:00 > To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org> > Cc: Kieran Kunhya <kieran618@googlemail.com> > Subject: Re: [FFmpeg-devel] Graphprint Patches Overview > > On Wed, 21 May 2025, 01:45 softworkz ., <softworkz-at-hotmail.com@ffmpeg.org> > wrote: > > > Hello, > > > > thanks again to all for the patches. I figured it might be a bit difficult > > to > > keep track of what has already been submitted and fixed and is still > > pending, and I'm sorry that there has been some duplicate effort to fix the > > same things - so here's an overview. The ones with X are the ones I would > > like to apply eventually: > > > > > > Timo Rothenpieler > > https://patchwork.ffmpeg.org/project/ffmpeg/list/?series=14528 > > (I would favor "ffbuild/commonmak" over for 1/3) > > > > [1/3] fftools/resources: fix preservation of intermediary resman build > > artifacts > > X [2/3] ffbuild: correctly silence and tag new css/html steps > > X [3/3] fftools/resources: add missing extensions to .gitignore > > > > > > > > Mark Thompson (already merged) > > https://patchwork.ffmpeg.org/project/ffmpeg/list/?series=14537 > > > > X [1/3] ffmpeg: Don't print graphs if there are no graphs to print > > X [2/3] fftools/graphprint: Fix leak of graphprint object > > X [3/3] fftools/graphprint: Fix leak of graph section header string > > > > > > softworkz > > https://patchwork.ffmpeg.org/project/ffmpeg/list/?series=14563 > > > > X [1/5] fftools/makefile: Remove resources from ffprobe > > X [2/5] fftools/resources: Use .SECONDARY in Makefile comment > > X [3/5] fftools/ffmpeg: Free print_graph option variables > > X [4/5] fftools/graphprint: Fix memory leaks > > X [5/5] fftools/tf_mermaid: Add missing uninit and fix leaks > > > > https://patchwork.ffmpeg.org/project/ffmpeg/list/?series=14570 > > X [v3] ffbuild/commonmak: Fix rebuild check with implicit rule chains > > > > > > Derek Buitenhuis > > https://patchwork.ffmpeg.org/project/ffmpeg/list/?series=14569 > > (1/3 and 2/3 correspond to 2/3 from Timo, and 3/3 doesn't fix > > the rebuild check like "commonmak" above does) > > > > [1/3] ffbuild/common: Remove what appears to be a temporary debugging > > comment > > [2/3] ffbuild/common: Properly tag/suppress sed command > > [3/3] fftools/resoirces: Mark .css.min and .css.min.gz as NOTINTERMEDIATE > > > > > > Thanks again, > > sw > > > > Can we just revert the whole set until it's cleaned up properly? So that it can be ignored for another 15 revisions? I'm glad and thankful that others have looked at it now and I don't expect so many more things to come. > There are more patches to fix issues than the set itself. This is > understandable if it's a bit architectural change like threading but it's > not. Please note that many of those patches are just single-line changes and the "more fixes than patches" is not that unusual. It's only less visible because nobody normally sends an overview like I did. Even though it's not an architectural change, it's still an entirely new feature with 2k lines of new code (counting the last 4 commits only). I really would have preferred this to have happened while the patches were submitted for review and I don't know why nobody had responded to my messages asking whether somebody would still like to review it and would need more time. I would have happily waited longer - even weeks, if someone had said anything. Finally, for a "better picture", I could have excluded the patches from Mark (already merged) and Derek (duplicate), but I still listed them to acknowledge their efforts. Best, sw _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".