[DVDnav-discuss] [PATCH] Use 64bits file IO API and set to binary mode on OS/2

Erik Hovland erik at hovland.org
Thu Sep 3 17:11:05 CEST 2009


>>> This patch makes to use 64bits file IO API and to set a file handle to a
>>> binary mode explicitly on OS/2.
>>>
>>
>> Please separate functional and cosmetical changes in two patches. (You may
>> send
>> the cosmetic change after the first patch was approved.)
>>
>>
>
> The cosmetic change is caused by a functional change, that is, DosOpenL
> instead of DosOpen.  Nevertheless, should I separate the patch in two parts
> ?

I personally don't think this one requires two patches. There is no
trailing whitespace and the style of the patch mixes w/ the style
of the rest of the function. Sadly the function doesn't match the
style of the rest of the file. But don't submit a patch for that.

Barring any dissent from the cognizanti this patch will be committed
to libdvdnav in about 48 hours.

E

-- 
Erik Hovland
erik at hovland.org
http://hovland.org/


More information about the DVDnav-discuss mailing list