[dm-devel] [PATCH 3/6] libmultipath, alloc_path_with_pathinfo(): Declare third argument const
Martin Wilck
mwilck at suse.com
Mon Mar 5 16:18:59 UTC 2018
On Thu, 2018-03-01 at 11:29 -0800, Bart Van Assche wrote:
> Since the third argument of this function is not modified inside that
> function, declare it const.
>
> Signed-off-by: Bart Van Assche <bart.vanassche at wdc.com>
ACK, but this hunk is part of my previously posted patch "libmultipath:
const qualifier for wwid and alias", which includes some more changes
along similar lines and is part of a larger patch set that tries to
improve "const" usage in libmultipath.
> ---
> libmultipath/discovery.c | 2 +-
> libmultipath/discovery.h | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/libmultipath/discovery.c b/libmultipath/discovery.c
> index fe50ce5062ea..d84715e15db1 100644
> --- a/libmultipath/discovery.c
> +++ b/libmultipath/discovery.c
> @@ -34,7 +34,7 @@
>
> int
> alloc_path_with_pathinfo (struct config *conf, struct udev_device
> *udevice,
> - char *wwid, int flag, struct path
> **pp_ptr)
> + const char *wwid, int flag, struct path
> **pp_ptr)
> {
> int err = PATHINFO_FAILED;
> struct path * pp;
> diff --git a/libmultipath/discovery.h b/libmultipath/discovery.h
> index b151cc7a39a8..bd5e6678a26d 100644
> --- a/libmultipath/discovery.h
> +++ b/libmultipath/discovery.h
> @@ -38,7 +38,7 @@ int get_state (struct path * pp, struct config *
> conf, int daemon, int state);
> int get_vpd_sgio (int fd, int pg, char * str, int maxlen);
> int pathinfo (struct path * pp, struct config * conf, int mask);
> int alloc_path_with_pathinfo (struct config *conf, struct
> udev_device *udevice,
> - char *wwid, int flag, struct path
> **pp_ptr);
> + const char *wwid, int flag, struct
> path **pp_ptr);
> int store_pathinfo (vector pathvec, struct config *conf,
> struct udev_device *udevice, int flag,
> struct path **pp_ptr);
--
Dr. Martin Wilck <mwilck at suse.com>, Tel. +49 (0)911 74053 2107
SUSE Linux GmbH, GF: Felix Imendörffer, Jane Smithard, Graham Norton
HRB 21284 (AG Nürnberg)
More information about the dm-devel
mailing list