[prev in list] [next in list] [prev in thread] [next in thread] 

List:       nas
Subject:    Re: [nas] [patch] clients cleanup
From:       Jon Trulson <jon () radscan ! com>
Date:       2006-08-21 1:44:18
Message-ID: Pine.LNX.4.61.0608201943190.9722 () m5 ! radscan ! com
[Download RAW message or body]

On Sun, 20 Aug 2006, Stefan Huehner wrote:

> Hi,
> 
> attached patch cleans up several of the client programs. Mostly adding
> const to parameter/variable declarations. Additionally an unsused variable was \
> removed and some prototype in audemo.c were fixed. 

         Applied, thanks!

> While modifying i noticed that at least the fatalError method is
> included in every client program as a copy. What about extracting this
> and perhaps more common code (i.e. argument parsing) in an extra object
> file and linking this into each binary?
> 

         So a small 'clinet' library for these clients?  Sure - I'm
         all for removing duplication :)



-- 
Jon Trulson
mailto:jon@radscan.com http://radscan.com/~jon
#include <std/disclaimer.h>
"No Kill I" -Horta


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic