Only report failure to find OSS devices once, not every time we think

about them.

diff --git a/lib/mixer-oss.c b/lib/mixer-oss.c
index 7ce57be881b02ad1e3037a689f6c6fc72cf65610..e1c8b19547dfd7351a1351d79d2a1e21137b9a65 100644 (file)
--- a/lib/mixer-oss.c
+++ b/lib/mixer-oss.c
@@ -1,6 +1,6 @@
 /*
  * This file is part of DisOrder
- * Copyright (C) 2004, 2007 Richard Kettlewell
+ * Copyright (C) 2004, 2007, 2008 Richard Kettlewell
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -44,6 +44,7 @@
 #include "mixer.h"
 #include "log.h"
 #include "syscalls.h"
+#include "mem.h"
 
 /* documentation does not match implementation! */
 #ifndef SOUND_MIXER_READ
@@ -74,8 +75,16 @@ static int mixer_channel(const char *c) {
 static int oss_do_open(void) {
   int fd;
   
-  if((fd = open(config->mixer, O_RDWR, 0)) < 0)
-    error(errno, "error opening %s", config->mixer);
+  if((fd = open(config->mixer, O_RDWR, 0)) < 0) {
+    static char *reported;
+
+    if(!reported || strcmp(reported, config->mixer)) {
+      if(reported)
+       xfree(reported);
+      reported = xstrdup(config->mixer);
+      error(errno, "error opening %s", config->mixer);
+    }
+  }
   return fd;
 }
 

diff --git a/server/speaker-oss.c b/server/speaker-oss.c
index 69322ff73157da1f3e383518d5dd04d2660e0db9..36ef4a85c250b9b91b5acdc41cd663575004b15d 100644 (file)
--- a/server/speaker-oss.c
+++ b/server/speaker-oss.c
@@ -1,6 +1,6 @@
 /*
  * This file is part of DisOrder
- * Copyright (C) 2007 Richard Kettlewell
+ * Copyright (C) 2007, 2008 Richard Kettlewell
  * Portions copyright (C) 2007 Ross Younger
  *
  * This program is free software; you can redistribute it and/or modify
@@ -73,7 +73,12 @@ static void oss_activate(void) {
       else if(access("/dev/audio", W_OK) == 0)
        device = "/dev/audio";
       else {
-        error(0, "cannot determine default OSS device");
+        static int reported;
+        
+        if(!reported) {
+          error(0, "cannot determine default OSS device");
+          reported = 1;
+        }
         goto failed;
       }
     } else