Subject: CVS commit: pkgsrc/editors/mflteco
From: David A. Holland
Date: 2012-12-27 07:13:20
Message id: 20121227061320.EC060175DD@cvs.netbsd.org

Log Message:
Clean up legacy C.
Now passes gcc -Wall -Wextra -Wmissing-declarations -Wwrite-strings on gcc45.
May build with clang now too.

Changes:
   - use standard headers
   - don't declare own errno, fopen(), getenv(), or malloc()
   - declare own functions
   - declare void functions void
   - avoid implicit int
   - use C89
   - use some const
   - use some static
   - fix signed/unsigned mismatches
   - call execl() properly
   - don't cast return value of malloc
   - initialize struct sigaction correctly
   - use <ctype.h> functions correctly
   - don't index arrays with (signed) char
   - don't put function declarations inside function bodies
   - patch up two uninitialized variables
   - silence some compiler warnings that are not bugs
   - remove some unused objects
and:
   - add patch comments for preexisting patches
   - correct some minor pkglint

As a couple of these things fixed bugs (e.g. the execl calls), bump
PKGREVISION.

Files:
RevisionActionfile
1.7modifypkgsrc/editors/mflteco/Makefile
1.3modifypkgsrc/editors/mflteco/distinfo
1.2modifypkgsrc/editors/mflteco/patches/patch-aa
1.2modifypkgsrc/editors/mflteco/patches/patch-ab
1.2modifypkgsrc/editors/mflteco/patches/patch-ac
1.2modifypkgsrc/editors/mflteco/patches/patch-ad
1.2modifypkgsrc/editors/mflteco/patches/patch-ae
1.1addpkgsrc/editors/mflteco/patches/patch-te__data.c
1.1addpkgsrc/editors/mflteco/patches/patch-te__exec0.c
1.1addpkgsrc/editors/mflteco/patches/patch-te__exec1.c
1.1addpkgsrc/editors/mflteco/patches/patch-te__fxstub.c
1.1addpkgsrc/editors/mflteco/patches/patch-te__main.c
1.1addpkgsrc/editors/mflteco/patches/patch-te__srch.c
1.1addpkgsrc/editors/mflteco/patches/patch-te__subs.c
1.1addpkgsrc/editors/mflteco/patches/patch-te__utils.c
1.1addpkgsrc/editors/mflteco/patches/patch-te__window.c