<html><head></head><body>Hi,<br><br>I think you're missing the point here. You can't just pass opaque data through filters, and you don't control what header lines get sent or when. This is the block flags discussion all over again. What you're doing just isn't a good fit with the stream abstraction.<br><br>Besides, how will this work if the HTTP access does not have a special case for sending the ICY header line in requests??<br><br><div class="gmail_quote">Le 12 août 2019 15:26:27 GMT+03:00, Marvin Scholz <epirat07@gmail.com> a écrit :<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<pre class="k9mail">On 12 Aug 2019, at 14:00, Rémi Denis-Courmont wrote:<br><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;"> This looks like horrible violation of abstraction and layering. We <br> already have higher level getters for metadata.<br><br></blockquote><br>I considered using metadata for this but there were some issues:<br><br>- How would you ensure HTTP headers never collide with other metadata <br>keys?<br>- How to handle that HTTP headers are case-insensitive? Just lowercase <br>them?<br><br>I am happy to rewrite this patchset using stream metadata instead if <br>thats considered a better approach.<br><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;">Le 12 août 2019 12:56:06 GMT+03:00, Marvin Scholz <br><epirat07@gmail.com> a écrit :<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #ad7fa8; padding-left: 1ex;"><hr> include/vlc_stream.h | 1 +<br> 1 file changed, 1 insertion(+)<br><br> diff --git a/include/vlc_stream.h b/include/vlc_stream.h<br> index bc6c0d30b7..1b6186baeb 100644<br> --- a/include/vlc_stream.h<br> +++ b/include/vlc_stream.h<br> @@ -168,6 +168,7 @@ enum stream_query_e<br> STREAM_GET_CONTENT_TYPE, /**< arg1= char ** res=can fail <br> */<br> STREAM_GET_SIGNAL, /**< arg1=double *pf_quality, arg2=double<br> *pf_strength res=can fail */<br> STREAM_GET_TAGS, /**< arg1=const block_t ** res=can fail <br> */<br> + STREAM_GET_HTTP_HEADER, /**< arg1=const char *header arg2=char<br> ** res=can fail */<br><br> STREAM_SET_PAUSE_STATE = 0x200, /**< arg1= bool res=can fail <br> */<br> STREAM_SET_TITLE, /**< arg1= int res=can fail */<br> -- <br> 2.19.1<hr> vlc-devel mailing list<br> To unsubscribe or modify your subscription options:<br> <a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a><br></blockquote>-- <br>Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez <br>excuser ma brièveté._______________________________________________<br>vlc-devel mailing list<br>To unsubscribe or modify your subscription options:<br><a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a><br></blockquote><hr>vlc-devel mailing list<br>To unsubscribe or modify your subscription options:<br><a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a></pre></blockquote></div><br>-- <br>Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.</body></html>