<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01//EN" "http://www.w3.org/TR/html4/strict.dtd">
<html><head></head><body><div style="font-size: 12pt; font-family: Calibri,sans-serif;"><div>Havent had time to read the patches but in any case fix the bug before you fix the test. Unless you want git-bisect to point the finger at you all the time.</div><div><br></div><div>-- </div><div>Rémi Denis-Courmont</div><div>Sent from my NVIDIA Tegra-powered device</div><br><div id="htc_header">----- Reply message -----<br>De : "Hugo Beauzée-Luyssen" <hugo@beauzee.fr><br>Pour : <vlc-devel@videolan.org><br>Objet : [vlc-devel] [PATCH 0/2] Fix URL option parsing<br>Date : mar., mars 10, 2015 16:45</div></div><br><pre style="word-wrap: break-word; white-space: pre-wrap;">Hi,
vlc_UrlParse's options parsing seems to be broken.
The attached addition to the test suite shows that psz_option is always NULL,
which makes sense since we're using strchr on a string that has been forced
to contain nothing but a '\0'
Hugo Beauzée-Luyssen (2):
Add vlc_UrlParse tests
text: url: Fix options parsing
src/test/url.c | 26 ++++++++++++++++++++++++++
src/text/url.c | 2 +-
2 files changed, 27 insertions(+), 1 deletion(-)
--
2.1.0
_______________________________________________
vlc-devel mailing list
To unsubscribe or modify your subscription options:
<a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a>
</pre></body></html>