output/oss: AFMT_S24_PACKED is little-endian

According to the Solaris dsp manpage, AFMT_S24_PACKED is little-endian: http://download.oracle.com/docs/cd/E19963-01/821-1475/6nmf5baot/index.html The Minix soundcard.h header says the same.

output/oss: AFMT_S24_PACKED is little-endian
ee9c60fa · Max Kellermann · e2574848 · ee9c60fa · ee9c60fa
Commit ee9c60fa authored Feb 28, 2011 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 13 additions and 0 deletions

NEWS NEWS +1 -0

oss_plugin.c src/output/oss_plugin.c +12 -0

No files found.
--- a/NEWS
+++ b/NEWS
@@ -4,6 +4,7 @@ ver 0.16.2 (2011/??/??)
  - gme: detect end of song
 * output:
  - httpd: fix uninitialized variable
+  - oss: AFMT_S24_PACKED is little-endian


 ver 0.16.1 (2011/01/09)

--- a/src/output/oss_plugin.c
+++ b/src/output/oss_plugin.c
@@ -461,6 +461,12 @@ oss_setup_sample_format(int fd, struct audio_format *audio_format,
 			break;

 		audio_format->format = mpd_format;
+
+#ifdef AFMT_S24_PACKED
+		if (oss_format == AFMT_S24_PACKED)
+			audio_format->reverse_endian =
+				G_BYTE_ORDER != G_LITTLE_ENDIAN;
+#endif
 		return true;

 	case ERROR:
@@ -502,6 +508,12 @@ oss_setup_sample_format(int fd, struct audio_format *audio_format,
 				break;

 			audio_format->format = mpd_format;
+
+#ifdef AFMT_S24_PACKED
+			if (oss_format == AFMT_S24_PACKED)
+				audio_format->reverse_endian =
+					G_BYTE_ORDER != G_LITTLE_ENDIAN;
+#endif
 			return true;

 		case ERROR: