[DVDnav-discuss] r1176 - trunk/libdvdread/src/dvd_reader.c

[Nico Sabbi]

Il mercoledì 2 settembre 2009 03:34:23 erik ha scritto:
> Author: erik
> Date: Wed Sep  2 03:34:23 2009
> New Revision: 1176
>
> Log:
> 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.
>
> Modified:
>    trunk/libdvdread/src/dvd_reader.c
>
> Modified: trunk/libdvdread/src/dvd_reader.c
> ===================================================================
>=========== --- trunk/libdvdread/src/dvd_reader.c	Tue Aug  4
> 23:21:49 2009	(r1175) +++ trunk/libdvdread/src/dvd_reader.c	Wed Sep
>  2 03:34:23 2009	(r1176) @@ -882,9 +882,7 @@ void DVDCloseFile(
> dvd_file_t *dvd_file 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] );


another example of the quality of these libs ;-(