> - if (tw_dev->srb[request_id] != 0) {> + if (tw_dev->srb[request_id]) {For pointers we should use == NULL or != NULL.Removing the comparison removes the warning but it doesn't replace itwith any useful typechecking as can be doing with NULL set as (void *)0.This is thus a backward step.Alan