blk_name() is not modifying data passed to it through pointer and it
returns also a pointer to const so the argument can be made const for
code safeness.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
block/block-backend.c | 2 +-
include/sysemu/block-backend.h | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/block/block-backend.c b/block/block-backend.c
index daa7908d0151..2d2169a5dfb0 100644
--- a/block/block-backend.c
+++ b/block/block-backend.c
@@ -409,7 +409,7 @@ void monitor_remove_blk(BlockBackend *blk)
* Return @blk's name, a non-null string.
* Returns an empty string iff @blk is not referenced by the monitor.
*/
-const char *blk_name(BlockBackend *blk)
+const char *blk_name(const BlockBackend *blk)
{
return blk->name ?: "";
}
diff --git a/include/sysemu/block-backend.h b/include/sysemu/block-backend.h
index 096c17fce064..d16c85d67b5c 100644
--- a/include/sysemu/block-backend.h
+++ b/include/sysemu/block-backend.h
@@ -91,7 +91,7 @@ int blk_get_refcnt(BlockBackend *blk);
void blk_ref(BlockBackend *blk);
void blk_unref(BlockBackend *blk);
void blk_remove_all_bs(void);
-const char *blk_name(BlockBackend *blk);
+const char *blk_name(const BlockBackend *blk);
BlockBackend *blk_by_name(const char *name);
BlockBackend *blk_next(BlockBackend *blk);
bool monitor_add_blk(BlockBackend *blk, const char *name, Error **errp);
--
2.9.3
On 03/05/2017 06:44 PM, Krzysztof Kozlowski wrote: > blk_name() is not modifying data passed to it through pointer and it > returns also a pointer to const so the argument can be made const for > code safeness. > > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org> Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> > --- > block/block-backend.c | 2 +- > include/sysemu/block-backend.h | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/block/block-backend.c b/block/block-backend.c > index daa7908d0151..2d2169a5dfb0 100644 > --- a/block/block-backend.c > +++ b/block/block-backend.c > @@ -409,7 +409,7 @@ void monitor_remove_blk(BlockBackend *blk) > * Return @blk's name, a non-null string. > * Returns an empty string iff @blk is not referenced by the monitor. > */ > -const char *blk_name(BlockBackend *blk) > +const char *blk_name(const BlockBackend *blk) > { > return blk->name ?: ""; > } > diff --git a/include/sysemu/block-backend.h b/include/sysemu/block-backend.h > index 096c17fce064..d16c85d67b5c 100644 > --- a/include/sysemu/block-backend.h > +++ b/include/sysemu/block-backend.h > @@ -91,7 +91,7 @@ int blk_get_refcnt(BlockBackend *blk); > void blk_ref(BlockBackend *blk); > void blk_unref(BlockBackend *blk); > void blk_remove_all_bs(void); > -const char *blk_name(BlockBackend *blk); > +const char *blk_name(const BlockBackend *blk); > BlockBackend *blk_by_name(const char *name); > BlockBackend *blk_next(BlockBackend *blk); > bool monitor_add_blk(BlockBackend *blk, const char *name, Error **errp); >
Am 05.03.2017 um 22:44 hat Krzysztof Kozlowski geschrieben: > blk_name() is not modifying data passed to it through pointer and it > returns also a pointer to const so the argument can be made const for > code safeness. > > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org> Thanks, applied to block-next for qemu 2.10. Kevin
© 2016 - 2024 Red Hat, Inc.