[MPlayer-dev-eng] Re: [PATCH] Incomplete patch for USF (correct diff) NOVIRUS

[Joey Parrish]

On Wed, Dec 10, 2003 at 09:27:54AM +0100, Moritz Bunkus wrote:
> +	if (!sscanf(line, " <subtitle start=\"%d:%d:%d.%d\" stop=\"%d:%d:%d.%d\"", &a1, &a2, &a3, &a4, &b1, &b2, &b3, &b4)) {
> 
> sscanf returns the number of items assigned, so you should better test
> for 'sscanf(...) != 8'.
that's the same thing.  if (!x) vs if (x != 0).

--Joey

-- 
"Of the seven dwarves, the only one who shaved was Dopey.
That should tell us something about the wisdom of shaving."