[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