From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pj1-f45.google.com (mail-pj1-f45.google.com [209.85.216.45]) by sourceware.org (Postfix) with ESMTPS id 4A436385735F for ; Tue, 10 May 2022 04:21:43 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 4A436385735F Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=maskray.me Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=gmail.com Received: by mail-pj1-f45.google.com with SMTP id iq10so14871393pjb.0 for ; Mon, 09 May 2022 21:21:43 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=TzO/osP9SJvVO3QNByqS1pJfnQYbcdVL7/5MzP4mx04=; b=Y6ZcbVcWVwHVaUdU4ppg8+rwO0e1lVNXQg1CrLXmMsEzOD28xLnHhKgzpiGbNw/gU8 P98BONYVdMy5WBczOGzPvMPbNjAp9vvI+LRVOPgJRtnMRD0prz+6hNOvUxdU9UI+P6bU Jk4FJOLxIzu3fELo3q5547zvu31/n0IHLiiZJgzZNw6uM3jZpT9CYyWGzd9n7v02PvEI 2dUeecQ4IMxmekdcc8lcFOUULCKb6fMU3XEzgr3GiQYSRS8YmZ40AxXErvoq5OPwaETb 1bfyi3p6HuFilrXz98j/+YGStXQjcs6FDsQJRu4wVO9klKvVvOci9HiMaZrQnGekqnf4 l5dg== X-Gm-Message-State: AOAM532D+YawZo/Wop+5nQIw7z2AUGahW2T0X+xkdZ+Xdwb6/Y21YTC1 OP2F1hThXxBW/1fOu7/64QWAVR1xUjI= X-Google-Smtp-Source: ABdhPJwKnRHKhEkvwCU5EBw69CPUbxbpuvqNe9QwId1IDbDwgucqn2GDIhyJZMe6B4z9U22sDM3j6A== X-Received: by 2002:a17:90a:1507:b0:1d9:9ddd:1f84 with SMTP id l7-20020a17090a150700b001d99ddd1f84mr21091441pja.75.1652156502320; Mon, 09 May 2022 21:21:42 -0700 (PDT) Received: from localhost ([12.147.0.60]) by smtp.gmail.com with ESMTPSA id u67-20020a626046000000b0050dc762814esm9522210pfb.40.2022.05.09.21.21.41 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 09 May 2022 21:21:41 -0700 (PDT) Date: Mon, 9 May 2022 21:21:41 -0700 From: Fangrui Song To: Nick Clifton Cc: binutils@sourceware.org Subject: Re: Commit: Add support for displaying unicode characters Message-ID: <20220510042141.ujhdwgimtdqjsmfq@gmail.com> References: <87v9115um8.fsf@redhat.com> <8756161e-4ba8-f552-5cf4-d1adcad33534@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii; format=flowed Content-Disposition: inline In-Reply-To: <8756161e-4ba8-f552-5cf4-d1adcad33534@redhat.com> X-Spam-Status: No, score=-1.4 required=5.0 tests=BAYES_00, FREEMAIL_FORGED_FROMDOMAIN, FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS, KAM_DMARC_STATUS, KAM_INFOUSMEBIZ, KAM_SHORT, RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL, SPF_HELO_NONE, SPF_PASS, TXREP, T_SCC_BODY_TEXT_LINE autolearn=no autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on server2.sourceware.org X-BeenThere: binutils@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Binutils mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 10 May 2022 04:21:45 -0000 Hi Nick, Thanks for the reply and openness to make tools more compatible! (even when llvm-nm made some coordination mistakes...) On 2022-05-09, Nick Clifton wrote: >Hi Fangrui, > >>I just learned that this patch added -U as an alias for --unicode. In >>macOS nm and llvm-nm, -U is an alias for --defined-only. >>--defined-only seems much more useful than --unicode. I wonder if it >>is too late to redefine -U as --defined-only... > >No, it could be changed. I have to say that I would have thought that -d >was a better choice for the short version. But anyway. > >Before making the change however, I thought that it would be best to check >to see if there are any other inconsistencies and I did find a couple: > > * nm uses -s as the short version of --print-armap, whereas llvm-nm uses -M. -M seems more like a llvm-nm problem and I believe it is almost unused. I think llvm-nm folks should think how to handle the -s conflict with macOS. nm probably should not be bothered with this llvm-nm problem. I think -M can be used for possibly more useful options in the future. > * nm uses -V as the short version of --version, whereas llvm-nm does not have one. The latest version of llvm-nm has -V: https://llvm.org/docs/CommandGuide/llvm-nm.html Having -V as an alias for --version seems a convention for quite a few utilities. >The support/lack of -V does not strike me as important, and it should be a simple >matter to add -M as another short-version of --print-armap, so unless you have >any objections I will create a patch to change -U and add -M. Thanks! Having -U for --defined-only will be great. >Cheers > Nick > >PS. I also saw that llvm-nm has a -W/--no-weak option which is missing from nm. >Is there a binutils PR filed to add this feature ? Thanks for noticing this. I just filed one: https://sourceware.org/bugzilla/show_bug.cgi?id=29135 :) Having -W/--no-weak will be great, too.