Remove the misguided encoding option for VIM
Cast the trailing NULL on execl() calls

From exec(3): ?The list of arguments must be terminated by a NULL
pointer, and, since these are variadic functions, this pointer must be
cast (char *) NULL.?

This prevents crashes on 64-bit systems, where 0 is a 32-bit integer
and (char *) NULL is a 64-bit pointer.

Signed-off-by: Anders Kaseorg <andersk at mit.edu>
Handle editor commands with arguments

Previously when the EDITOR environment variable is set to a command
with arguments, such as ?emacsclient --alternate-editor emacs?, ldapvi
would fail to launch the editor:

$ ldapvi
     26 entries read
error (misc.c line 180): No such file or directory
editor died
error (ldapvi.c line 83): No such file or directory

Fix this by launching the editor via /bin/sh.

Signed-off-by: Anders Kaseorg <andersk at mit.edu>
Handle pager commands with arguments

This handles PAGER in the same way the last commit handled EDITOR.

Signed-off-by: Anders Kaseorg <andersk at mit.edu>
Merge remote-tracking branch 'lichtenblau/master'

Catch up.