<!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 myself,</p>
<p>On 2016-11-18 03:59, Filip Roséen wrote:</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> The previous implementation assumed that the data passed was a local
 path, which limited the usability of the function (effectively
 preventing subtitles passed as URIs from being added).

 These changes allow both local paths and URIs to be handled by the
 function, effectively making sure that file://... are subject to
 transformation (while leaving other entities untouched).

 refs #17657</code></pre>
</blockquote>
<p>I hope you realize that you just submitted the <em>work-in-progress</em> patch instead of what you actually wanted to submit (since you forgot to <code>git commit --amend</code>)?</p>
<p>As I am being the responsible (awake) second half, please see the attached patch for one that is actually correct (and not just a draft).</p>
<p>Best Regards,<br />
Me, Myself, and I.</p>
</body>
</html>