Bug 139593

Summary: FA_clalsadrv-1.0.1-6: 2 * buglets
Product: [openSUSE] SUSE LINUX 10.0 Reporter: David Binderman <dcb314>
Component: BasesystemAssignee: Marcus Rückert <mrueckert>
Status: RESOLVED FIXED QA Contact: E-mail List <qa-bugs>
Severity: Minor    
Priority: P5 - None    
Version: unspecified   
Target Milestone: ---   
Hardware: All   
OS: SUSE Other   
Whiteboard:
Found By: Other Services Priority:
Business Priority: Blocker: ---
Marketing QA Status: --- IT Deployment: ---

Description David Binderman 2005-12-16 14:42:27 UTC 
I just tried to compile package FA_clalsadrv-1.0.1-6 with the Intel C compiler 

It said

1.

clalsadrv.cc(606): warning #1011: missing return statement at end of non-void function "Alsa_driver::set_hwpar"

The source code is

    }
}

I have read the source code, and I agree with the compiler. Suggest add return statement.

2.

clalsadrv.cc(579): warning #592: variable "n" is used before its value is set

The source code is

fprintf (stderr, "Alsa_driver: can't set %s channel count to %u.\n", sname, n);

Suggest initialise local variable "n" before first use.

I tried to contact the author [ fons.adriaensen@skynet.be ] but didn't get an answer.
Comment 1 Marcus Rückert 2006-01-23 15:55:07 UTC 
seems the update to 1.0.3 fixed that warning.