Re: [dev] [dwm] code cleanup

On Tue, Jul 26, 2011 at 11:16:30PM +0400, anonymous wrote:
> Style fixes, removed
>
> missing = NULL;
>
> line. XCreateFontSet do it for you at the beginning of function [1].
> Added missing blank line, still 5 lines removed.
>
> dwm page [2] says "dwm is only a single binary, and its source code is
> intended to never exceed 2000 SLOC". wc -l says there are 2077 lines
> (2072 with patch). Some code should be removed or rewritten or dwm will
> grow over 2100 lines soon.
>

The output `wc -l` is a bad measure for the actual SLOC. It counts
comments, empty lines, wrapped lines etc. sloccount(1) says "dwm.c" from
the 5.9 release tarball has 1836 SLOC which is fine.

> [1] http://www.openbsd.org/cgi-bin/cvsweb/xenocara/lib/libX11/src/FSWrap.c?rev=1.3
> [2] http://dwm.suckless.org/
Received on Tue Jul 26 2011 - 21:22:23 CEST