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/
