Get rid of duplicate const declaration specifier warning in tst-resolv-qtypes.c.

Get rid of duplicate const declaration specifier warning in tst-resolv-qtypes.c.

[Stefan Liebler]

[-- Attachment #1: Type: text/plain, Size: 425 bytes --]

Hi,

Compiling resolv/tst-resolv-qtypes.c with GCC 7 results in:
tst-resolv-qtypes.c:53:14: error: duplicate ‘const’ declaration 
specifier [-Werror=duplicate-decl-specifier]
  static const const char *domain = "www.example.com";

This patch removes one of the two const specifiers.

Ok to commit (after release)?

Bye.
Stefan

ChangeLog:

	* resolv/tst-resolv-qtypes.c (domain):
	Remove duplicate const qualifier.

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 20170131_tst_resolv_qtypes_const.patch --]
[-- Type: text/x-patch; name="20170131_tst_resolv_qtypes_const.patch", Size: 1105 bytes --]

commit 8967a533335bff0feaa2abae242de94c928c9214
Author: Stefan Liebler <stli@linux.vnet.ibm.com>
Date:   Mon Jan 30 09:06:44 2017 +0100

    Get rid of duplicate const declaration specifier warning in tst-resolv-qtypes.c.
    
    Compiling resolv/tst-resolv-qtypes.c with GCC 7 results in:
    tst-resolv-qtypes.c:53:14: error: duplicate ‘const’ declaration specifier [-Werror=duplicate-decl-specifier]
     static const const char *domain = "www.example.com";
    
    This patch removes one of the two const specifiers.
    
    ChangeLog:
    
    	* resolv/tst-resolv-qtypes.c (domain):
    	Remove duplicate const qualifier.

diff --git a/resolv/tst-resolv-qtypes.c b/resolv/tst-resolv-qtypes.c
index dcb39e5..2d91e92 100644
--- a/resolv/tst-resolv-qtypes.c
+++ b/resolv/tst-resolv-qtypes.c
@@ -50,7 +50,7 @@ response (const struct resolv_response_context *ctx,
   resolv_response_close_record (b);
 }
 
-static const const char *domain = "www.example.com";
+static const char *domain = "www.example.com";
 
 static int
 wrap_res_query (int type, unsigned char *answer, int answer_length)

Re: Get rid of duplicate const declaration specifier warning in tst-resolv-qtypes.c.

[Andreas Schwab]

On Jan 31 2017, Stefan Liebler <stli@linux.vnet.ibm.com> wrote:

>  static const const char *domain = "www.example.com";

domain should be const.

Andreas.

-- 
Andreas Schwab, schwab@linux-m68k.org
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."

Re: Get rid of duplicate const declaration specifier warning in tst-resolv-qtypes.c.

[Stefan Liebler]

[-- Attachment #1: Type: text/plain, Size: 295 bytes --]

On 01/31/2017 11:23 AM, Andreas Schwab wrote:
> On Jan 31 2017, Stefan Liebler <stli@linux.vnet.ibm.com> wrote:
>
>>  static const const char *domain = "www.example.com";
>
> domain should be const.
>
> Andreas.
>
Okay.

Here is the updated patch.

Okay to commit (after release)?

Bye.
Stefan


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 20170131_1610_tst_resolv_qtypes_const.patch --]
[-- Type: text/x-patch; name="20170131_1610_tst_resolv_qtypes_const.patch", Size: 1169 bytes --]

commit 322394d80126fb540026818bb81f0df79d2fa3cf
Author: Stefan Liebler <stli@linux.vnet.ibm.com>
Date:   Mon Jan 30 09:06:44 2017 +0100

    Get rid of duplicate const declaration specifier warning in tst-resolv-qtypes.c.
    
    Compiling resolv/tst-resolv-qtypes.c with GCC 7 results in:
    tst-resolv-qtypes.c:53:14: error: duplicate ‘const’ declaration specifier [-Werror=duplicate-decl-specifier]
     static const const char *domain = "www.example.com";
    
    This patch removes the duplicate const and makes domain a const pointer
    to const char literal.
    
    ChangeLog:
    
    	* resolv/tst-resolv-qtypes.c (domain):
    	Change type to const pointer to const char.

diff --git a/resolv/tst-resolv-qtypes.c b/resolv/tst-resolv-qtypes.c
index dcb39e5..06ea3db 100644
--- a/resolv/tst-resolv-qtypes.c
+++ b/resolv/tst-resolv-qtypes.c
@@ -50,7 +50,7 @@ response (const struct resolv_response_context *ctx,
   resolv_response_close_record (b);
 }
 
