[x264-devel] [PATCH] MPEG2 Program Stream output

[Lorenzo Pallara]

Hi all,
	I didn't get any more feedback for the patch since some days, I updated 
it to the current git, it should have the changes suggested up to now, 
you can find it in attach.

best regards,
	Lorenzo

On 01/26/2011 12:55 PM, Lorenzo Pallara wrote:
> Jason Garrett-Glaser wrote:
>> Nitpicks:
>
> [...]
>
> They should be all fix now...
>
>>> + return (unsigned long long int) ( x + 0.5 );
>>> + return (unsigned long long int) ( x - 0.5 );
>>
>> Use uint64_t not this crap. Does it really have to handle negatives?
>
> good question, I thought about it and it shouldn't happen to have
> negative values so I removed the < 0 case and the function call too
> adding always the + 0.5
>
> the new version is in attach, it changed the file output
> description as pointed out by Gil Pedersen the other description could
> be not proper.
>
> best regards,
> Lorenzo
>
>
>
>
> _______________________________________________
> x264-devel mailing list
> x264-devel at videolan.org
> http://mailman.videolan.org/listinfo/x264-devel

-------------- next part --------------
A non-text attachment was scrubbed...
Name: pes.patch
Type: text/x-patch
Size: 8805 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/x264-devel/attachments/20110223/e3dd9454/attachment.bin>