[Skiboot] [PATCH 1/2] libflash/libffs: Rename ffs_cmp() to ffs_equal()
Stewart Smith
stewart at linux.vnet.ibm.com
Tue Jan 12 16:57:22 AEDT 2016
Cyril Bur <cyril.bur at au1.ibm.com> writes:
> Some confusion has arisen from the first consumer of ffs_cmp() in that they
> expected a strcmp style less than one, zero or greater than one return value.
>
> This has been addressed in this patch in two ways, by changing the return
> type to a boolean in order to (attempt) to alert the programmer that this
> is not the case and by renaming it to equal to avoid the implied comparison
> and imply very much a boolean outcome.
>
> Signed-off-by: Cyril Bur <cyril.bur at au1.ibm.com>
> ---
> libflash/libffs.c | 2 +-
> libflash/libffs.h | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
If I was a better person, I'd have said this before merging it
originally - so I'll go sit in the corner looking stupid for a
while. It's okay, the Corner has coffee[1]
Series merged as of 1663987
(assuming this didn't need to go to stable)
[1] while in Melbourne, The Corner Hotel has live music and beer!
--
Stewart Smith
OPAL Architect, IBM.
More information about the Skiboot
mailing list