shithub: opus

Download patch

ref: dd91338b0a7ec2a7675a9aaf22a9377c34a10674
parent: f0d6e319031ceee8300a7f3b37c316011b9cb973
author: Gregory Maxwell <[email protected]>
date: Thu Feb 26 12:05:37 EST 2009 

Add version info to celtenc/celtdec output as suggested on the
mailing list by Rob Til Freedmen. Also add a bitstream version
check to these applications.


--- a/tools/celtdec.c
+++ b/tools/celtdec.c
@@ -276,13 +276,13 @@
 
 void version(void)
 {
-   printf ("celtenc (CELT encoder)\n");
+   printf ("celtenc (CELT %s encoder)\n",CELT_VERSION);
    printf ("Copyright (C) 2008 Jean-Marc Valin\n");
 }
 
 void version_short(void)
 {
-   printf ("celtenc (CELT encoder)\n");
+   printf ("celtenc (CELT %s encoder)\n",CELT_VERSION);
    printf ("Copyright (C) 2008 Jean-Marc Valin\n");
 }
 
@@ -290,6 +290,7 @@
 {
    CELTDecoder *st;
    CELTHeader header;
+   int bitstream;
       
    celt_header_from_packet(op->packet, op->bytes, &header);
 
@@ -304,6 +305,12 @@
       fprintf (stderr, "Mode initialization failed.\n");
       return NULL;
    }
+
+   
+   celt_mode_info(*mode, CELT_GET_BITSTREAM_VERSION, &bitstream);
+   if (bitstream!=header.version_id)
+     fprintf(stderr, "WARNING: Input was encoded with a CELT bitstream version %d. This decoder uses %d. Output will probably be corrupted.\n",header.version_id,bitstream);
+   
    *channels = header.nb_channels;
    *overlap=header.overlap;
    st = celt_decoder_create(*mode);
--- a/tools/celtenc.c
+++ b/tools/celtenc.c
@@ -185,13 +185,13 @@
 
 void version(void)
 {
-   printf ("celtenc (CELT encoder)\n");
+   printf ("celtenc (CELT %s encoder)\n",CELT_VERSION);
    printf ("Copyright (C) 2008 Jean-Marc Valin\n");
 }
 
 void version_short(void)
 {
-   printf ("celtenc (CELT encoder)\n");
+   printf ("celtenc (CELT %s encoder)\n",CELT_VERSION);
    printf ("Copyright (C) 2008 Jean-Marc Valin\n");
 }
 
@@ -295,11 +295,9 @@
    celt_int32_t lookahead = 0;
    int bytes_per_packet=48;
    int complexity=-127;
-   
-   snprintf(vendor_string, sizeof(vendor_string), "Encoded with CELT\n");
-   
-   comment_init(&comments, &comments_length, vendor_string);
+   int bitstream;
 
+
    /*Process command-line options*/
    while(1)
    {
@@ -469,6 +467,12 @@
    mode = celt_mode_create(rate, chan, frame_size, NULL);
    if (!mode)
       return 1;
+
+  celt_mode_info(mode,CELT_GET_BITSTREAM_VERSION,&bitstream);      
+
+   snprintf(vendor_string, sizeof(vendor_string), "Encoded with CELT %s (bitstream: %d)\n",CELT_VERSION,bitstream);
+   comment_init(&comments, &comments_length, vendor_string);
+
    celt_mode_info(mode, CELT_GET_FRAME_SIZE, &frame_size);   
    
    celt_header_init(&header, mode);
@@ -479,8 +483,8 @@
       if (chan==2)
          st_string="stereo";
       if (!quiet)
-         fprintf (stderr, "Encoding %d Hz %s audio in %d sample packets at %0.3fkbit/sec (%d bytes per packet)\n", 
-               header.sample_rate, st_string, frame_size, bitrate, bytes_per_packet);
+         fprintf (stderr, "Encoding %d Hz %s audio in %d sample packets at %0.3fkbit/sec (%d bytes per packet) with bitstream version %d\n", 
+               header.sample_rate, st_string, frame_size, bitrate, bytes_per_packet,bitstream);
    }
 
    /*Initialize CELT encoder*/