[DVDnav-discuss] please review: DVDFileStat, close dev on error, use parens and BSD porting bits

Erik Hovland erik at hovland.org
Mon Aug 31 18:05:04 CEST 2009


> fine, but please fix those horrible
> - if (  x  )
> and
> if (  y  ) {
>  empty line
>
> }
>
> syntaxes.

I hate to bother, but could you be more specific? Does it mean
you prefer 'if(x)' over if( x )? The empty line I get.

Attached is a new DVDFileStat patch w/ the whitespace
removed and the parens format made to match that of the
functions on either side of the newly add functions.

I am assuming that since niether Diego or Nico has said
anything else about the other three patches, that they are
generally OK.

E

-- 
Erik Hovland
erik at hovland.org
http://hovland.org/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Add-DVD-file-stat-from-version-0.9.7-to-libdvdread.patch
Type: text/x-diff
Size: 7982 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/dvdnav-discuss/attachments/20090831/afa559bf/attachment.patch>


More information about the DVDnav-discuss mailing list