Re: [PATCH] v850: const-qualify first parameter offind_next_zero_bit

From: Bernd Petrovitsch
Date: Wed Jul 20 2005 - 03:47:25 EST


On Wed, 2005-07-20 at 17:38 +0900, Miles Bader wrote:
[...]
> @@ -157,7 +157,7 @@
> #define find_first_zero_bit(addr, size) \
> find_next_zero_bit ((addr), (size), 0)
>
> -extern __inline__ int find_next_zero_bit (void *addr, int size, int offset)
> +extern __inline__ int find_next_zero_bit(const void *addr, int size, int offset)
> {
> unsigned long *p = ((unsigned long *) addr) + (offset >> 5);

Why not const-qualify *p and the cast also (avoiding warnings and
actually making the change complete)?

Bernd
--
Firmix Software GmbH http://www.firmix.at/
mobil: +43 664 4416156 fax: +43 1 7890849-55
Embedded Linux Development and Services

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/