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

Martin Storsjö martin
Tue Nov 16 18:14:31 CET 2010


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.

// Martin



More information about the ffmpeg-devel mailing list