Bulk update copyright dates
[flac.git] / src / share / grabbag / seektable.c
index 993bc32..01118d5 100644 (file)
@@ -1,34 +1,40 @@
 /* grabbag - Convenience lib for various routines common to several tools
- * Copyright (C) 2002  Josh Coalson
+ * Copyright (C) 2002-2009  Josh Coalson
+ * Copyright (C) 2011-2016  Xiph.Org Foundation
  *
- * 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 the Free Software Foundation; either version 2
- * of the License, or (at your option) any later version.
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2.1 of the License, or (at your option) any later version.
  *
- * This program is distributed in the hope that it will be useful,
+ * This library is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
  *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111-1307, USA.
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301  USA
  */
 
+#ifdef HAVE_CONFIG_H
+#  include <config.h>
+#endif
+
 #include "share/grabbag.h"
+#include "share/compat.h"
 #include "FLAC/assert.h"
 #include <stdlib.h> /* for atoi() */
 #include <string.h>
 
-GRABBAG_API FLAC__bool grabbag__seektable_convert_specification_to_template(const char *spec, FLAC__bool only_explicit_placeholders, FLAC__uint64 total_samples_to_encode, unsigned sample_rate, FLAC__StreamMetadata *seektable_template, FLAC__bool *spec_has_real_points)
+FLAC__bool grabbag__seektable_convert_specification_to_template(const char *spec, FLAC__bool only_explicit_placeholders, FLAC__uint64 total_samples_to_encode, unsigned sample_rate, FLAC__StreamMetadata *seektable_template, FLAC__bool *spec_has_real_points)
 {
        unsigned i;
        const char *pt;
 
        FLAC__ASSERT(0 != spec);
        FLAC__ASSERT(0 != seektable_template);
-       FLAC__ASSERT(seektable_template->type = FLAC__METADATA_TYPE_SEEKTABLE);
+       FLAC__ASSERT(seektable_template->type == FLAC__METADATA_TYPE_SEEKTABLE);
 
        if(0 != spec_has_real_points)
                *spec_has_real_points = false;
@@ -47,8 +53,10 @@ GRABBAG_API FLAC__bool grabbag__seektable_convert_specification_to_template(cons
                                        if(0 != spec_has_real_points)
                                                *spec_has_real_points = true;
                                        if(!only_explicit_placeholders) {
-                                               if(!FLAC__metadata_object_seektable_template_append_spaced_points(seektable_template, atoi(pt), total_samples_to_encode))
-                                                       return false;
+                                               const int n = (unsigned)atoi(pt);
+                                               if(n > 0)
+                                                       if(!FLAC__metadata_object_seektable_template_append_spaced_points(seektable_template, (unsigned)n, total_samples_to_encode))
+                                                               return false;
                                        }
                                }
                        }
@@ -58,11 +66,16 @@ GRABBAG_API FLAC__bool grabbag__seektable_convert_specification_to_template(cons
                                        if(0 != spec_has_real_points)
                                                *spec_has_real_points = true;
                                        if(!only_explicit_placeholders) {
-                                               double sec = atof(pt);
+                                               const double sec = atof(pt);
                                                if(sec > 0.0) {
-                                                       unsigned n = (unsigned)((double)total_samples_to_encode / (sec * (double)sample_rate));
-                                                       if(!FLAC__metadata_object_seektable_template_append_spaced_points(seektable_template, n, total_samples_to_encode))
-                                                               return false;
+                                                       unsigned samples = (unsigned)(sec * (double)sample_rate);
+                                                       /* Restrict seekpoints to two per second of audio. */
+                                                       samples = samples < sample_rate / 2 ? sample_rate / 2 : samples;
+                                                       if(samples > 0) {
+                                                               /* +1 for the initial point at sample 0 */
+                                                               if(!FLAC__metadata_object_seektable_template_append_spaced_points_by_samples(seektable_template, samples, total_samples_to_encode))
+                                                                       return false;
+                                                       }
                                                }
                                        }
                                }
@@ -71,9 +84,14 @@ GRABBAG_API FLAC__bool grabbag__seektable_convert_specification_to_template(cons
                                if(0 != spec_has_real_points)
                                        *spec_has_real_points = true;
                                if(!only_explicit_placeholders) {
-                                       FLAC__uint64 n = (unsigned)atoi(pt);
-                                       if(!FLAC__metadata_object_seektable_template_append_point(seektable_template, n))
-                                               return false;
+                                       char *endptr;
+                                       const FLAC__int64 n = (FLAC__int64)strtoll(pt, &endptr, 10);
+                                       if(
+                                               (n > 0 || (endptr > pt && *endptr == ';')) && /* is a valid number (extra check needed for "0") */
+                                               (total_samples_to_encode == 0 || (FLAC__uint64)n < total_samples_to_encode) /* number is not >= the known total_samples_to_encode */
+                                       )
+                                               if(!FLAC__metadata_object_seektable_template_append_point(seektable_template, (FLAC__uint64)n))
+                                                       return false;
                                }
                        }
                }