Add functions fwrite_le16() and fwrite_le32() master
authorAntonio Ospite <ospite@studenti.unina.it>
Thu, 6 Jan 2011 12:16:38 +0000 (13:16 +0100)
committerAntonio Ospite <ospite@studenti.unina.it>
Thu, 6 Jan 2011 12:23:17 +0000 (13:23 +0100)
These make the code more readable and fix the following error from some
compilers:
  error: lvalue required as unary '&' operand

Signed-off-by: Antonio Ospite <ospite@studenti.unina.it>
wav_header.c

index 6cb5157..2a90ea7 100644 (file)
@@ -79,6 +79,18 @@ struct extradata {
 };
 
 
 };
 
 
+static inline void fwrite_le16(uint16_t x, FILE *file)
+{
+       uint16_t tmp = htole16(x);
+       fwrite(&tmp, sizeof(tmp), 1, file);
+}
+
+static inline void fwrite_le32(uint32_t x, FILE *file)
+{
+       uint32_t tmp = htole32(x);
+       fwrite(&tmp, sizeof(tmp), 1, file);
+}
+
 void write_wav_header(FILE *file, struct fmt *format,
                struct extradata *extradata, unsigned data_len)
 {
 void write_wav_header(FILE *file, struct fmt *format,
                struct extradata *extradata, unsigned data_len)
 {
@@ -104,27 +116,27 @@ void write_wav_header(FILE *file, struct fmt *format,
 
        /* RIFF */
        fwrite(&r.riffheader, 1, 4, file);
 
        /* RIFF */
        fwrite(&r.riffheader, 1, 4, file);
-       fwrite(&htole32(r.WAVElen), 4, 1, file);
+       fwrite_le32(r.WAVElen, file);
 
        /* WAVE */
        fwrite(&w.fmtheader, 1, 8, file);
 
        /* WAVE */
        fwrite(&w.fmtheader, 1, 8, file);
-       fwrite(&htole32(w.fmtlen), 4, 1, file);
+       fwrite_le32(w.fmtlen, file);
 
        /* fmt */
 
        /* fmt */
-       fwrite(&htole16(format->wFormatTag), 2, 1, file);
-       fwrite(&htole16(format->nChannels), 2, 1, file);
-       fwrite(&htole32(format->nSamplesPerSec), 4, 1, file);
-       fwrite(&htole32(format->nAvgBytesPerSec), 4, 1, file);
-       fwrite(&htole16(format->nBlockAlign), 2, 1, file);
-       fwrite(&htole16(format->wBitsPerSample), 2, 1, file);
-       fwrite(&htole16(format->cbSize), 2, 1, file);
+       fwrite_le16(format->wFormatTag, file);
+       fwrite_le16(format->nChannels, file);
+       fwrite_le32(format->nSamplesPerSec, file);
+       fwrite_le32(format->nAvgBytesPerSec, file);
+       fwrite_le16(format->nBlockAlign, file);
+       fwrite_le16(format->wBitsPerSample, file);
+       fwrite_le16(format->cbSize, file);
 
        /* extradata */
        fwrite(extradata->data, 1, extradata->len, file);
 
        /* data */
        fwrite(&d.dataheader, 1, 4, file);
 
        /* extradata */
        fwrite(extradata->data, 1, extradata->len, file);
 
        /* data */
        fwrite(&d.dataheader, 1, 4, file);
-       fwrite(&htole32(d.datalen), 4, 1, file);
+       fwrite_le32(d.datalen, file);
 }
 
 int main(void)
 }
 
 int main(void)