string: Amend grammar and add description <byteswap.h>
Commit Message
Please find attached patch created against latest git.
Regards, Jonny
Comments
On 10/26/20 10:57 AM, Jonny Grant wrote:
> Please find attached patch created against latest git.
Thank you very much for the patch!
I don't see an assignment on file for you for "GLIBC."
Just for clarity...
Could you please confirm that you would like to submit this patch
under the exception for copyright relevance?
That is to say that these changes can't really be copyrighted
and that they are below the 15 line exemption even if they were.
On 26/10/2020 15:32, Carlos O'Donell wrote:
> On 10/26/20 10:57 AM, Jonny Grant wrote:
>> Please find attached patch created against latest git.
>
> Thank you very much for the patch!
>
> I don't see an assignment on file for you for "GLIBC."
>
> Just for clarity...
>
> Could you please confirm that you would like to submit this patch
> under the exception for copyright relevance?
>
> That is to say that these changes can't really be copyrighted
> and that they are below the 15 line exemption even if they were.
>
Hi Carlos
Yes, happy to submit under the exception.
Happy to submit copyright assignment for GLIBC as well for the future.
Jonny
On 10/26/20 11:44 AM, Jonny Grant wrote:
>
>
> On 26/10/2020 15:32, Carlos O'Donell wrote:
>> On 10/26/20 10:57 AM, Jonny Grant wrote:
>>> Please find attached patch created against latest git.
>>
>> Thank you very much for the patch!
>>
>> I don't see an assignment on file for you for "GLIBC."
>>
>> Just for clarity...
>>
>> Could you please confirm that you would like to submit this patch
>> under the exception for copyright relevance?
>>
>> That is to say that these changes can't really be copyrighted
>> and that they are below the 15 line exemption even if they were.
>>
>
> Hi Carlos
>
> Yes, happy to submit under the exception.
>
> Happy to submit copyright assignment for GLIBC as well for the future.
Jonny,
It would be great if you had a future assignment in place which would
allow me to just accept these patches without further review.
Thank you again for the patch.
On 10/26/20 10:57 AM, Jonny Grant wrote:
> Please find attached patch created against latest git.
>
> Regards, Jonny
>
OK for master. Obvious improvement. I've pushed this.
Removed Signed-off-by: line (we don't use it in glibc), but
added my Reviewed-by line.
For more details see:
https://sourceware.org/glibc/wiki/Contribution%20checklist
Reviewed-by: Carlos O'Donell <carlos@redhat.com>
On 26/10/2020 15:55, Carlos O'Donell wrote:
> On 10/26/20 10:57 AM, Jonny Grant wrote:
>> Please find attached patch created against latest git.
>>
>> Regards, Jonny
>>
>
> OK for master. Obvious improvement. I've pushed this.
>
> Removed Signed-off-by: line (we don't use it in glibc), but
> added my Reviewed-by line.
>
> For more details see:
> https://sourceware.org/glibc/wiki/Contribution%20checklist
>
> Reviewed-by: Carlos O'Donell <carlos@redhat.com>
>
Thank you Carlos.
I couldn't see exactly what to do next from the wiki, so I've emailed assign@gnu.org to ask them what I should do next for GLIBC copyright assignment.
Jonny
On 10/26/20 4:53 PM, Jonny Grant wrote:
> I couldn't see exactly what to do next from the wiki, so I've emailed
> assign@gnu.org to ask them what I should do next for GLIBC copyright
> assignment.
Download this:
http://git.savannah.gnu.org/cgit/gnulib.git/plain/doc/Copyright/request-assign.future
Fill in the details and send to assign@gnu.org to kick off the future
assignment process.
If you have any questions please don't hesitate to ask for help.
From db2140094c0948b2dca801aa1253af0aeb8241e7 Mon Sep 17 00:00:00 2001
From: Jonny Grant <jg@jguk.org>
Date: Mon, 26 Oct 2020 14:55:14 +0000
Subject: [PATCH] Amend grammar and add a description
Signed-off-by: Jonny Grant <jg@jguk.org>
---
string/byteswap.h | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
@@ -1,4 +1,5 @@
-/* Copyright (C) 1997-2020 Free Software Foundation, Inc.
+/* Swap byte order for 16, 32 and 64 bit values
+ Copyright (C) 1997-2020 Free Software Foundation, Inc.
This file is part of the GNU C Library.
The GNU C Library is free software; you can redistribute it and/or
@@ -24,10 +25,10 @@
#include <bits/byteswap.h>
-/* The following definitions must all be macros since otherwise some
+/* The following definitions must all be macros, otherwise some
of the possible optimizations are not possible. */
-/* Return a value with all bytes in the 16 bit argument swapped. */
+/* Return a value with both bytes in the 16 bit argument swapped. */
#define bswap_16(x) __bswap_16 (x)
/* Return a value with all bytes in the 32 bit argument swapped. */
--
2.25.1