Attention is currently required from: laforge, pespin, dexter.

View Change

1 comment:

• Patchset:

  • Patch Set #1:

    Did you look at git history? which commit was that code introduced in and by who? Any other possible […]

    Yes, and unfortunately all the history can tell us is that AMR support (channel coding, interleaving, scheduling, and the link adaptation loop) was added in a big commit with no commit message back in 2013:

      commit a7f5e077121f857e373b6e68c5fdf803933a12dc
      Author: Andreas Eversberg <jolly@eversberg.eu>
      Date:   Tue Mar 26 09:05:14 2013 +0100

          TRX: Support for AMR full speech

    I think it makes no sense to ask jolly what was the motivation behind the loop enable/disable switch. It's unlikely that he remembers the details.

To view, visit change 27777. To unsubscribe, or for help writing mail filters, visit settings.