I found a minor annoyance. When building 3.93.1, the lame executable and the
ACM both claim to be 3.93. This was very confusing as I was dling many
different compiled binaries yet they all claimed to be version 3.93 and not
3.93.1, which is what they were.

I found that this is due to the calls in version.c. In get_lame_version,
there is only support for a version of format "x.x" and not "x.x.x"  As
there are seperate functions to get shorter version strings for embedding in
the output file, the get_lame_version.

Here are some suggested patches:

version.c

const char*  get_lame_version ( void )  /* primary to write screen reports
*/
{
    /* Here we can also add informations about compile time configurations
*/

#if   LAME_ALPHA_VERSION > 0
    static /[EMAIL PROTECTED]@*/ const char *const str =
        XSTR(LAME_MAJOR_VERSION) "." XSTR(LAME_MINOR_VERSION) " " V
        "(alpha " XSTR(LAME_ALPHA_VERSION) ", " __DATE__ " " __TIME__ ")";
#elif LAME_BETA_VERSION > 0
    static /[EMAIL PROTECTED]@*/ const char *const str =
        XSTR(LAME_MAJOR_VERSION) "." XSTR(LAME_MINOR_VERSION) " " V
        "(beta " XSTR(LAME_BETA_VERSION) ", " __DATE__ ")";
#else
    static /[EMAIL PROTECTED]@*/ const char *const str =
        XSTR(LAME_MAJOR_VERSION) "." XSTR(LAME_MINOR_VERSION) "." V
        XSTR(LAME_PATCH_VERSION);
#endif

    return str;
}

version.h:

# define LAME_MAJOR_VERSION      3      /* Major version number */
# define LAME_MINOR_VERSION     93      /* Minor version number */
# define LAME_PATCH_VERSION      1       /* Patch version number */

_______________________________________________
mp3encoder mailing list
[EMAIL PROTECTED]
http://minnie.tuhs.org/mailman/listinfo/mp3encoder

Reply via email to