# HG changeset patch # User erik # Date 1251855263 0 # Node ID 3aa39a1cff0f9bd4dc1d2e636b84713b68fb48b7 # Parent ef679522d00fd60f12842c1840216d977c1f95a6 Remove empty if clause. An if clause is empty. This makes the else do all of the work and still ends up w/ a larger object file because the if clause is stuck into the object file. By moving to the logical not of the if we can remove the empty clause. diff -r ef679522d00f -r 3aa39a1cff0f dvd_reader.c --- a/dvd_reader.c Tue Aug 04 21:21:49 2009 +0000 +++ b/dvd_reader.c Wed Sep 02 01:34:23 2009 +0000 @@ -882,9 +882,7 @@ int i; if( dvd_file ) { - if( dvd_file->dvd->isImageFile ) { - ; - } else { + if( !dvd_file->dvd->isImageFile ) { for( i = 0; i < TITLES_MAX; ++i ) { if( dvd_file->title_devs[ i ] ) { dvdinput_close( dvd_file->title_devs[i] );