[FFmpeg-devel] [RFC] switch to poll()

Luca Barbato lu_zero
Tue Nov 23 20:03:22 CET 2010


On 11/16/2010 06:14 PM, Martin Storsj? wrote:
> On Tue, 16 Nov 2010, Martin Storsj? wrote:
> 
>> Yes, but that's not what I meant. In patch #1, you do this:
>>
>> int j = 1 - (tcp_fd == -1);
>> [...] (code not touching j)
>> for (j = 1; j < max_p; j++)
>>
>> In that case, the complicated initialization of j is just confusing, since 
>> the initial value isn't used at all. In patch #2, that is of course 
>> totally correct, but there, you also move the line. That is, if applied 
>> separately, this shouldn't be in patch #1. If both of them are applied at 
>> once, this point is moot.
> 
> Actually, on second thought, the separate initialization above, int j = 1 
> - (tcp_fd == -1); is the correct one, you should just remove the j = 1 
> initialization in the for loop instead.

Right ^^;

Given it should work properly now, everybody is fine with the patchset?


lu

-- 

Luca Barbato
Gentoo/linux
http://dev.gentoo.org/~lu_zero




More information about the ffmpeg-devel mailing list