Hi Everyone,
After researching this some more I realized that I think this integer
array may be in this structure for padding/data alignment. When I
compile this on a 64-bit computer with -Wpadded I do not receive any
errors regarding this structure, but I think this must have been an
issue with other architectures since it was put there in the first
place! I apologize for not realizing this before posting. I was
wondering--is this something the compiler/processor should adjust for
on modern systems?
Sorry for any inconvenience and thanks for your time,
Peter
On Tue, Dec 26, 2017 at 2:28 PM, Peter Williams <pete...@gmail.com> wrote:
> Hi Everyone,
>
> I recently noticed that there is an unused variable in gomoku.h. I
> have included a diff that removes the unused variable, and have
> verified that gomoku compiles cleanly afterwards. I have performed a
> search for this header file within the source tree, and it looks like
> gomoku is the only program that references this header file.
>
> I thought I should share this in case it should be removed to cleanup the
> code!
>
> Thanks and have a nice day,
> Peter
>
>
> Index: gomoku.h
> ===================================================================
> RCS file: /cvs/src/games/gomoku/gomoku.h,v
> retrieving revision 1.12
> diff -u -p -u -p -r1.12 gomoku.h
> --- gomoku.h 4 Jan 2016 17:33:24 -0000 1.12
> +++ gomoku.h 26 Dec 2017 22:05:11 -0000
> @@ -222,7 +222,6 @@ struct spotstr {
> u_char s_nforce[2]; /* number of <1,x> combos */
> struct elist *s_empty; /* level n combo completion spots */
> struct elist *s_nempty; /* level n+1 combo completion spots */
> - int dummy[2]; /* XXX */
> };
>
> /* flag values for s_flg */
