with the changes bluhm suggested OK florian@

On Thu, Feb 04, 2016 at 02:36:25PM +0100, Alexander Bluhm wrote:
> On Thu, Feb 04, 2016 at 02:02:46PM +0100, J??r??mie Courr??ges-Anglas wrote:
> > +static void usage(void);
> 
> Can you make the usage static __dead void?
> 
> > +#define OPTIONS ":c:ds"
> 
> Our other daemons don't have a leading ':', why to be special here?
> 
> >     while ((ch = getopt(argc, argv, OPTIONS)) != -1) {
> 
> This OPTIONS define looks quite useless.  Why not put the string here?
> 
> anyway, OK bluhm@
> 

-- 
I'm not entirely sure you are real.

Reply via email to