cvs commit: ALFS/nALFS/src/handlers download.c

Jamie Bennett jamie_bennett at pcpmicro.co.uk
Wed Feb 4 00:08:56 PST 2004


Kevin P. Fleming wrote on 04 February 2004 01:38
>Jamie, do you want to update the CVS DTD to match these changes or 
>should I do that?

I'll do it.

>>   +static int download_main_3_2(element_s *el)
>
>I could be really picky and say that the bulk of this function could 
>have been factored into a third do-the-work function and both *_main 
>functions could have been just setup/teardown code, but this looks fine 
>and will certainly do the job.

Agreed, its cleaner that way. I'd rather get it right than just have a
working example so I'll change that tonight.

...
Jamie Bennett	-	jamie at linuxfromscratch dot org



More information about the alfs-log mailing list