On Fri, Mar 28, 2014 at 05:49:11PM +0000, Alan Barrett wrote:
> Module Name: src
> Committed By: apb
> Date: Fri Mar 28 17:49:11 UTC 2014
>
> Modified Files:
> src/games/hunt/huntd: get_names.c
>
> Log Message:
> If you are going to use memcpy instead of struct assignment, then at
> least use sizeof(the correct variable). The use of memcmp to test
> two pointers for equality, one line earlier, is also weird but may be
> correct.
My todo file for huntd (dated 2009) says:
get_remote_name():
his_address, his_name, his_machine_name, ptr should be const
really stupid memcmp() call
following memcpy is also wrong
I have no idea why I didn't fix it when I found it.
--
David A. Holland
[email protected]
