# HG changeset patch # User erik # Date 1251855798 0 # Node ID 6177a05fa534113aae67a2446cf168d400d4e2ed # Parent 3aa39a1cff0f9bd4dc1d2e636b84713b68fb48b7 Use parens to make the 'is present' mask explicit W/o parens the check for the 'is present' mask will not work right because the ! operator will evaluate first. But by wrapping the & operation in parens, we make it explicit that this goes first. And so the is present flag will be properly evaluated in the conditional diff -r 3aa39a1cff0f -r 6177a05fa534 ifo_read.c --- a/ifo_read.c Wed Sep 02 01:34:23 2009 +0000 +++ b/ifo_read.c Wed Sep 02 01:43:18 2009 +0000 @@ -844,10 +844,10 @@ /* verify time (look at print_time) */ for(i = 0; i < 8; i++) - if(!pgc->audio_control[i] & 0x8000) /* The 'is present' bit */ + if(!(pgc->audio_control[i] & 0x8000)) /* The 'is present' bit */ CHECK_ZERO(pgc->audio_control[i]); for(i = 0; i < 32; i++) - if(!pgc->subp_control[i] & 0x80000000) /* The 'is present' bit */ + if(!(pgc->subp_control[i] & 0x80000000)) /* The 'is present' bit */ CHECK_ZERO(pgc->subp_control[i]); /* Check that time is 0:0:0:0 also if nr_of_programs == 0 */