On 3/13/24 08:26, Andrea della Porta wrote:
On 08:00 Wed 13 Mar , Florian Fainelli wrote:On 3/13/2024 7:08 AM, Andrea della Porta wrote:Fixed Codespell reported warnings about spelling and coding convention violations, among which there are also a couple potential operator precedence issue in macroes.
and s/macroes/macros/
Signed-off-by: Andrea della Porta <andrea.porta@xxxxxxxx>There are no spelling errors being fixed in this commit, this is purely stylistic and conforming to the Linux coding style guidelines.- /* detect a size missmatch */ - if (buf_len && (d->size != buf_len)) + /* detect a size mismatch */ + if (buf_len && d->size != buf_len) Isn't 'missmatch' a spelling error? Maybe I can drop the word 'several', since it's indeed only one...
Can we agree this was easy to miss when 99% of the changes are stylistic? The summary is that the commit subject and the message are not describing what this patch is about.
-- Florian
Attachment:
smime.p7s
Description: S/MIME Cryptographic Signature