[IRCServices Coding] coding... (svcs 5)

[Andrew Church]

>On Sat, 15 Dec 2001, Andrew Church wrote:
>> >For services 5:
>> >I believe code like this (from operserv/main.c: do_killclones() )
>[...]
>> >should exist in seperate functions like:
>> >int kill_mask(const char *mask, const char *reason)
>[...]
>> >}
>>
>>      Not worth it.
>
>Why not? Wouldn't this help code re-use in 3rd party modules? One wouldn't
>have to re-invent the wheel, and if there was an error, it would only have
>to be fixed in one place..

     If it was actually being used in multiple places, I'd agree with you,
but (unless I'm forgetting somewhere) it's not, and splitting the code up
anyway would be a waste of time.  If this actually becomes an issue later
I'll reconsider it then.

  --Andrew Church
    achurch at achurch.org
    http://achurch.org/