Bug 117098

Summary: three buglets
Product: [Fedora] Fedora Reporter: d.binderman
Component: timidity++Assignee: Thomas Woerner <twoerner>
Status: CLOSED NEXTRELEASE QA Contact:
Severity: medium Docs Contact:
Priority: medium    
Version: 2Keywords: EasyFix
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2004-06-19 00:39:59 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description d.binderman 2004-02-28 14:28:02 UTC
Description of problem:

I just tried to compile Core2 Test1 package timidity++-2.11.3-8

The compiler said

1.

mloader.c(635): warning #140: too many arguments in function call

The source code is

          FreeSampleList (musiclist);

but, earlier in the same file

static void
FreeSampleList ()
{

Suggest change definition of FreeSampleList to

static void
FreeSampleList ( void)
{

and delete musiclist from call.

2.

mloader.c(642): warning #140: too many arguments in function call

Duplicate.

3.

playmidi.c(4667): remark #592: variable "rc" is used before its value
is set

The source code is

        return rc;

Suggest init rc before first use.


Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:

Comment 1 Alan Cox 2004-06-19 00:38:21 UTC
Committed to CVS