GitLab

fix stuttering AAC SBR playback when PSRAM is available

It was allocated in one 16x chunk, so only freeing one chunk is enough

The ESP-IDF core code actually uses PSRAM also for standard "malloc" &
friends if it is available and the allocation id bigger than a certain
threshold (4kB by default). To make sure that the AAC decoder buffers,
especially the m_PSInfoSBR end up in RAM, use low level allocator calls
to specify where to alloc stuff. This also gets rid of the "try malloc
RAM first, if this fails fall back to ps_malloc" code, which probably
did not do what was wanted anyway.
This fixes stuttering SBR playback for me, which was caused by
m_PSInfoSBR ending up in PSRAM unexpectedly.

getBitRate: add option for average bitrate

add getCodec() and getCodecname() functions

getCodecname() allows for simple display of the used codec,
getCodec() together with the exported enum of codec values allows for
user defined codec names like this stupid example:
  switch (audio.getCodec()) {
    case Audio::CODEC_M4A:
    case Audio::CODEC_AAC:
      Serial.println("AAC codec!");
      break;
    case Audio::CODEC_FLAC:
    case AUDIO::CODEC_OGG_FLAC:
      Serial.println("FLAC!");
      break;
  }

Buffersize configurable

(no functional change)

instead of the fixed buffer size of 300000 for PSRAM and 8000 for
non-PSRAM setup, allow to configure the buffer sizes, example code
for that is
    audio.setBuffersize(32768, 0);
which will set the RAM buffer size to 32k and disable PSRAM usage.
I am investigating "bad WiFi throughput with PSRAM" issues and disabling
PSRAM is one workaround I found.

if InBuff is initialized should be a property of the AudioBuffer, the
calling code should not keep track of that.

Optimize audio info, avoid strdup where easily possible

this saves about 1k binary size

if no audio_info() function is available, still lots of strings are
sprintf()'ed into buffers, just to be not used at all.
Check for existence of audio_info() before doing that.
For many cases, this is achieved with a simple macro. Ugly but
effective.

simplify http / https handling

this got (accidentally?) lost in the previous unrelated commit
31e32ccc

instead of duplicating code for both WiFiClient and WiFiClientSecure,
use one pointer that gets assigned the corresponding class.

 [-Wformat-overflow=] and [-Wmisleading-indentation]