maintainer-approval requested: [Bug 245156] audio/pulseaudio: parse /dev/sndstat correctly to get device name : [Attachment 212854] Proposed patch (since 516738 revision, v1)

[bugzilla-noreply at freebsd.org]

lightside <lightside at gmx.com> has asked freebsd-gnome mailing list
<gnome at FreeBSD.org> for maintainer-approval:
Bug 245156: audio/pulseaudio: parse /dev/sndstat correctly to get device name
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=245156

Attachment 212854: Proposed patch (since 516738 revision, v1)
https://bugs.freebsd.org/bugzilla/attachment.cgi?id=212854&action=edit



--- Comment #5 from lightside <lightside at gmx.com> ---
Created attachment 212854
  --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=212854&action=edit
Proposed patch (since 516738 revision, v1)

I created some testcase:
-8<--
#include <stdio.h>
#include <string.h>

int main() {
	const char *line = "pcm4: <Realtek (0x1168) (Rear Analog 5.1/2.0)>
(play/rec) default";
	int device;

	if (sscanf(line, "pcm%i: ", &device) != 1)
		return 0;
	{
		char *k = strchr(line, ':');
		k++;
		k += strspn(k, " <");
		char *e = strrchr(k, '>');

		if (!e)
			return 0;

		const size_t len = strlen(k) - strlen(e);
		char m[len];
		strncpy(m, k, len);

		printf("pcm%d: %s\n", device, m);
		// pcm4: Realtek (0x1168) (Rear Analog 5.1/2.0)
	}
	{
		char *k = strrchr(line, '>');

		if (!k)
			return 0;

		const size_t len = strlen(line) - strlen(k) + 1;
		char m[len];
		strncpy(m, line, len);

		printf("%s\n", m);
		// pcm4: <Realtek (0x1168) (Rear Analog 5.1/2.0)>
	}

	return 0;
}
-->8-

which outputs two variants for "pcm4: <Realtek (0x1168) (Rear Analog 5.1/2.0)>
(play/rec) default" line:
-8<--
pcm4: Realtek (0x1168) (Rear Analog 5.1/2.0)
pcm4: <Realtek (0x1168) (Rear Analog 5.1/2.0)>
-->8-

Attached patch for first variant.