[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Bug-wget] Please review my patch for bug #29518
From: |
Leonard Ehrenfried |
Subject: |
[Bug-wget] Please review my patch for bug #29518 |
Date: |
Fri, 14 Jan 2011 11:04:24 +0100 |
Hi,
I've been poking around the wget bug tracker and implemented the following
feature request: https://savannah.gnu.org/bugs/?29518
I have put up a repo at launchpad here:
http://bazaar.launchpad.net/~leonard-ehrenfried/wget/purge-input-file/changes
The way I have implemented is, that the input file is re-read into a linked
list after each successful download (status==RETROK) and the the URL that
has just been read is not written back to the input file. Is that an
appropriate way to go about this?
I'd be greatful for a ruthless code review - it's the only way to learn!
Thanks
Lenni
PS: I might need a better name for the command line option:
'purge-input-file' sounds a bit drastic.
- [Bug-wget] Please review my patch for bug #29518,
Leonard Ehrenfried <=
- Re: [Bug-wget] Please review my patch for bug #29518, Giuseppe Scrivano, 2011/01/25
- Re: [Bug-wget] Please review my patch for bug #29518, Leonard Ehrenfried, 2011/01/26
- Re: [Bug-wget] Please review my patch for bug #29518, Leonard Ehrenfried, 2011/01/29
- Re: [Bug-wget] Please review my patch for bug #29518, Micah Cowan, 2011/01/29
- Re: [Bug-wget] Please review my patch for bug #29518, Leonard Ehrenfried, 2011/01/29
- Re: [Bug-wget] Please review my patch for bug #29518, Leonard Ehrenfried, 2011/01/29
- Re: [Bug-wget] Please review my patch for bug #29518, Giuseppe Scrivano, 2011/01/29