Complete.Org: Mailing Lists: Archives: freeciv-dev: September 2002:
[Freeciv-Dev] Re: fix for esd audio plugin
Home

[Freeciv-Dev] Re: fix for esd audio plugin

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: "Per I. Mathisen" <per@xxxxxxxxxxx>
Cc: freeciv-dev@xxxxxxxxxxx
Subject: [Freeciv-Dev] Re: fix for esd audio plugin
From: rf13@xxxxxxxxxxxxxxxxx
Date: Thu, 26 Sep 2002 06:19:30 -0700

On Wed, Sep 25, 2002 at 10:15:20AM +0000, Per I. Mathisen wrote:
> This fixes a complaint Christian came up with a while ago regarding the
> esd audio plugin: Occasionally, when used with units that are moved on
> rail or moved off-screen on goto, you get a machine gun like series of
> sounds being played, because they play too fast. This patch solves
> this.

> This isn't a problem with the sdl plugin, so I didn't solve this for the
> general case (ie in audio.c), but maybe I should do that if this is also a
> problem for the Windows sound plugin?

IMHO this should be audio.c. Even if there are no problems with the
current set of other plugins.

> +    (void) gettimeofday(&tv, &tz); /* error unthinkable */

unthinkable? IMHO this should be an assert.

> +    if (abs(tv.tv_usec - time_last_path) < 45000

This looks wrong. You should also compare tv_sec.

        Raimar

-- 
 email: rf13@xxxxxxxxxxxxxxxxx


[Prev in Thread] Current Thread [Next in Thread]