On Wed, 2 May 2007, Jan Kratochvil wrote:

>   i found a bug in ff-memless.c so i fixed it. As you can see I am doing 
> check if effect_id is less then 0, but I am aware that it is useless 
> because effect_id is actually input_event.code which is __u16, but on 
> the other side as long as the effect_id is int I feel it is correct to 
> check whether it is > 0.

Dmitry,

actually, looking at the code - is there any particular reason for the 
inconsistency between the types used in struct input_event ( __u16 type; 
__u16 code) and in the rest of the code (basically unsigned int 
everywhere)?

Thanks,

-- 
Jiri Kosina
SUSE Labs

Reply via email to