Charles Levert <[EMAIL PROTECTED]> writes: > + unsigned u = kwset->mind - (i + 1);
A minor point: the usual GNU style is to say "unsigned int" rather than plain "unsigned". Another point, still minor (though less minor): Why not just use "int"? That will work here; no overflow is possible, since both values are nonnegative. In general it's better to stick with the data types that the rest of the code is using, and in this case it's int.