[quake3-bugzilla] [Bug 4930] New: vm_powerpc_asm needs to include the header for ARRAY_LEN

bugzilla-daemon at icculus.org bugzilla-daemon at icculus.org
Sun Mar 13 08:49:19 EDT 2011


https://bugzilla.icculus.org/show_bug.cgi?id=4930

           Summary: vm_powerpc_asm needs to include the header for
                    ARRAY_LEN
           Product: ioquake3
           Version: SVN HEAD
          Platform: Macintosh
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: P3
         Component: Platform
        AssignedTo: zakk at icculus.org
        ReportedBy: smcv-ioquake3 at pseudorandom.co.uk
         QAContact: quake3-bugzilla at icculus.org


Created attachment 2638
  --> https://bugzilla.icculus.org/attachment.cgi?id=2638
include vm_local.h in vm_powerpc_asm.c

I uploaded a recent snapshot (r1921 with Debian patches) to Debian
experimental, and it failed on the powerpc autobuilder:

code/qcommon/vm_powerpc_asm.c:392: warning: implicit declaration of function
'ARRAY_LEN'
code/qcommon/vm_powerpc_asm.c:392: error: initializer element is not constant

(<https://buildd.debian.org/fetch.cgi?pkg=ioquake3&arch=powerpc&ver=1.36%2Bsvn1921-1&stamp=1299807225&file=log>)

I think this just needs a header added. Patch description:

    include vm_local.h in vm_powerpc_asm.c

    This indirectly includes q_shared.h, which is needed for ARRAY_LEN;
    I didn't use that header directly to be consistent with the other VM
    files.

-- 
Configure bugmail: https://bugzilla.icculus.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.


More information about the quake3-bugzilla mailing list