[ofa-general] Re: [PATCH] opensm/osm_subnet.c: fixing compiler warnings

Sasha Khapyorsky sashak at voltaire.com
Thu Mar 12 07:17:47 PDT 2009


On 14:02 Thu 12 Mar     , Yevgeny Kliteynik wrote:
> Hi Sasha,
> 
> There are couple of meaningless CL_ASSERT() lines in the
> parsing functions that are checking the parsed value.
> The compiler says that "comparison is always true due
> to limited range of data type". So either the 'val' type
> should be more general when parsing it, or the CL_ASSERT()
> lines are not needed. The following patch removes them.
> 
> Signed-off-by: Yevgeny Kliteynik <kliteyn at dev.mellanox.co.il>

Applied. Thanks.

Sasha



More information about the general mailing list