From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTP id A867449202 for ; Sat, 6 Apr 2024 11:33:46 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 80DF668D07E; Sat, 6 Apr 2024 14:33:43 +0300 (EEST) Received: from mail-ed1-f53.google.com (mail-ed1-f53.google.com [209.85.208.53]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 1865768CA3D for ; Sat, 6 Apr 2024 14:33:37 +0300 (EEST) Received: by mail-ed1-f53.google.com with SMTP id 4fb4d7f45d1cf-56e0e1d162bso3084126a12.1 for ; Sat, 06 Apr 2024 04:33:37 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1712403215; x=1713008015; darn=ffmpeg.org; h=user-agent:in-reply-to:content-disposition:mime-version:references :mail-followup-to:message-id:subject:to:from:date:from:to:cc:subject :date:message-id:reply-to; bh=a5834rYoUXyOzDCy6pZUozJc3sdg5fl+n8QQF9KKPVo=; b=K5E19lzTGBLUVSgNghBB+8ZIjI+G0fJ5yf4iRI6whP2CUEcULIGi7lMMraQ5tqw1DB lFV39bPp2b8vjbyn931HQTt1FkQ3pmEmu3ivnxtH5FfOEBdDCuSQWZr7ig1Qk1BsolEJ 8eOmwYYURda5QH2J6EnClTlQNaLHS65e1nZ5Kv7m/36LsFZUaawjuqBlD4us1DCtERdt nk5EvFEW3oGSgIS2HKyC8vCKBv5JoWNMuUC+RaTo8dO3ZsPYgmfwDP/gKbeIFBx7skw2 QUGYfi89i5N3fJO5oVoLm9i7qgcuZ049QM7ZdfAUFpHEAT1D8+67uR3TiCKhW+qaU05S VayQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1712403215; x=1713008015; h=user-agent:in-reply-to:content-disposition:mime-version:references :mail-followup-to:message-id:subject:to:from:date:x-gm-message-state :from:to:cc:subject:date:message-id:reply-to; bh=a5834rYoUXyOzDCy6pZUozJc3sdg5fl+n8QQF9KKPVo=; b=DOINtdNnPlbkXXJApSyC3m/o6/1frrzxFuO7QjA4qVHMil7X2JhnE6Fnbtc75tDabQ q3tRDAgodEVgfqtbHhMeGKlODkfq1oGyouJji1RUO6KzCtlQrrjntND/kJH/wHz/BH7Q 0hvgo0rKYHoeNbbzNmhgjKKFcFApO4Kp6f/02Gib1NdGV1fuDUidIwRMrV0YKJ81o7cW MJqW+PNCt8WgK9xEr/yyjQxteOEwTTprEPioJfrSx5qKjVppPXTuq/TAaHxaC5suyT85 AtaZ6rYK37K5v/sVcTPKEC//ds099EMYqjGjpoeVfp9MCPTfj3mqlrOzFI1PUAFHYsZz HB6Q== X-Gm-Message-State: AOJu0YzTkreItP5aHVzqST7boIgxFzhjwiIknXq11+oaG+CbsogjKrRZ dKM8rPJZZDV/e/cpiMIitM9Xddj6xJA0A8qNodWfg+CJts17L6nAKYq499wa X-Google-Smtp-Source: AGHT+IFdzRbKt81WH7IKKnzlSjIJncT54H6uYWPNytD211eTirN8MLx4lWgu+lKPpFC5Aq/qcct+NQ== X-Received: by 2002:a17:906:7f1a:b0:a51:91ac:ad9d with SMTP id d26-20020a1709067f1a00b00a5191acad9dmr2605845ejr.63.1712403215298; Sat, 06 Apr 2024 04:33:35 -0700 (PDT) Received: from mariano (host-79-42-51-175.retail.telecomitalia.it. [79.42.51.175]) by smtp.gmail.com with ESMTPSA id o11-20020a17090611cb00b00a4e7dfb5abasm1946011eja.171.2024.04.06.04.33.34 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 06 Apr 2024 04:33:35 -0700 (PDT) Received: by mariano (Postfix, from userid 1000) id C1D73BFCE8; Sat, 6 Apr 2024 13:33:33 +0200 (CEST) Date: Sat, 6 Apr 2024 13:33:33 +0200 From: Stefano Sabatini To: FFmpeg development discussions and patches Message-ID: Mail-Followup-To: FFmpeg development discussions and patches References: <20240406111014.6916-1-stefasab@gmail.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/2.1.4 (2021-12-11) Subject: Re: [FFmpeg-devel] [PATCH] lavf/matroskaenc: sort options by name X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 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 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: On date Saturday 2024-04-06 13:25:49 +0200, Andreas Rheinhardt wrote: > Stefano Sabatini: > > --- > > libavformat/matroskaenc.c | 12 ++++++------ > > 1 file changed, 6 insertions(+), 6 deletions(-) > > [...] > See https://ffmpeg.org/pipermail/ffmpeg-devel/2024-February/320849.html > Additionally I do not agree that sorting options by name is the best > way; > it should be sorted by what are (believed to be) the most commonly > used options. This is highly arbitrary and subjective, the best way is sort-by-name as already done in other places (and I still find the git blame argument rather weak for the reasons I already explained). The alternative - which I observe all the time - is that when a new option is added an arbitrary place is assigned and more confusion is added. _______________________________________________ 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".