Mailinglist Archive: zypp-devel (53 mails)
| < Previous | Next > |
Re: [zypp-devel] Re: MediaAria2c merged
- From: Michael Matz <matz@xxxxxxx>
- Date: Mon, 6 Oct 2008 18:13:17 +0200 (CEST)
- Message-id: <Pine.LNX.4.64.0810061812400.6093@xxxxxxxxxxxxx>
Hi,
On Mon, 6 Oct 2008, Duncan Mac-Vicar Prett wrote:
No, these are carefully crafted to do slightly different things
everywhere, and hence can't share any code, of course :-))
Ciao,
Michael.
--
To unsubscribe, e-mail: zypp-devel+unsubscribe@xxxxxxxxxxxx
For additional commands, e-mail: zypp-devel+help@xxxxxxxxxxxx
On Mon, 6 Oct 2008, Duncan Mac-Vicar Prett wrote:
Michael Matz wrote:
But it would be very nice if someone cleaned up the code ofYes, that is planned.
MediaAria2c.cc. From a cursory read some weeks ago it is mostly a c&p of
MediaCurl.cc (and in fact still uses curl for most of it's things, except
of course for the crucial one of downloading files), and exchanged only a
couple of functions. I'd much rather see MediaAria2c be a subclass of
MediaCurl reusing much of its code.
Code duplication is bad.Does that include string join and split functions? ;-)
No, these are carefully crafted to do slightly different things
everywhere, and hence can't share any code, of course :-))
Ciao,
Michael.
--
To unsubscribe, e-mail: zypp-devel+unsubscribe@xxxxxxxxxxxx
For additional commands, e-mail: zypp-devel+help@xxxxxxxxxxxx
| < Previous | Next > |