Changeset 485281e in mainline for uspace/app/wavplay/main.c

Timestamp:

2018-12-18T18:38:49Z (5 years ago)

Author:

Jiri Svoboda <jiri@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

336b739

Parents:

c65590a

Message:

Allow specifying non-default audio target

Needed when there is more than one audio device. It would be nice
complementary functionality if we could actually configure the
default audio target.

File:

• uspace/app/wavplay/main.c (modified) (8 diffs)

uspace/app/wavplay/main.c

@@ -119 +119 @@
  * @return Error code
  */
-static errno_t hplay(const char *filename)
+static errno_t hplay(const char *filename, const char *target)
 {
         printf("Hound playback: %s\n", filename);
@@ -158 +158 @@
         }
 
-        ret = hound_context_connect_target(hound, HOUND_DEFAULT_TARGET);
+        ret = hound_context_connect_target(hound, target);
         if (ret != EOK) {
-                printf("Failed to connect to default target: %s\n",
+                printf("Failed to connect to target '%s': %s\n", target,
                     str_error(ret));
+
+                char **names = NULL;
+                size_t count = 0;
+                ret = hound_context_get_available_targets(hound, &names, &count);
+                if (ret == EOK) {
+                        printf("Available targets:\n");
+                        for (size_t i = 0; i < count; i++)
+                                printf(" - %s\n", names[i]);
+                }
+
                 hound_context_destroy(hound);
                 fclose(source);
@@ -215 +225 @@
         { "parallel", no_argument, 0, 'p' },
         { "record", no_argument, 0, 'r' },
+        { "target", required_argument, 0, 't' },
         { "help", no_argument, 0, 'h' },
         { 0, 0, 0, 0 }
@@ -230 +241 @@
         printf("\t -r, --record\t Start recording instead of playback. "
             "(Not implemented)\n");
-        printf("\t -d, --device\t Use specified device instead of the sound "
-            "service. Use location path or a special device `default'\n");
+        printf("\t -d, --device\t Direct output to specified device instead of "
+            "the sound service. Use location path or a special device `default'\n");
+        printf("\t -t, --target\t Output to the specified audio target.\n");
         printf("\t -p, --parallel\t Play given files in parallel instead of "
             "sequentially (does not work with -d).\n");
@@ -239 +251 @@
 {
         const char *device = "default";
+        const char *target = HOUND_DEFAULT_TARGET;
         int idx = 0;
         bool direct = false, record = false, parallel = false;
@@ -246 +259 @@
         /* Parse command line options */
         while (ret != -1) {
-                ret = getopt_long(argc, argv, "d:prh", opts, &idx);
+                ret = getopt_long(argc, argv, "d:prt:h", opts, &idx);
                 switch (ret) {
                 case 'd':
@@ -257 +270 @@
                 case 'p':
                         parallel = true;
+                        break;
+                case 't':
+                        target = optarg;
                         break;
                 case 'h':
@@ -334 +350 @@
                                 fibril_add_ready(fid);
                         } else {
-                                hplay(file);
+                                hplay(file, target);
                         }
                 }