This is the mail archive of the
gcc@gcc.gnu.org
mailing list for the GCC project.
Re: gcc problem with new kde build
- From: Andrew Pinski <pinskia at physics dot uc dot edu>
- To: "Johnson, Tony -Research" <gjohnson at ADMWORLD dot com>
- Cc: Andrew Pinski <pinskia at physics dot uc dot edu>, "'gcc at gnu dot org'" <gcc at gnu dot org>
- Date: Wed, 3 Sep 2003 10:03:14 -0700
- Subject: Re: gcc problem with new kde build
There doesn't appear to be anything wrong with the c code at the line
in question (715)
void DeviceManager::seqbuf_clean(void)
{
#ifdef HAVE_ALSA_SUPPORT
if (alsa)
((AlsaOut *)device[default_dev])->seqbuf_clean();
else
#endif
#ifdef HAVE_OSS_SUPPORT
_seqbufptr=0;
#endif
} <---- line 715
Actually if HAVE_ALSA_SUPPORT is defined and HAVE_OSS_SUPPORT is not
(which looks like what is happening with you), you get:
void DeviceManager::seqbuf_clean(void)
{
if (alsa)
((AlsaOut *)device[default_dev])->seqbuf_clean();
else
}
so the code is messed up, report it to the KDE guys.
I think a better way of doing this is:
void DeviceManager::seqbuf_clean(void)
{
#ifdef HAVE_ALSA_SUPPORT
if (alsa)
{
((AlsaOut *)device[default_dev])->seqbuf_clean();
return;
}
#endif
#ifdef HAVE_OSS_SUPPORT
_seqbufptr=0;
#endif
} <---- line 715
Thanks,
Andrew Pinski