<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">

<html xmlns="http://www.w3.org/1999/xhtml">

<head>

  <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />

  <meta http-equiv="Content-Style-Type" content="text/css" />

  <meta name="generator" content="pandoc" />

  <title></title>

  <style type="text/css">code{white-space: pre;}</style>

</head>

<body>

<p>Hi,</p>

<p>On 2016-11-06 19:25, Pierre Ynard wrote:</p>

<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">

<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">

<pre><code> These changes adds "URL" as a property to the metadata-table,

 containing what the name implies.</code></pre>

</blockquote>

<pre><code> I'm not too excited by adding fields that are not metadata i.e.

 arbitrary descriptive strings, to the lua metadata array. And there is

 already a URL metadata field with an arbitrary value relating to the

 content, so that would be confusing.</code></pre>

</blockquote>

<p>When I noticed that <code>"filename"</code> was there I thought that maybe there was a exception to the “rule” regarding what should be in the metadata table. Though I fully agree with you.</p>

<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">

<pre><code> Anyway, I think the functionality is already available through

 vlc.input.item():uri()</code></pre>

</blockquote>

<p>That is a very good point, as such I agree with you that the second patch should be discarded (in favor of using <code>vlc.input.item():uri()</code>).</p>

<p>Thanks,<br />

Filip</p>

</body>

</html>