<!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 <code>vlc-devel</code>,</p>
<p>Attached to this post are <code>8</code> patches where <code>5</code> have been updated to comply with the desired changes as denoted in the below linked posts (by <em>Rémi Denis-Courmount</em>):</p>
<ul>
<li>https://mailman.videolan.org/pipermail/vlc-devel/2017-February/111570.html</li>
<li>https://mailman.videolan.org/pipermail/vlc-devel/2017-February/111571.html</li>
</ul>
<p>To summarize the differences between the originally proposed implementation, and the now attached revision, is:</p>
<ul>
<li><p>Merge <em>struct-declaration</em> and <em>typedef specifier</em> for <code>stream_extractor_t</code> and <code>stream_directory_t</code> in <code>include/vlc_stream_extractor.h</code>.</p></li>
<li><p>Fix unfortunate typos related to documentation of the implementation (“” vs a single backslash).</p></li>
<li><p>Include rationale for legacy code removal in commit message (it now mentions that the relevant functionality is handled by the introduced <em>libarchive</em> based <em>stream-extractor</em>).</p></li>
</ul>
<p>I expect these patches to be easily viewed at <em>patches.videolan.org</em>, and with a little bit of luck the below will refer to this very content (but I am basically just guessing now):</p>
<ul>
<li>https://patches.videolan.org/patch/15672/</li>
</ul>
<p>If anything b0rkes out, the relevant branch is also available here:</p>
<ul>
<li>https://github.com/FilipRoseen-refp/vlc/tree/stream_extractor-split</li>
</ul>
<p>Best Regards,<br />
Filip Roséen</p>
<p>On 2017-02-16 22:44, Filip Roséen wrote:</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> These set of patches are a direct adjustment to the thoughts expressed in the
replies to the previously submitted patch-batch (linked below):
- https://mailman.videolan.org/pipermail/vlc-devel/2017-January/111193.html
- https://mailman.videolan.org/pipermail/vlc-devel/2017-February/111354.html
Those patches that are affected by replies shall have a "REVISION #N" marker,
where the details of the changes and rationale are listed.
"Revision #1" corresponds to the first thread (as linked previously in this cover
letter), while "Revision #2" refers to the second thread (about the unnecessary
joint capability for the equivalent of pf_readdir vs pf_{read,block}).
Thanks to *Hugo Beauzée-Luyseen* and *Rémi Denis-Courmount* for their feedback.
Filip Roséen (8):
stream_extractor: split joint capability into two
modules/stream_extractor: added libarchive extractor
gui/skins2: add ThemeLoader::unarchive
gui/skins2: use ThemeLoader::unarchive
gui/skins2: remove legacy archive dependencies
modules/access: remove legacy rar access
modules/access: removal of legacy archive access/filter
access: remove legacy zip accessor/stream</code></pre>
</blockquote>
</body>
</html>