shithub: libsamplerate

Download patch

ref: 19b5adc11368778fa29e7c10b1fb752decf1222c
parent: 5b237387baafb091b05d79c5202b26feb292ae22
author: Erik de Castro Lopo <[email protected]>
date: Sun Jan 10 04:55:09 EST 2010

Prevent creation of double length output files when conversion is restarted
due to clipping of the output when the format is FLAC. This problem arises
due to libsndfile's in ability to seek in a FLAC file during write.

--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2010-01-10  Erik de Castro Lopo  <erikd AT mega-nerd DOT com>
+
+    * examples/sndfile-resample.c
+    Prevent creation of double length output files when conversion is restarted
+    due to clipping of the output when the format is FLAC. This problem arises
+    due to libsndfile's in ability to seek in a FLAC file during write.
+
 2009-12-01  Erik de Castro Lopo  <erikd AT mega-nerd DOT com>
 
     * doc/win32.html
--- a/examples/sndfile-resample.c
+++ b/examples/sndfile-resample.c
@@ -1,5 +1,5 @@
 /*
-** Copyright (C) 2002-2009 Erik de Castro Lopo <[email protected]>
+** Copyright (C) 2002-2010 Erik de Castro Lopo <[email protected]>
 **
 ** 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
@@ -39,7 +39,7 @@
 
 int
 main (int argc, char *argv [])
-{	SNDFILE	*infile, *outfile ;
+{	SNDFILE	*infile, *outfile = NULL ;
 	SF_INFO sfinfo ;
 
 	sf_count_t	count ;
@@ -137,28 +137,31 @@
 	/* Delete the output file length to zero if already exists. */
 	remove (argv [argc - 1]) ;
 
-	if ((outfile = sf_open (argv [argc - 1], SFM_WRITE, &sfinfo)) == NULL)
-	{	printf ("Error : Not able to open output file '%s'\n", argv [argc - 1]) ;
-		sf_close (infile) ;
-		exit (1) ;
-		} ;
-
-	if (max_speed)
-	{	/* This is mainly for the comparison program tests/src-evaluate.c */
-		sf_command (outfile, SFC_SET_ADD_PEAK_CHUNK, NULL, SF_FALSE) ;
-		}
-	else
-	{	/* Update the file header after every write. */
-		sf_command (outfile, SFC_SET_UPDATE_HEADER_AUTO, NULL, SF_TRUE) ;
-		} ;
-
-	sf_command (outfile, SFC_SET_CLIPPING, NULL, SF_TRUE) ;
-
 	printf ("Output file   : %s\n", argv [argc - 1]) ;
 	printf ("Sample Rate   : %d\n", sfinfo.samplerate) ;
 
 	do
+	{	sf_close (outfile) ;
+
+		if ((outfile = sf_open (argv [argc - 1], SFM_WRITE, &sfinfo)) == NULL)
+		{	printf ("Error : Not able to open output file '%s'\n", argv [argc - 1]) ;
+			sf_close (infile) ;
+			exit (1) ;
+			} ;
+
+		if (max_speed)
+		{	/* This is mainly for the comparison program tests/src-evaluate.c */
+			sf_command (outfile, SFC_SET_ADD_PEAK_CHUNK, NULL, SF_FALSE) ;
+			}
+		else
+		{	/* Update the file header after every write. */
+			sf_command (outfile, SFC_SET_UPDATE_HEADER_AUTO, NULL, SF_TRUE) ;
+			} ;
+
+		sf_command (outfile, SFC_SET_CLIPPING, NULL, SF_TRUE) ;
+
 		count = sample_rate_convert (infile, outfile, converter, src_ratio, sfinfo.channels, &gain) ;
+		}
 	while (count < 0) ;
 
 	printf ("Output Frames : %ld\n\n", (long) count) ;
@@ -239,8 +242,6 @@
 	if (max > 1.0)
 	{	*gain = 1.0 / max ;
 		printf ("\nOutput has clipped. Restarting conversion to prevent clipping.\n\n") ;
-		output_count = 0 ;
-		sf_command (outfile, SFC_FILE_TRUNCATE, &output_count, sizeof (output_count)) ;
 		return -1 ;
 		} ;