lynx-dev
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

LYNX-DEV Re: ... new temp patch


From: Klaus Weide
Subject: LYNX-DEV Re: ... new temp patch
Date: Mon, 14 Jul 1997 19:45:04 -0500 (CDT)

On Mon, 14 Jul 1997, Jonathan Sergent wrote:

> A new diff altogether is posted where the old one is.  This one should
> apply with patch -p1 (the other one needed patch -p5!)
> 
> The diffs are available at the io.com address I posted before.
> 
> Broken out trees are at http://csociety.ecn.purdue.edu/~sergent/lynx/
> where I have more space.

More problems with your code: 

It fails on blah.html.gz files - either local files, or served by HTTP
with headers

Content-encoding: x-gzip
Content-type: text/html

A more principal problem with your whole approach is that it creates
three(!) temp files for every attempt, of which 2 or all 3 will remain
unused and stay around until lynx exits. (or longer if lynx crashes).
So after 'd'ownloading 10 different files, you have up to 30 useless
temp files lying around, most of them empty.  I think this alone makes
your code unacceptable for a new lynx version.  I imagine there would
be lots of people reporting this as a bug, and I think they would be
right.

Maybe, if you want to continue with this, I can interest you to have
another look at <URL:http://www.slcc.edu/lynx/klaus/temp/>, and try
to improve on that...

   Klaus


;
; To UNSUBSCRIBE:  Send a mail message to address@hidden
;                  with "unsubscribe lynx-dev" (without the
;                  quotation marks) on a line by itself.
;

reply via email to

[Prev in Thread] Current Thread [Next in Thread]