[vlc-commits] sdp: drop URL and email

Rémi Denis-Courmont git at videolan.org
Fri Apr 3 17:32:39 CEST 2020


vlc | branch: master | Rémi Denis-Courmont <remi at remlab.net> | Wed Apr  1 22:02:37 2020 +0300| [4900e37a6528d9836a38eb0006f6ede93d5708d0] | committer: Rémi Denis-Courmont

sdp: drop URL and email

This is about as useless as the phone number which was dropped in 3.0.
At least, the VLC SDP parser ignores those fields.

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

 src/stream_output/sdp.c | 24 +-----------------------
 1 file changed, 1 insertion(+), 23 deletions(-)

diff --git a/src/stream_output/sdp.c b/src/stream_output/sdp.c
index 0c9ca23e42..304433beaf 100644
--- a/src/stream_output/sdp.c
+++ b/src/stream_output/sdp.c
@@ -148,29 +148,7 @@ int vlc_sdp_Start(struct vlc_memstream *restrict stream,
     else
         vlc_memstream_printf(stream, "i=%s\r\n", "N/A");
 
-    strcpy(subvar, "url");
-    str = var_GetNonEmptyString(obj, varname);
-    if (str != NULL)
-    {
-        if (!IsSDPString(str))
-            goto error;
-
-        vlc_memstream_printf(stream, "u=%s\r\n", str);
-        free(str);
-    }
-
-    strcpy(subvar, "email");
-    str = var_GetNonEmptyString(obj, varname);
-    if (str != NULL)
-    {
-        if (!IsSDPString(str))
-            goto error;
-
-        vlc_memstream_printf(stream, "e=%s\r\n", str);
-        free(str);
-    }
-
-    // no phone (useless)
+    // no URL, email, no phone (useless)
 
     vlc_memstream_printf(stream, "c=%s\r\n", connection);
     // bandwidth not specified



More information about the vlc-commits mailing list