[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] Fix build if OSS_GETVERSION is not defined
From: |
malc |
Subject: |
Re: [Qemu-devel] [PATCH] Fix build if OSS_GETVERSION is not defined |
Date: |
Sat, 9 Jan 2010 00:33:44 +0300 (MSK) |
On Fri, 8 Jan 2010, Juergen Lock wrote:
> On Fri, Jan 08, 2010 at 11:27:13AM +0300, malc wrote:
> > On Thu, 7 Jan 2010, Juergen Lock wrote:
> >
> > > In this case it was missing on FreeBSD <= 6.x (Which also doesn't have
> > > SNDCTL_DSP_POLICY yet so the version doesn't get used anyway.)
> >
> > I've commited slightly different fix for the issue, thanks.
> >
> Hmm looking at the last hunk of the commit,
>
> >[...]
> >@@ -289,9 +292,17 @@ static int oss_open (int in, struct oss_params *req,
> > if (conf.debug) {
> > dolog ("OSS version = %#x\n", version);
> > }
> >+#endif
> >
> > #ifdef SNDCTL_DSP_POLICY
> >- if (conf.policy >= 0 && version >= 0x040000) {
> >+ if (conf.policy >= 0
> >+#ifdef OSS_GETVERSION
> >+ && version >= 0x040000
> >+#else
> >+ 0
>
> ...these last two lines (#else and 0) probably should go, I dont think
> the compiler likes whitespace between digits. :)
Uh, yeah, my bad, sorry, hopefuly fixed now..
>
> >+#endif
> >+ )
> >+ {
> > int policy = conf.policy;
> > if (ioctl (fd, SNDCTL_DSP_POLICY, &policy)) {
> > oss_logerr2 (errno, typ, "Failed to set timing policy to %d\n",
> >--
> >1.6.6
>
> And also I forgot to say this is stable-0.12 material too.
You really should talk to the people who know what that means :)
> And thanx for committing!
> Juergen
>
--
mailto:address@hidden