[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] tools/xenstored: hashtable: Constify the parameters of hashfn/eqfn

To: Julien Grall <julien@xxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx
From: Juergen Gross <jgross@xxxxxxxx>
Date: Mon, 30 Jan 2023 07:15:15 +0100
Cc: Julien Grall <jgrall@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, Anthony PERARD <anthony.perard@xxxxxxxxxx>
Delivery-date: Mon, 30 Jan 2023 06:15:32 +0000
List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 27.01.23 19:55, Julien Grall wrote:

From: Julien Grall <jgrall@xxxxxxxxxx>

The parameters of hashfn/eqfn should never be modified. So constify
them and propagate the const to the users.

Take the opportunity to solve some coding style issues around the
code modified.

Signed-off-by: Julien Grall <jgrall@xxxxxxxxxx>


Reviewed-by: Juergen Gross <jgross@xxxxxxxx>


Juergen

Attachment: OpenPGP_0xB0DE9DD628BF132F.asc
Description: OpenPGP public key

Attachment: OpenPGP_signature
Description: OpenPGP digital signature

References:
- [PATCH] tools/xenstored: hashtable: Constify the parameters of hashfn/eqfn
  - From: Julien Grall

Prev by Date: RE: [PATCH v2 11/40] xen/mpu: build up start-of-day Xen MPU memory region map
Next by Date: RE: [PATCH v2 13/40] xen/mpu: introduce unified function setup_early_uart to map early UART
Previous by thread: RE: [PATCH] tools/xenstored: hashtable: Constify the parameters of hashfn/eqfn
Next by thread: [PATCH] xen/common: Constify the parameter of _spin_is_locked()
Index(es):
- Date
- Thread

Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.