[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Adonthell-devel] Audio feedback
From: |
Chris Frey |
Subject: |
Re: [Adonthell-devel] Audio feedback |
Date: |
Wed, 27 Jun 2012 16:38:51 -0400 |
User-agent: |
Mutt/1.4.2.2i |
On Wed, Jun 27, 2012 at 08:41:45PM +0200, Josh Glover wrote:
> For those of you with a computer, you can checkout the
> gui-audio-feedback branch:
>
> https://github.com/ksterker/adonthell/tree/gui-audio-feedback
>
> Run worldtest to hear the excitement. You should hear the background
> music fade in as worldtest starts. Enter a conversation with the guy,
> and you should hear clicks as you select responses.
Nice! :-) Sound works great here.
On my build, I get a signed comparision warning, so I'd suggest this
change, since the source comparisons are pointers anyway, and probably want
to compare as unsigned every time.
diff --git a/src/gui/ui_event.h b/src/gui/ui_event.h
index 4339d4d..dd5434b 100644
--- a/src/gui/ui_event.h
+++ b/src/gui/ui_event.h
@@ -45,7 +45,7 @@ namespace gui
/**
* Source for listeners that want to listen to events of a given type f
*/
- static const int ANY_SOURCE = 0x1; // definitely not a valid pointer
+ static const unsigned int ANY_SOURCE = 0x1; // definitely not a valid p
/**
* @name Initialization
I'll let Kai do the official code review. :-)
- Chris
- [Adonthell-devel] Audio feedback, Josh Glover, 2012/06/27
- Re: [Adonthell-devel] Audio feedback,
Chris Frey <=
- Re: [Adonthell-devel] Audio feedback, Kai Sterker, 2012/06/29
- Re: [Adonthell-devel] Audio feedback, Chris Frey, 2012/06/29
- Re: [Adonthell-devel] Audio feedback, Kai Sterker, 2012/06/29
- Re: [Adonthell-devel] Audio feedback, Chris Frey, 2012/06/29
- Re: [Adonthell-devel] Audio feedback, Kai Sterker, 2012/06/30
- Re: [Adonthell-devel] Audio feedback, Josh Glover, 2012/06/30
- Re: [Adonthell-devel] Audio feedback, Kai Sterker, 2012/06/30
- Re: [Adonthell-devel] Audio feedback, Josh Glover, 2012/06/30
- Re: [Adonthell-devel] Audio feedback, Josh Glover, 2012/06/30