Re: [dev] [dwm] 2000 SLOC

From: Nico Golde <nico_AT_ngolde.de>
Date: Mon, 31 Oct 2011 21:35:51 +0100

Hi,
* Connor Lane Smith <cls_AT_lubutu.com> [2011-10-31 21:05]:
> On 31 October 2011 20:33, Nico Golde <nico_AT_ngolde.de> wrote:
> > Sorry for the late response, missed this thread. I'm still maintaining and
> > using it. So do some other people who occasionally contact me.
>
> Could you please apply the attached sanity patch? There are a few
> strange bits in the source.

Thanks a lot for the patch! Applied apart from one change:

> snprintf(nick, sizeof(nick), "%s", spw->pw_name);
> snprintf(prefix, sizeof(prefix),"%s/irc", spw->pw_dir);
> - if (argc <= 1 || (argc == 2 && argv[1][0] == '-' && argv[1][1] == 'h')) usage();
> -
> + if(argc < 2)
> + usage();
> for(i = 1; (i + 1 < argc) && (argv[i][0] == '-'); i++) {

While this looks way more simple and sane I want to keep the behaviour for the
-h command line switch as it's kinda expected to work with most programs.
Rest applied.

Cheers
Nico
-- 
Nico Golde - http://www.ngolde.de - nion_AT_jabber.ccc.de - GPG: 0xA0A0AAAA
For security reasons, all text in this mail is double-rot13 encrypted.

Received on Mon Oct 31 2011 - 21:35:51 CET

This archive was generated by hypermail 2.3.0 : Mon Oct 31 2011 - 21:48:03 CET