[Bug 3776] New: OpenAL system reports errors inappropriately

[bugzilla-daemon at icculus.org]

http://bugzilla.icculus.org/show_bug.cgi?id=3776

           Summary: OpenAL system reports errors inappropriately
           Product: Quake 3
           Version: SVN HEAD
          Platform: PC
        OS/Version: Linux
            Status: NEW
          Severity: major
          Priority: P3
         Component: Sound
        AssignedTo: zakk at icculus.org
        ReportedBy: bugzilla at benmachine.co.uk
         QAContact: quake3-bugzilla at icculus.org


The OpenAL code frequently uses qalGetError() to test if a previously executed
function failed: however, if it succeeded, but a function called earlier
failed, then an error is still reported: at best this will result in an
erroneous message, but in some cases the code will fall back on defaults
unnecessarily.

Wrapping every function call that might give an error (but in real use probably
won't) is time-consuming and difficult, so instead I propose the following to
ensure the error state is cleared before execution of the function to be
tested. When called with qtrue, it simply eats the error; if called with qfalse
(as it is in every case included here) it prints a diagnostic message warning
that an unhandled error was found.

The two specific bugs which led me to discover this problem I will submit in
separate reports.


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