<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<meta content="text/html;charset=UTF-8" http-equiv="Content-Type">
</head>
<body bgcolor="#ffffff" text="#000000">
Carl-Daniel Hailfinger wrote:
<blockquote cite="mid:47B48895.20702@gmx.net" type="cite">
<pre wrap="">On 14.02.2008 19:15, Myles Watson wrote:
</pre>
<blockquote type="cite">
<pre wrap="">This patch removes elfboot and references to ELF from v3. Since lar
parses ELF, there's no need to have v3 parse ELF. The ELF files that
v3 could parse should always have been a subset of the ones that lar
could parse anyway.
Myles
Signed-off-by: Myles Watson <a class="moz-txt-link-rfc2396E" href="mailto:myles@pel.cs.byu.edu"><myles@pel.cs.byu.edu></a>
</pre>
</blockquote>
<pre wrap=""><!---->
Patches which remove stuff are always very welcome. A few comments below.
</pre>
</blockquote>
<br>
... as long as they do not remove functionality that is still used. <br>
<br>
Please don't just remove this code. If you don't like to compile it in,
create a config option to disable it. (There is such a config option
already, so I really don't see the gain)<br>
<br>
I say: NACK.<br>
<br>
Why? <br>
<br>
Because with this patch it is no longer possible to unpack a lar.<br>
<br>
Please don't do stuff like that with levity.<br>
<br>
Stefan<br>
<br>
<br>
<br>
<pre class="moz-signature" cols="72">--
coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br.
Tel.: +49 761 7668825 • Fax: +49 761 7664613
Email: <a class="moz-txt-link-abbreviated" href="mailto:info@coresystems.de">info@coresystems.de</a> • <a class="moz-txt-link-freetext" href="http://www.coresystems.de/">http://www.coresystems.de/</a>
Registergericht: Amtsgericht Freiburg • HRB 7656
Geschäftsführer: Stefan Reinauer • Ust-IdNr.: DE245674866
</pre>
</body>
</html>