-static const const char *domain = "www.example.com";
+static const char * const domain = "www.example.com";
 
 static int
 wrap_res_query (int type, unsigned char *answer, int answer_length)

[PING][PATCH] Re: Get rid of duplicate const declaration specifier warning in tst-resolv-qtypes.c.

[Stefan Liebler]

On 01/31/2017 04:13 PM, Stefan Liebler wrote:
> On 01/31/2017 11:23 AM, Andreas Schwab wrote:
>> On Jan 31 2017, Stefan Liebler <stli@linux.vnet.ibm.com> wrote:
>>
>>>  static const const char *domain = "www.example.com";
>>
>> domain should be const.
>>
>> Andreas.
>>
> Okay.
>
> Here is the updated patch.
>
> Okay to commit (after release)?
>
> Bye.
> Stefan
>

ping

Re: [PING 2][PATCH] Re: Get rid of duplicate const declaration specifier warning in tst-resolv-qtypes.c.

[Stefan Liebler]

On 02/08/2017 03:52 PM, Stefan Liebler wrote:
> On 01/31/2017 04:13 PM, Stefan Liebler wrote:
>> On 01/31/2017 11:23 AM, Andreas Schwab wrote:
>>> On Jan 31 2017, Stefan Liebler <stli@linux.vnet.ibm.com> wrote:
>>>
>>>>  static const const char *domain = "www.example.com";
>>>
>>> domain should be const.
>>>
>>> Andreas.
>>>
>> Okay.
>>
>> Here is the updated patch.
>>
>> Okay to commit (after release)?
>>
>> Bye.
>> Stefan
>>
>
> ping
>
ping

Re: Get rid of duplicate const declaration specifier warning in tst-resolv-qtypes.c.

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 1700 bytes --]

On Tue, Jan 31, 2017 at 04:13:29PM +0100, Stefan Liebler wrote:
> On 01/31/2017 11:23 AM, Andreas Schwab wrote:
> >On Jan 31 2017, Stefan Liebler <stli@linux.vnet.ibm.com> wrote:
> >
> >> static const const char *domain = "www.example.com";
> >
> >domain should be const.
> >
> >Andreas.
> >
> Okay.
> 
> Here is the updated patch.
> 
> Okay to commit (after release)?
> 
> Bye.
> Stefan
> 
> commit 322394d80126fb540026818bb81f0df79d2fa3cf
> Author: Stefan Liebler <stli@linux.vnet.ibm.com>
> Date:   Mon Jan 30 09:06:44 2017 +0100
> 
>     Get rid of duplicate const declaration specifier warning in tst-resolv-qtypes.c.
>     
>     Compiling resolv/tst-resolv-qtypes.c with GCC 7 results in:
>     tst-resolv-qtypes.c:53:14: error: duplicate ‘const’ declaration specifier [-Werror=duplicate-decl-specifier]
>      static const const char *domain = "www.example.com";
>     
>     This patch removes the duplicate const and makes domain a const pointer
>     to const char literal.
>     
>     ChangeLog:
>     
>     	* resolv/tst-resolv-qtypes.c (domain):
>     	Change type to const pointer to const char.
> 
> diff --git a/resolv/tst-resolv-qtypes.c b/resolv/tst-resolv-qtypes.c
> index dcb39e5..06ea3db 100644
> --- a/resolv/tst-resolv-qtypes.c
> +++ b/resolv/tst-resolv-qtypes.c
> @@ -50,7 +50,7 @@ response (const struct resolv_response_context *ctx,
>    resolv_response_close_record (b);
>  }
>  
> -static const const char *domain = "www.example.com";
> +static const char * const domain = "www.example.com";
>  
>  static int
>  wrap_res_query (int type, unsigned char *answer, int answer_length)

Yes, this is OK.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 801 bytes --]

Re: Get rid of duplicate const declaration specifier warning in tst-resolv-qtypes.c.

[Stefan Liebler]

