Jump to content

New Sound Manager svn patch

stwf
 Share

Recommended Posts

  • Replies 295
  • Created
  • Last Reply

Top Posters In This Topic

Popular Days

Top Posters In This Topic

Popular Days

Popular Posts

stwf
stwf

OK the new SoundManager code is in a patch attached to its trac ticket (I'll also attach it here) http://trac.wildfire...com/ticket/1766 This is a diff against the latest svn trunk, so patching should

k776
k776

Thanks stwf. I'm giving this a go now. For those with git repos of the project, here is stwf's work: https://github.com/stwf/0ad/commits/new-sound-manager

stwf
stwf

OK, the long awaited final version of the first pass of the sound manager is here. Should build and run cleanly on Mac, linux and Windows. I'll keep on the lookout for bug reports... fingers crossed..

Posted Images

historic_bruno Primus Pilus

historic_bruno

Posted
Posted

Thanks, I tried the new patch.

  • It's still missing a return statement in OggStreamImpl::ResetFile which is a build error in MSVC
  • I saw two errors on startup:
      Quote

    ERROR: OpenAL error: (null); called from CSoundManager::AlcInit (line 122)
    ERROR: Sound: AlcInit success, using OpenAL Soft
    The second error should be left as a debug_printf or similar since it's not an error and need not be displayed. The first error seems a result of calling AL_CHECK before OpenAL is initialized (not a real error).
  • Then I tried compiling --without-audio. lib/external_libraries/openal.h is included outside CONFIG2_AUDIO in several places: OggData.h, SoundData.h, CSoundBase.h, and ISoundItem.h, which cause build errors since the OpenAL library has been excluded from the build. Probably it can be cleaned up a bit by moving all includes except config2.h inside the CONFIG2_AUDIO check.
  • After fixing those errors, audio was indeed disabled and I had no runtime errors (y)
  • Minor issue: warning about unused variable doEnable in CSoundManager::SetEnabled - this might be fixed by handled the !CONFIG2_AUDIO case and inserting something like UNUSED2(doEnabled); to make the warning go away.

Link to comment
Share on other sites
historic_bruno Primus Pilus

historic_bruno

Posted
Posted
  On 30/08/2012 at 2:57 AM, stwf said:

OK, thanks I don't get any of those errors on the mac... and its hard to see warnings, but this new version was built in linux too and seems to work fine.

Thanks for the testing...

Works like a charm, tested with and without audio build, with and without archived data. I say commit it when you get the chance (y)

Link to comment
Share on other sites
zoot Primus Pilus

zoot

Posted
Posted

A recent commit has broken my build:

In file included from ../../../source/soundmanager/data/OggData.cpp:20:0:
../../../source/soundmanager/data/OggData.h:27:17: fatal error: ogg.h: No such file or directory
compilation terminated.
make[1]: *** [obj/engine_Release/OggData.o] Error 1
make: *** [engine] Error 2

Link to comment
Share on other sites
historic_bruno Primus Pilus

historic_bruno

Posted
Posted
  On 31/08/2012 at 7:22 PM, zoot said:

A recent commit has broken my build:

In file included from ../../../source/soundmanager/data/OggData.cpp:20:0:
../../../source/soundmanager/data/OggData.h:27:17: fatal error: ogg.h: No such file or directory
compilation terminated.
make[1]: *** [obj/engine_Release/OggData.o] Error 1
make: *** [engine] Error 2

They were left out by mistake, it should be fixed now :wallbash:

Link to comment
Share on other sites
zoot Primus Pilus

zoot

Posted
Posted

The build is fixed, but I got this after exiting a game (session):

ERROR: OpenAL error: Invalid Value; called from ~CSoundItem (line 48)

ERROR: OpenAL error: Invalid Value; called from ~CSoundItem (line 48)

ERROR: OpenAL error: Invalid Value; called from ~CSoundItem (line 48)

ERROR: OpenAL error: Invalid Value; called from ~CSoundItem (line 48)

ERROR: OpenAL error: Invalid Value; called from ~CSoundItem (line 48)

ERROR: OpenAL error: Invalid Value; called from ~CSoundItem (line 48)

ERROR: OpenAL error: Invalid Value; called from ~CSoundItem (line 48)

ERROR: OpenAL error: Invalid Value; called from ~CSoundItem (line 48)

ERROR: OpenAL error: Invalid Value; called from ~CSoundItem (line 48)

ERROR: OpenAL error: Invalid Value; called from ~CSoundItem (line 48)

Link to comment
Share on other sites
stwf Sesquiplicarius

stwf

Posted
  • Author
Posted
  On 31/08/2012 at 9:47 PM, zoot said:

The build is fixed, but I got this after exiting a game (session):

ERROR: OpenAL error: Invalid Value; called from ~CSoundItem (line 48)

ERROR: OpenAL error: Invalid Value; called from ~CSoundItem (line 48)

ERROR: OpenAL error: Invalid Value; called from ~CSoundItem (line 48)

ERROR: OpenAL error: Invalid Value; called from ~CSoundItem (line 48)

ERROR: OpenAL error: Invalid Value; called from ~CSoundItem (line 48)

ERROR: OpenAL error: Invalid Value; called from ~CSoundItem (line 48)

ERROR: OpenAL error: Invalid Value; called from ~CSoundItem (line 48)

ERROR: OpenAL error: Invalid Value; called from ~CSoundItem (line 48)

ERROR: OpenAL error: Invalid Value; called from ~CSoundItem (line 48)

ERROR: OpenAL error: Invalid Value; called from ~CSoundItem (line 48)

Platform info? I don't see this on the mac...

Link to comment
Share on other sites
myconid Centurio

myconid

Posted
Posted

The game segfaults when starting up, Atlas segfaults when clicking the "Play" button. There was no issue a few revisions back, and it still works fine with the -nosound flag.

gdb backtrace:

  Reveal hidden contents

Link to comment
Share on other sites
historic_bruno Primus Pilus

historic_bruno

Posted
Posted
  On 01/09/2012 at 12:34 PM, myconid said:

The game segfaults when starting up, Atlas segfaults when clicking the "Play" button. There was no issue a few revisions back, and it still works fine with the -nosound flag.

gdb backtrace:

  Reveal hidden contents

Can you also provide the terminal output from the game's startup?

Link to comment
Share on other sites
myconid Centurio

myconid

Posted
Posted
  On 01/09/2012 at 4:19 PM, historic_bruno said:

Can you also provide the terminal output from the game's startup?

I can, but it's not helpful at all:


Cache: 500 (total: 3929) MiB
TIMER| InitVfs: 511.094 us
AL lib: pulseaudio.c:331: PulseAudio returned minreq > tlength/2; expect break up
Sound: AlcInit success, using PulseAudio Default
TIMER| InitScripting: 2.42154 ms
TIMER| CONFIG_Init: 3.67938 ms
TIMER| RunHardwareDetection: 18.8531 ms
TIMER| write_sys_info: 17.6893 ms
TIMER| InitRenderer: 2.01848 ms
TIMER| ps_console: 1.81371 ms
TIMER| ps_lang_hotkeys: 802.757 us
TIMER| common/setup.xml: 1.08079 ms
TIMER| common/styles.xml: 65.371 us
TIMER| common/sprite1.xml: 2.10942 ms
TIMER| common/init.xml: 1.47044 ms
TIMER| common/common_sprites.xml: 2.47259 ms
TIMER| common/common_styles.xml: 340.896 us
TIMER| pregame/sprites.xml: 723.277 us
TIMER| pregame/styles.xml: 62.787 us
TIMER| pregame/mainmenu.xml: 7.70251 ms
TIMER| common/global.xml: 447.194 us
Segmentation fault

(I've always gotten that warning about break up. It works just fine regardless.)

Link to comment
Share on other sites
historic_bruno Primus Pilus

historic_bruno

Posted
Posted

While looking at the strange use of CStrW in CSoundData, I noticed that apparently m_FileName is never assigned to, yet GetFileName() is used in ReleaseSoundData(), so it seems that's an oversight and possibly a memory leak if it never cleans up the sound data.

  On 01/09/2012 at 12:34 PM, myconid said:

The game segfaults when starting up, Atlas segfaults when clicking the "Play" button. There was no issue a few revisions back, and it still works fine with the -nosound flag.

gdb backtrace:

  Reveal hidden contents

Obviously a silent failure somewhere, which is annoying :( We don't check the return value of OpenOggNonstream() so that's bad, it could fail for one of many reasons.

Link to comment
Share on other sites
fabio Primus Pilus

fabio

Posted
Posted
  On 01/09/2012 at 7:42 PM, historic_bruno said:

Some of the in-game sounds are quite loud already, maybe music needs to be boosted slightly, or which sounds are you thinking are too quiet?

Starting music of main menu. It also seems it's missing some sounds from alpha 10, e.g. the ride cymbal entry sound from which the music start. Maybe it just was removed in the music file (it sounded better before to me).

Link to comment
Share on other sites
stwf Sesquiplicarius

stwf

Posted
  • Author
Posted

Hello All,

OK, I added some more error handling to the ogg file inits, and hopefully it will both report and handle errors better... Sorry about the mFilename thing. Originally The sounds were reference counted and created, freed, and reinitialized dynamically. But it became apparent that the javascript engine isn't always good about reporting when things are deleted or go out of scope, also most sounds are rused again and again, so they should really be cached anyway.

I will go through and make sure all remnants of that system are cleaned up.

Link to comment
Share on other sites
Yves Primus Pilus

Yves

Posted
Posted

I just wanted to mention here that some people are saying that the music quality isn't as good as in alpha 10 anymore.

Maybe it's the same that fabio mentioned about "missing some sounds".

I compared with alpha 10 and can't hear a difference, but my speakers aren't very good ones.

Check the discussion about that topic in the irc-logs(starts about 21:37)

http://irclogs.wildfiregames.com/2012-08-31-QuakeNet-%230ad-dev.log

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
 Share
Go to topic listing
×
×
  • Create New...