From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 121628 invoked by alias); 2 Nov 2016 19:47:24 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-alpha-owner@sourceware.org Received: (qmail 121594 invoked by uid 89); 2 Nov 2016 19:47:22 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.8 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_NONE,SPF_HELO_PASS autolearn=ham version=3.3.2 spammy=HX-HELO:sk:NAM03-B, 6820, H*r:sk:NAM03-B, H*r:sk:mail-by X-HELO: NAM03-BY2-obe.outbound.protection.outlook.com Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Steve.Ellcey@cavium.com; Message-ID: <1478116026.2891.53.camel@caviumnetworks.com> Subject: Re: [PATCH] Fix -Wformat-length warning in time/tst-strptime2.c From: Steve Ellcey To: Date: Wed, 02 Nov 2016 19:47:00 -0000 In-Reply-To: <1477326501.8523.51.camel@caviumnetworks.com> References: <1477326501.8523.51.camel@caviumnetworks.com> Content-Type: text/plain; charset="UTF-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-ClientProxiedBy: SN1PR0701CA0068.namprd07.prod.outlook.com (10.163.126.36) To DM2PR0701MB1066.namprd07.prod.outlook.com (10.160.26.19) X-MS-Office365-Filtering-Correlation-Id: 8f6bbcfa-26ee-4155-1226-08d4035908aa X-Microsoft-Exchange-Diagnostics: 1;DM2PR0701MB1066;2:73LXRrEErdj9jVioHPVM0G2Z3uNVubyQc/vrFkfHd9XY93beT1/0xCTzjS6WBfKEPFOqZvQeTfdZjTExe+kaMaqg4bTN2qwG4IzbN/Nd75lVFNwNjhZp8UWEonY+qBxmzdC4LWgcoxsvyG1ImvIe3DOSiUzdGvSeu/qzGZqqj1NT5JOhFSsr2Wj4tR5blqNsoAqFPq1zOzyX3fRrhyxsdw==;3:A92ZMBkNoCdfXGI6C3yPRTFYJ/DnQODjDFfGB99+YiSEtsxfm9tIRVeBr9c9ZgL9LfY7HKG3wSXTbKU1cPmGcqCHtOU31vYFWC2LxorQl7vc2eOzZQ4Bsj1veD40gc+olx3ghcYSWNy/XPo94BVyyg==;25:j5k37hccCDCue5qtNzF6eoVjkckjlQA2FKCXBcPAKh+plYfmDbPQM0fog+TjQ10Dr0TT6z2isZSuaUzijBClQtuN5A6DHd+y3x2I5Yb5o3AwBcJVFZL9sBSk3mLP9pLyELlMd51o7A48802W3fBsdeI2TKggDv24BSNkhDabxofWqbQy8b8ppaCuN4dclTrUiXS+UOx46Ak8AhWFyL9jJCF4hvfqGdpRDLwRHhym2nrXShCrL+ikQY1gA/2DKWzI4se9BQksLkUOlnjvN9EC0+1OTJMN7/yqw9qriE7TOrywk0uVpvs+PO7NSVh5S0UY4MkmV+bnihJwhT3LhGITi95ko55mAbvJ1Y7OS0e8qWlgc8Byin9VniWJiafp9orjZHFmcvXeXKtFBztSQ/A0s2oHqM1ASIZccue7VtwV6to= X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:DM2PR0701MB1066; X-Microsoft-Exchange-Diagnostics: 1;DM2PR0701MB1066;31:XKYihtM4al1jdq0HaeBcq/NDeT24M1/4QZoV3jD93XKOaLamCkjCHx6yxpdiNIS+cHXKgzBaBRXUlMoiUCe9lRjcaLMAq9iTPcZpTGn0ws8F+Vuv/CtyEZSh1Gu6itYVbqxq1zJAYBPEHnuCkow/iNVfUa1kt52OWZSAwzYQzynsiu3YyG34ZGYi2waWQsDFKEDTBC0vPaLiEJzmOk/TsAYeZ4zmMpnzU/bHwjI6/93Pulmw0U/1ekPMJ9en1gHU;20:wN3ccZ0JRiuQMwUUYhM4FG10IvzOO1zsH/w2kLbE9+6pG5aqC69W0wma0iJvw/Qex9gtt6yp1XyGePcU39t9JhRT4EWAiOqmZoI3xWmvWgpSY701BNhz0GotIrmfr9AnCcawvoxTBAPKI4OEzRZ+sylbNMBzHVNfV9UHNW8ctnpTZu20UF8S1ueVqv8nhOoQu6VjujiOumzRUAvWYbIQEnCM6b9MAYefuLVC98+37bNWApzX22AgJ/UcblFpAS54Ow2Iu5V6g8yjtyysZ3KbhrVmPDvlmaIlRoJqIpWnU03mSg6Bh+AwOXnGd/uXWoyN31HvSxNONqR+Ie0pbSBVf2ycfSlbjw8Slf6oGIAc8Nli1oyoTSFb0BAQlu2F33tadgeMD6WuGJrohqc0ts66zEshNL+pVrS9Szr6z4aCvDKjGZXhHeCbwoq+uX46Wp2RyMnvoW5NCDk12hDEwKQVCiOAm1FesnGq8UYCcujo4ndVNmV0mFWRweGIu3Cw2/f+nJH/x0hPVnQsgfO3kx/SfGfNER7SqwaBRemYgHILHs4MTZtF1zc9tFh6xN7EVW0zFI8KfyEWGDuUf7OyxpYp+a2uZUs11fOmdj/TnDGT6Jc= X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(6040176)(601004)(2401047)(8121501046)(5005006)(10201501046)(3002001);SRVR:DM2PR0701MB1066;BCL:0;PCL:0;RULEID:;SRVR:DM2PR0701MB1066; X-Microsoft-Exchange-Diagnostics: 1;DM2PR0701MB1066;4:2/TdCfcVKcVBp17FL3EcLYZf1GtltUF6ChN200HetBhlBw5pnXnGTcG1Q6cWS+UPVr9sgGj0FFwTuyT2YpwV4l1lB3Kd3ukmZ1FZYAyHfkN32WXybIKGxgnO5Wxt95JQ4SEyBhCGlHv0Il79jw1RHUWyiXgvjatc7JR7kOFTnfO2isSkzxQ+c+plYc5NS0zbc1vczHf/i+Eoywp0BUlyOz0uKqaeDlgpR6hcgV4k9zgvYzaEOGlAaadfqXH+2BM3a10vRUCNT65eote7PxDWQ2YTfRf94rbWvnNOdD4Uw3EA64//iY0fjUAL062+cmd9SbwA1re91LbbqdaxyKu0GE1KTNI7ht6dMygwCXdi9rMkNnnQZlYupzGrfBaKO06R0czk7JBT1O8KfpMyeSjrlg== X-Forefront-PRVS: 0114FF88F6 X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10009020)(4630300001)(6009001)(7916002)(199003)(377424004)(189002)(81166006)(19580405001)(110136003)(305945005)(8676002)(105586002)(2950100002)(68736007)(7736002)(19580395003)(77096005)(7846002)(23676002)(42882006)(69596002)(2906002)(6916009)(5660300001)(97736004)(586003)(50226002)(103116003)(47776003)(3846002)(81156014)(2870700001)(36756003)(107886002)(42186005)(53416004)(106356001)(50466002)(450100001)(6116002)(92566002)(50986999)(76176999)(5820100001)(101416001)(2351001)(33646002)(189998001)(6666003)(66066001)(99106002);DIR:OUT;SFP:1101;SCL:1;SRVR:DM2PR0701MB1066;H:sellcey-lt.caveonetworks.com;FPR:;SPF:None;PTR:InfoNoRecords;A:1;MX:1;LANG:en; Received-SPF: None (protection.outlook.com: cavium.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: =?utf-8?B?MTtETTJQUjA3MDFNQjEwNjY7MjM6YjF2N3JVLyt5Q2NpK3llSEFuOVJoQ2Z1?= =?utf-8?B?empOZW41by9Md2h5dXlVK25XRWpGak1XWGQ4UmkzMFl3MVVHSmY5enZBV2Fh?= =?utf-8?B?VCtCelkzbkFhcHZHOVZZdmV4akRTTDJaZVpUMTBCUXVUTFpQbUgyNjlIT3pZ?= =?utf-8?B?czN3OE5aVndNQ3ZKbUY0Mmo4d1RDUXJKeUVERHJGSGdDVmtXSlZpbUxGYmJJ?= =?utf-8?B?TzZTS3l4NjBDNTJjQWhicE0xb3VWZHJmdVpqSU0wVllQdGpicHdMUnpVeFhh?= =?utf-8?B?V21lUnAxdWtYNGJHVitLVkRicy96ZSsxYnFiaCtZZ1M4UmVYdTY4Y3BrTWNV?= =?utf-8?B?U2NCR0poelJvWHhxV0VUVTY3bDJqY0J2YittSUp6YWxpd0JaN0s5SFc3ZVdU?= =?utf-8?B?MU0zVHFqZkZHZlpEKzBCSklHZkgyYThYTU9jU0VXV0NRTkNMcnZYdXM0YytT?= =?utf-8?B?V1V4QThaM092RVp1RTgxTnJUL1lwdHBUMDY1em9qOGFXaDgyQnRINlZENUp6?= =?utf-8?B?bWRmWTV0R0NVcGhyVDBuZk1qSTliQmVvZXU5Y3RVUWtmdFlVN3BHbEtHTjBx?= =?utf-8?B?SFI2N0MyUXM5K1ZKaUE2TXhUOGpLTktWb3hxdFdXeGhsMkdCMnl5ejVHYkJX?= =?utf-8?B?L1FlU1V1RWlWb0tFUFo3R3B5ZmFsbEFnTGgrbUFZQXJKU2lISTh3YTlicER1?= =?utf-8?B?MnZ5aVlJd1ZQZ1A4Y0VXMXhyMVJ2OFIxUmNMNG5YQ3NDOGxSd3dTVWlZUGRV?= =?utf-8?B?YUtid2QxeGQvVEZXWndiaGU0RStMdUxyMGt5aDROQkRnVlNDNmVwRHpZUzdK?= =?utf-8?B?NWZzNTh5MENlc3dwdWxrVzNNYjBNV3EzOGN4VXd3aE4yQzVSbFpqd000ME1F?= =?utf-8?B?dG1ML0RZd0xzNVdUWXloMGhybWMrQWVOM2ExOFI0ekozdjQ1MWJ1TytIdmNm?= =?utf-8?B?dlVIYXNLMjRGbkZTVFV3aTQraUZCeDJYaFJES1hNTDFiYjhMWlhkdEdwZ2hC?= =?utf-8?B?VXVMZVUvZzdNZnhZYm1GVXloMk1IOWZxNW9ST2ZKcTBKRlJCancwVWJTdnhm?= =?utf-8?B?dWxpN3BCY0ZaTXgvMEhURDFjdDFwbjRLQUNXK2tBSTNVR0krTVdabDVaYk1T?= =?utf-8?B?ZzNxV1pCQ2VaelphdWViQUNuL1N3cmdtWDYxM2NBRkY4SjlraDFiM0wvZG9C?= =?utf-8?B?akpVYzBsYjJnc0w2VFNPZmpNV2NFMlYyZS9JT0JEQXZzaUpxSTBvaWE5aTNM?= =?utf-8?B?QUtLeHd6Z1p4dzRxWFd2WG9hM2V5Y2Zra012STAyTFRCMW1GbDA5bHhwK0N1?= =?utf-8?B?RCtGSTZvZlpSdlZSWFpFTVVhRDlubi9aRTN1Z0ZUam4yMzAweERjVW00MzNN?= =?utf-8?B?Z3dJUWdlMkxmQ2NCU01jT2xjWUFpNG1oNXVsa1huTDI4TDlzUDI0Zmhpa0FL?= =?utf-8?B?b0hpci9VRExrWGFaMnpOTE55TDM2WkNLNU9uR3hRNU93VE9uMXo3TlNOdVdW?= =?utf-8?B?NUluRFpLc0JQeEMrRSt6ek5jWnd1Yy9RcHlxTzZQd3hTdFNLMEUvbkxGZG9B?= =?utf-8?B?ZnFsMnhOVVpNWFA1elJ6cDJJZkpLd2pMNWE3M0RVMjdqb0huOHY0K0FQL01z?= =?utf-8?B?Z3VMYlVLY1duVWFLZHFhT2o2eWRjZzdRT3AyRmpjRDFkV05HZHREcFBpZmdV?= =?utf-8?Q?fyKyAe72IA6BeYH6sNU/hhC+heMYUef5RUuFS8lDw?= X-Microsoft-Exchange-Diagnostics: 1;DM2PR0701MB1066;6:/Ujjo+7fGr2QNM9VuOUBhCZxV573sKqCdRj3cchHemCrPi6Lw4qmOTLeebLQ3hRmjazAuD46hs557c+VTgpSD8AdIgBqK7UGWFbWXoa7lnr2ZiNOaKP58dR8taj3AqTYG5dhrtVho8sWDgMFZi8RSnMBWcB2wc+RD1sPQ2c/BENP+qMmWuzFmCg6UPT0Bw9Ga95X7dSBKQLqhcbrcNiHaLNbrVzIospfAuRDCnC/EQiIyN+SgM+vply/jonpoMrpAcyvODD1EHL30VHiK29J+6EfBs5rtZWT0GSzsqXVCNwOgy6ydHeLnWviZv+jn6hA;5:z10F0l+j143SjyrUOtPdzQ4b72C/ANPQxJmlol8KC1r1DSmOFq8XSXJlFvjnWZONkzedsDQbPrTYzUDfkOBOKwNA0pURfCGe2z9tdiZ/lTWZ4b+PuCCSDxKfnNmw21FekcdS6mZOuU81KxJ6XabZww==;24:d8keOWXHFNm0KNgymBNBrkuGIc0TbSo7M8Z56dXSi9836Ms2Mxu/EtA7gHauG9/HnxDRt+2KYEykmMHqi75b+AgOjMRN9z8qJQVh8veVwKU= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1;DM2PR0701MB1066;7:2TxfClhA2RKleVlR7jdE/i77gfa50Hk4ZUTO1C4G7nm1y/vrrFAAd8692Y0xv39NZWcPoZbm1se9/1GuPKMlzg3bTCDyuPOJKXRiMfXQb08rBR84k+O6koRr5Tw3MyDzqNYtJzRijDbakI9UtHcjMNBAjf3gDOeriTKrw+fx8GrwKCvKLdxoG56l+rWXnjYC+g5xqCDVFT90F4RVtFBBcv0r5npGutWBccTC/ZBPx32Xp3lL2En4UWbfB9KfgP1mHMW9IVjn6aUYW/WFuA4hFjNEfNhcXHoKXcc9NaOPMbgHU+MXOcKJt+m2WCDSUiKWBmWqPKW6dbUCIydb5DWi36l5Ci4sIYdxhJWb6+j+y5k= X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Nov 2016 19:47:09.5916 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM2PR0701MB1066 X-SW-Source: 2016-11/txt/msg00086.txt.bz2 Here is an updated version of my earlier patch, I hadn't tested the original one with an old GCC so I hadn't realized I needed to put the DIAG_IGNORE_NEEDS_COMMENT in an ifdef.  This version was tested with GCC 5.4 and GCC 7.0 with no regressions. I also double checked that the test program is deliberately using snprintf to truncate the output so changing the test by increasing the buffer size and avoiding the warnings that way would change what was being tested. OK to checkin? 2016-11-02  Steve Ellcey   * time/tst-strptime2.c: Ignore -Wformat-length warning. diff --git a/time/tst-strptime2.c b/time/tst-strptime2.c index 7fe7350..53b366b 100644 --- a/time/tst-strptime2.c +++ b/time/tst-strptime2.c @@ -4,6 +4,7 @@  #include  #include  #include +#include    /* Dummy string is used to match strptime's %s specifier.  */   @@ -67,10 +68,20 @@ mkbuf (char *buf, bool neg, bool colon, unsigned int hhmm, s ize_t ndigits)    long int expect = LONG_MAX;      i = sprintf (buf, "%s %c", dummy_string, sign); +#if __GNUC_PREREQ (7, 0) +  /* GCC issues a warning when it thinks the snprintf buffer may be too short. +     This test is explicitly using short buffers to force snprintf to truncate +     the output so we ignore the warnings.  */ +  DIAG_PUSH_NEEDS_COMMENT; +  DIAG_IGNORE_NEEDS_COMMENT (7.0, "-Wformat-length"); +#endif    if (colon)      snprintf (buf + i, ndigits + 2, "%02u:%02u", hh, mm);    else      snprintf (buf + i, ndigits + 1, "%04u", hhmm); +#if __GNUC_PREREQ (7, 0) +  DIAG_POP_NEEDS_COMMENT; +#endif      if (mm <= mm_max && (ndigits == 2 || ndigits == 4))      {