On 02/24/2017 10:57 AM, Dmitry V. Levin wrote:
> On Tue, Jan 31, 2017 at 04:13:29PM +0100, Stefan Liebler wrote:
>> On 01/31/2017 11:23 AM, Andreas Schwab wrote:
>>> On Jan 31 2017, Stefan Liebler <stli@linux.vnet.ibm.com> wrote:
>>>
>>>> static const const char *domain = "www.example.com";
>>>
>>> domain should be const.
>>>
>>> Andreas.
>>>
>> Okay.
>>
>> Here is the updated patch.
>>
>> Okay to commit (after release)?
>>
>> Bye.
>> Stefan
>>
>> commit 322394d80126fb540026818bb81f0df79d2fa3cf
>> Author: Stefan Liebler <stli@linux.vnet.ibm.com>
>> Date:   Mon Jan 30 09:06:44 2017 +0100
>>
>>     Get rid of duplicate const declaration specifier warning in tst-resolv-qtypes.c.
>>
>>     Compiling resolv/tst-resolv-qtypes.c with GCC 7 results in:
>>     tst-resolv-qtypes.c:53:14: error: duplicate ‘const’ declaration specifier [-Werror=duplicate-decl-specifier]
>>      static const const char *domain = "www.example.com";
>>
>>     This patch removes the duplicate const and makes domain a const pointer
>>     to const char literal.
>>
>>     ChangeLog:
>>
>>     	* resolv/tst-resolv-qtypes.c (domain):
>>     	Change type to const pointer to const char.
>>
>> diff --git a/resolv/tst-resolv-qtypes.c b/resolv/tst-resolv-qtypes.c
>> index dcb39e5..06ea3db 100644
>> --- a/resolv/tst-resolv-qtypes.c
>> +++ b/resolv/tst-resolv-qtypes.c
>> @@ -50,7 +50,7 @@ response (const struct resolv_response_context *ctx,
>>    resolv_response_close_record (b);
>>  }
>>
>> -static const const char *domain = "www.example.com";
>> +static const char * const domain = "www.example.com";
>>
>>  static int
>>  wrap_res_query (int type, unsigned char *answer, int answer_length)
>
> Yes, this is OK.
>
>
Committed.

Thanks.
Stefan

Re: Get rid of duplicate const declaration specifier warning in tst-resolv-qtypes.c.

[H.J. Lu]

On Fri, Feb 24, 2017 at 1:57 AM, Dmitry V. Levin <ldv@altlinux.org> wrote:
> On Tue, Jan 31, 2017 at 04:13:29PM +0100, Stefan Liebler wrote:
>> On 01/31/2017 11:23 AM, Andreas Schwab wrote:
>> >On Jan 31 2017, Stefan Liebler <stli@linux.vnet.ibm.com> wrote:
>> >
>> >> static const const char *domain = "www.example.com";
>> >
>> >domain should be const.
>> >
>> >Andreas.
>> >
>> Okay.
>>
>> Here is the updated patch.
>>
>> Okay to commit (after release)?
>>
>> Bye.
>> Stefan
>>
>> commit 322394d80126fb540026818bb81f0df79d2fa3cf
>> Author: Stefan Liebler <stli@linux.vnet.ibm.com>
>> Date:   Mon Jan 30 09:06:44 2017 +0100
>>
>>     Get rid of duplicate const declaration specifier warning in tst-resolv-qtypes.c.
>>
>>     Compiling resolv/tst-resolv-qtypes.c with GCC 7 results in:
>>     tst-resolv-qtypes.c:53:14: error: duplicate ‘const’ declaration specifier [-Werror=duplicate-decl-specifier]
>>      static const const char *domain = "www.example.com";
>>
>>     This patch removes the duplicate const and makes domain a const pointer
>>     to const char literal.
>>
>>     ChangeLog:
>>
>>       * resolv/tst-resolv-qtypes.c (domain):
>>       Change type to const pointer to const char.
>>
>> diff --git a/resolv/tst-resolv-qtypes.c b/resolv/tst-resolv-qtypes.c
>> index dcb39e5..06ea3db 100644
>> --- a/resolv/tst-resolv-qtypes.c
>> +++ b/resolv/tst-resolv-qtypes.c
>> @@ -50,7 +50,7 @@ response (const struct resolv_response_context *ctx,
>>    resolv_response_close_record (b);
>>  }
>>
>> -static const const char *domain = "www.example.com";
>> +static const char * const domain = "www.example.com";
>>
>>  static int
>>  wrap_res_query (int type, unsigned char *answer, int answer_length)
>
> Yes, this is OK.
>

I'd like to backport it to 2.25 branch.   Any objections?

Thanks.

-- 
H.J.

