[SLOF] [PATCH slof 00/13] Compile with -Wextra

Alexey Kardashevskiy aik at ozlabs.ru
Thu Jan 28 15:32:35 AEDT 2021



On 28/01/2021 14:52, Segher Boessenkool wrote:
> On Thu, Jan 28, 2021 at 01:23:24PM +1100, Alexey Kardashevskiy wrote:
>> On 28/01/2021 10:02, Segher Boessenkool wrote:
>>> On Wed, Jan 27, 2021 at 07:57:39PM +1100, Alexey Kardashevskiy wrote:
>>>> Following the conversation at
>>>> https://patchwork.ozlabs.org/project/slof/patch/20210122172133.2074177-1-f4bug@amsat.org/
>>>> this enables -Wextra with the exception of -Wno-unused-parameter.
>>>>
>>>> This is a series to make possible bisecting easier.
>>>>
>>>>
>>>> This is based on sha1
>>>> 6184ca06c8d8 Gustavo Romero "board-qemu: Fix comment about SLOF start
>>>> address".
>>>>
>>>> Please comment. Thanks.
>>>
>>> I commented on 2/13.  I also don't like the casts in 11/13, but the code
>>> does seem to do the right thing.
>>>
>>> 1/13 is actually a bugfix, no?
>>
>>
>> I did not even realize that, good eye! :)
> 
> I typoed that, wow...  12/13!  (It decreases "size" in every fallthrough
> case, which might or might now work, bu is surely not intended).

First I thought it is for when the size is unaligned but it is not :)


> 
>>> Thank you for doing this,
>>
>> I kinda have to, as a slof emperor :)
> 
> There are emperors now?  I'm jealous!

Want to own SLOF again? ;)


-- 
Alexey


More information about the SLOF mailing list