Declare void argument lists
kern/mach_clock.c: update comment

This is mach_clock.c, not clock_prim.c.

* kern/mach_clock.c: Update comment.
i386/i386/ldt.c: remove forward declaration

* i386/i386/ldt.c (syscall): Remove forward declaration.
Include locore.h.
* i386/i386/locore.h (syscall): Add prototype.
vm/vm_kern.c (kmem_submap): remove unnecessary cast

The return value from vm_map_min() is already of vm_offset_t type.

* vm/vm_kern.c (kmem_submap) (addr): Remove unnecessary cast.
vm/vm_resident.c (vm_page_print): remove unnecessary casts

Members offset and phys_addr are of vm_offset_t types.

* vm/vm_resident.c (vm_page_print) (offset, phys_addr): Remove unnecessary casts.
ddb/db_break.c (db_find_breakpoint_here): remove unnecessary casts

Variable addr and member address are already of db_addr_t type which
is type defined as vm_offset_t.

* ddb/db_break.c (db_find_breakpoint_here) (DB_PHYS_EQ) (addr, address): Remove unecessary casts.
ddb/db_break.c (db_delete_cmd): remove unnecessary initialization

Now that we have returns there are no more warnings from GCC about
uninitialized variable. Remove unnecessary initialization of variable
bkpt.

* ddb/db_break.c (db_delete_cmd) (bkpt): Remove unnecessary initialization.
ddb/db_expr.c (db_mult_expr): initialize lhs

Initialize lhs to zero to avoid uninitialized usage in db_unary().

* ddb/db_expr.c (db_mult_expr) (lhs): Initialize to zero.
Type definition

* i386/i386/ast_check.c (init_ast_check, cause_ast_check): Define return type.
* i386/i386/pic.c (intnull, prtnull): Define argument types.
* i386/i386at/com.c (compr_addr): Likewise.
(compr): Likewise.
(compr_addr): Fix printf format.
* i386/i386at/kd.c (kd_cmdreg_write, kd_kbd_magic): Define argument types.
* i386/i386at/kd_mouse.c (init_mouse_hw): Likewise.
* ipc/mach_port.c (sact_count): Define return type.
* ipc/mach_rpc.c (mach_port_rpc_sig): Define argument types.
* kern/act.c (dump_act): Define return type.
* kern/lock_mon.c (simple_lock, simple_lock_try, simple_unlock, lip, lock_info_sort, lock_info_clear, print_lock_info): Define return type.
(time_lock): Define return type and argument type.
* kern/timer.c (time_trap_uexit): Define argument type.
Fix gcc signedness warning

Thanks Marin Ramesa and Richard Braun for investigating.

* i386/i386at/kd.c (kdintr): Make `char_idx' and `max' unsigned.
(kdstate2idx): Make function take and return unsigned ints for the state.
* i386/i386at/kd.h (kdstate2idx): Likewise.
ddb: qualify pointers whose dereferenced values are constant with const
kern/sched_prim.c: avoid casts

Avoid the casts by passing the address of the links thread
structure member to enqueue_tail().

* kern/sched_prim.c: Avoid casts.
device/dev_pager.c: remove unnecessary casts

Variable pager is already of ipc_port_t type.

* device/dev_pager.c: Remove unnecessary casts.
device/dev_pager.c (device_pager_data_request_done): cast to size_t instead to unsigned

* device/dev_pager.c (device_pager_data_request_done) (memset) (io_residual): Cast to size_t instead to unsigned.
i386/i386/db_trace.c: use (long *) instead of an (int *)

* i386/i386/db_trace.c (db_lookup_i386_kreg) (kregp): Use (long *) instead of an (int *).
Quiet GCC warning about uninitialized variable

* ddb/db_command.h (db_error): Mark with attribute noreturn.
* i386/i386/setjmp.h (_longjmp): Likewise.
kern: quiet GCC warnings about set but unused variables

* kern/lock.h (simple_lock_data_empty): Define.
(decl_simple_lock_data, simple_unlock): Likewise.
* kern/sched_prim.c (lock): Declare.
[MACH_SLOCKS]: Remove #ifs.
* kern/task.c (task_lock, task_unlock): Remove address operator.