[vlc-commits] demux: adaptive: remove redundant check of CRLF

Zhao Zhili git at videolan.org
Tue Oct 17 19:35:35 CEST 2017


vlc | branch: master | Zhao Zhili <quinkblack at foxmail.com> | Thu Oct 12 15:30:35 2017 +0800| [ef281573e6d24520c9579f5b0f57f17e75940c22] | committer: Francois Cartegnie

demux: adaptive: remove redundant check of CRLF

CRLF is stripped by readLine(), and the following code doesn't deal with
CRLF, so the check is redundant and suspicious.

Signed-off-by: Francois Cartegnie <fcvlcdev at free.fr>

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=ef281573e6d24520c9579f5b0f57f17e75940c22
---

 modules/demux/adaptive/http/HTTPConnection.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/modules/demux/adaptive/http/HTTPConnection.cpp b/modules/demux/adaptive/http/HTTPConnection.cpp
index 5934fb7270..6fb2093a2f 100644
--- a/modules/demux/adaptive/http/HTTPConnection.cpp
+++ b/modules/demux/adaptive/http/HTTPConnection.cpp
@@ -239,7 +239,7 @@ int HTTPConnection::parseReply()
     for( ;; )
     {
         std::string l = readLine();
-        if(l.empty() || l.compare("\r\n") == 0)
+        if(l.empty())
             break;
         lines.append(l);
 



More information about the vlc-commits mailing list