Re: Get rid of duplicate const declaration specifier warning in tst-resolv-qtypes.c.

[Paul Eggert]

H.J. Lu wrote:
> -static const const char *domain = "www.example.com";
> +static const char * const domain = "www.example.com";

Why not:

static const char domain[] = "www.example.com";

instead, before you backport?  It's simpler and less confusing.

Re: Get rid of duplicate const declaration specifier warning in tst-resolv-qtypes.c.

[H.J. Lu]

On Thu, Sep 7, 2017 at 10:38 AM, Paul Eggert <eggert@cs.ucla.edu> wrote:
> H.J. Lu wrote:
>>
>> -static const const char *domain = "www.example.com";
>> +static const char * const domain = "www.example.com";
>
>
> Why not:
>
> static const char domain[] = "www.example.com";
>
> instead, before you backport?  It's simpler and less confusing.

If we want to do this, it should go into master branch first and
backport it 2.26 branch before putting in 2.25 branch.    Should
we do it?


-- 
H.J.

Re: Get rid of duplicate const declaration specifier warning in tst-resolv-qtypes.c.

[Florian Weimer]

* H. J. Lu:

> On Thu, Sep 7, 2017 at 10:38 AM, Paul Eggert <eggert@cs.ucla.edu> wrote:
>> H.J. Lu wrote:
>>>
>>> -static const const char *domain = "www.example.com";
>>> +static const char * const domain = "www.example.com";
>>
>>
>> Why not:
>>
>> static const char domain[] = "www.example.com";
>>
>> instead, before you backport?  It's simpler and less confusing.
>
> If we want to do this, it should go into master branch first and
> backport it 2.26 branch before putting in 2.25 branch.    Should
> we do it?

I have no objections.

Re: Get rid of duplicate const declaration specifier warning in tst-resolv-qtypes.c.

[Paul Eggert]

H.J. Lu wrote:
>> Why not:
>>
>> static const char domain[] = "www.example.com";
>>
>> instead, before you backport?  It's simpler and less confusing.

> If we want to do this, it should go into master branch first and
> backport it 2.26 branch before putting in 2.25 branch.    Should
> we do it?

Yes, that's the idea.

Re: Get rid of duplicate const declaration specifier warning in tst-resolv-qtypes.c.

[H.J. Lu]

[-- Attachment #1: Type: text/plain, Size: 525 bytes --]

On Thu, Sep 7, 2017 at 12:51 PM, Paul Eggert <eggert@cs.ucla.edu> wrote:
> H.J. Lu wrote:
>>>
>>> Why not:
>>>
>>> static const char domain[] = "www.example.com";
>>>
>>> instead, before you backport?  It's simpler and less confusing.
>
>
>> If we want to do this, it should go into master branch first and
>> backport it 2.26 branch before putting in 2.25 branch.    Should
>> we do it?
>
>
> Yes, that's the idea.

Here is the patch.  Tested with GCC 7 on x86-64.

OK for master and 2.25/2.26 branches?

Thanks.


-- 
H.J.

[-- Attachment #2: 0001-Use-static-const-char-domain.patch --]
[-- Type: text/x-patch, Size: 846 bytes --]

From cb99c41d24dbba967974ac6ed3d1a23dc7f66cfd Mon Sep 17 00:00:00 2001
From: "H.J. Lu" <hjl.tools@gmail.com>
Date: Thu, 7 Sep 2017 13:02:36 -0700
Subject: [PATCH] Use "static const char domain[] ="

	* resolv/tst-resolv-qtypes.c (domain): Changed to
	"const char domain[] =".
---
 resolv/tst-resolv-qtypes.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/resolv/tst-resolv-qtypes.c b/resolv/tst-resolv-qtypes.c
index 06ea3dbd14..da3325f80c 100644
--- a/resolv/tst-resolv-qtypes.c
+++ b/resolv/tst-resolv-qtypes.c
@@ -50,7 +50,7 @@ response (const struct resolv_response_context *ctx,
   resolv_response_close_record (b);
 }
 
-static const char * const domain = "www.example.com";
+static const char domain[] = "www.example.com";
 
 static int
 wrap_res_query (int type, unsigned char *answer, int answer_length)
-- 
2.13.5

Re: Get rid of duplicate const declaration specifier warning in tst-resolv-qtypes.c.

[Paul Eggert]

H.J. Lu wrote:
> OK for master and 2.25/2.26 branches?

Thanks, it looks good.