sox-users@lists.sourceforge.net unofficial mirror
 help / color / mirror / code / Atom feed
From: "Måns Rullgård" <mans@mansr.com>
To: Jan Stary <hans@stare.cz>
Cc: sox-users@lists.sourceforge.net
Subject: Re: gsm
Date: Wed, 12 Aug 2020 20:18:05 +0100	[thread overview]
Message-ID: <yw1xsgcrsmfm.fsf@mansr.com> (raw)
In-Reply-To: <20200812164112.GA44586@www.stare.cz> (Jan Stary's message of "Wed, 12 Aug 2020 18:41:12 +0200")

Jan Stary <hans@stare.cz> writes:

> Here is a branch that drops the internal gsm
> and only supports gsm as an external format.
> https://sourceforge.net/u/janstary/sox/ci/gsm/tree/
>
> I also propose to drop the support for an external gsm altogether.
> The format is supported by libsndfile, which is actively maintained,
> as opposed to gsm implementations out there.

Libsndfile has its own copy of the same gsm library.  I don't think an
extra level of indirection will fix any bugs there, regardless of what
David Wheeler said.

> And I also propose to drop the internal lpc in the same way;
> in fact, to drop LPC support altogether. Is anyone using it?

Well, someone saw fit to add it.  That particular library seems to be
dead, and distro packages don't exist.  The decision isn't as simple as
for gsm.  Then again, the absence of packages suggest that nobody cares
about the format.  The Asterisk phone package includes a copy of the
same library, but that's all I can find.

Nonetheless, I don't want to rush ahead and remove features without due
warning.  It's not causing me any trouble, so I'm going to leave it for
now.

What I might do is disable some of the more obscure formats by default.
That way, any bugs in that code will only affect users who have actively
enabled them.

-- 
Måns Rullgård


_______________________________________________
Sox-users mailing list
Sox-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sox-users

      reply	other threads:[~2020-08-12 19:18 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-08-12 16:41 gsm Jan Stary
2020-08-12 19:18 ` Måns Rullgård [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://lists.sourceforge.net/lists/listinfo/sox-users

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=yw1xsgcrsmfm.fsf@mansr.com \
    --to=sox-users@lists.sourceforge.net \
    --cc=hans@stare.cz \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://80x24.org/mirrors/sox.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).