L'octidi 8 fructidor, an CCXXIV, liu jc a écrit : > Signed-off-by: liujingchao <jc...@outlook.com> > --- > libavformat/tcp.c | 10 ++++++++++ > 1 file changed, 10 insertions(+) > mode change 100644 => 100755 libavformat/tcp.c > > diff --git a/libavformat/tcp.c b/libavformat/tcp.c > old mode 100644 > new mode 100755 > index c105479..61b68ca > --- a/libavformat/tcp.c > +++ b/libavformat/tcp.c > @@ -122,6 +122,16 @@ static int tcp_open(URLContext *h, const char *uri, int > flags) > cur_ai = ai; > > restart: > + > + if (cur_ai->ai_family == AF_INET6) > + { > + struct sockaddr_in6 * sockaddr_v6 = (struct sockaddr_in6 > *)cur_ai->ai_addr; > + if (sockaddr_v6->sin6_port == 0) > + { > + sockaddr_v6->sin6_port = htons(port); > + } > + }
As already pointed out: Please respect the surrounding coding style. From my point of view, this is dead code that can be removed. A comment is needed to avoid that. Obvious breaks from the coding style: braces placement, braces on trivial statements, comparison to 0, pointer mark placement. > + > fd = ff_socket(cur_ai->ai_family, > cur_ai->ai_socktype, > cur_ai->ai_protocol);
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel