From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from gnu.wildebeest.org (gnu.wildebeest.org [45.83.234.184]) by sourceware.org (Postfix) with ESMTPS id A73CA3858420 for ; Wed, 15 Dec 2021 23:54:47 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org A73CA3858420 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=klomp.org Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=klomp.org Received: from reform (deer0x12.wildebeest.org [172.31.17.148]) (using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by gnu.wildebeest.org (Postfix) with ESMTPSA id 4FC1B302FBA5; Thu, 16 Dec 2021 00:54:46 +0100 (CET) Received: by reform (Postfix, from userid 1000) id E64782E80658; Thu, 16 Dec 2021 00:54:45 +0100 (CET) Date: Thu, 16 Dec 2021 00:54:45 +0100 From: Mark Wielaard To: "Frank Ch. Eigler" Cc: Alexander Kanavin , elfutils-devel@sourceware.org Subject: Re: [PATCH] debuginfod/debuginfod-client.c: use long for cache time configurations Message-ID: References: <20211209095123.3178606-1-alex@linutronix.de> <20211209113514.GB21366@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20211209113514.GB21366@redhat.com> X-Spam-Status: No, score=-3.7 required=5.0 tests=BAYES_00, JMQ_SPF_NEUTRAL, KAM_DMARC_STATUS, SPF_HELO_NONE, SPF_PASS, TXREP 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: elfutils-devel@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Elfutils-devel mailing list List-Unsubscribe: , List-Archive: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 15 Dec 2021 23:54:49 -0000 Hi, On Thu, Dec 09, 2021 at 06:35:14AM -0500, Frank Ch. Eigler via Elfutils-devel wrote: > > x32, riscv32, arc use 64bit time_t even while they are 32bit > > architectures, therefore directly using integer printf formats will not > > work portably. > > > Use a plain long everywhere as the intervals are small enough > > that it will not be problematic. > > lgtm! Added ChangeLog entry and pushed. Thanks, Mark