unfinished work
[vg.git] / vg_audio.h
index 279b7f4ea57316c6c9680cf5fdf9be631ee0c8e0..8f68f82c3e4f193379023284baa24983e81b0c0e 100644 (file)
@@ -48,6 +48,7 @@
 #define AUDIO_LFOS            8
 #define AUDIO_FILTERS         16
 #define AUDIO_FLAG_LOOP       0x1
+#define AUDIO_FLAG_NO_DOPPLER 0x2
 #define AUDIO_FLAG_SPACIAL_3D 0x4
 #define AUDIO_FLAG_AUTO_START 0x8
 
@@ -88,17 +89,14 @@ typedef struct audio_clip audio_clip;
 typedef struct audio_channel audio_channel;
 typedef struct audio_lfo audio_lfo;
 
-struct audio_clip
-{
+struct audio_clip{
    const char *path;
    u32 flags;
-
    u32 size;
    void *data;
 };
 
-static struct vg_audio_system
-{
+static struct vg_audio_system{
    SDL_AudioDeviceID sdl_output_device;
 
    void             *audio_pool, 
@@ -111,15 +109,12 @@ static struct vg_audio_system
    SDL_mutex        *mux_checker,
                     *mux_sync;
 
-   struct audio_lfo
-   {
+   struct audio_lfo{
       u32 time, time_startframe;
       float sqrt_polynomial_coefficient;
 
-      struct
-      {
-         enum lfo_wave_type
-         {
+      struct{
+         enum lfo_wave_type{
             k_lfo_triangle,
             k_lfo_square,
             k_lfo_saw,
@@ -135,9 +130,10 @@ static struct vg_audio_system
    }
    oscillators[ AUDIO_LFOS ];
 
-   struct audio_channel
-   {
+   struct audio_channel{
       int allocated;
+      u32 group;
+
       char name[32];       /* only editable while allocated == 0 */
       audio_clip *source;  /* ... */
       u32 flags;           /* ... */
@@ -153,16 +149,14 @@ static struct vg_audio_system
       u32 volume_movement,
           pan_movement;
 
-      union
-      {
+      union{
          struct synth_bird *bird_handle;
          stb_vorbis *vorbis_handle;
       };
 
       stb_vorbis_alloc vorbis_alloc;
 
-      enum channel_activity
-      {
+      enum channel_activity{
          k_channel_activity_reset,   /* will advance if allocated==1, to wake */
          k_channel_activity_wake,    /* will advance to either of next two */
          k_channel_activity_alive,
@@ -177,8 +171,7 @@ static struct vg_audio_system
        * the edit mask tells which to copy into internal _, or to discard
        * ----------------------------------------------------------------------
        */
-      struct channel_state
-      {
+      struct channel_state{
          int   relinquished;
 
          float volume,          /* current volume */
@@ -318,12 +311,10 @@ VG_STATIC void vg_audio_init(void)
    vg_audio.sdl_output_device = 
       SDL_OpenAudioDevice( NULL, 0, &spec_desired, &spec_got,0 );
 
-   if( vg_audio.sdl_output_device )
-   {
+   if( vg_audio.sdl_output_device ){
       SDL_PauseAudioDevice( vg_audio.sdl_output_device, 0 );
    }
-   else
-   {
+   else{
       vg_fatal_exit_loop( 
          "SDL_OpenAudioDevice failed. Your default audio device must support:\n"
          "  Frequency: 44100 hz\n"
@@ -354,37 +345,40 @@ VG_STATIC void vg_audio_free(void)
 #define AUDIO_EDIT_OWNERSHIP      0x40
 #define AUDIO_EDIT_SAMPLING_RATE  0x80
 
-static audio_channel *audio_request_channel( audio_clip *clip, u32 flags )
+static void audio_channel_init( audio_channel *ch, audio_clip *clip, u32 flags )
 {
-   for( int i=0; i<AUDIO_CHANNELS; i++ )
-   {
-      audio_channel *ch = &vg_audio.channels[i];
+   ch->group = 0;
+   ch->source = clip;
+   ch->flags = flags;
+   ch->colour = 0x00333333;
 
-      if( !ch->allocated )
-      {
-         ch->source = clip;
-         ch->flags = flags;
-         ch->colour = 0x00333333;
+   if( (ch->source->flags & AUDIO_FLAG_FORMAT) == k_audio_format_bird )
+      strcpy( ch->name, "[array]" );
+   else
+      strncpy( ch->name, clip->path, 31 );
+
+   ch->allocated = 1;
+
+   ch->editable_state.relinquished = 0;
+   ch->editable_state.volume = 1.0f;
+   ch->editable_state.volume_target = 1.0f;
+   ch->editable_state.pan = 0.0f;
+   ch->editable_state.pan_target = 0.0f;
+   ch->editable_state.volume_rate = 0;
+   ch->editable_state.pan_rate = 0;
+   v4_copy((v4f){0.0f,0.0f,0.0f,1.0f},ch->editable_state.spacial_falloff);
+   ch->editable_state.lfo = NULL;
+   ch->editable_state.lfo_amount = 0.0f;
+   ch->editable_state.sampling_rate = 1.0f;
+   ch->editble_state_write_mask = 0x00;
+}
 
-         if( (ch->source->flags & AUDIO_FLAG_FORMAT) == k_audio_format_bird )
-            strcpy( ch->name, "[array]" );
-         else
-            strncpy( ch->name, clip->path, 31 );
-
-         ch->allocated = 1;
-
-         ch->editable_state.relinquished = 0;
-         ch->editable_state.volume = 1.0f;
-         ch->editable_state.volume_target = 1.0f;
-         ch->editable_state.pan = 0.0f;
-         ch->editable_state.pan_target = 0.0f;
-         ch->editable_state.volume_rate = 0;
-         ch->editable_state.pan_rate = 0;
-         v4_copy((v4f){0.0f,0.0f,0.0f,1.0f},ch->editable_state.spacial_falloff);
-         ch->editable_state.lfo = NULL;
-         ch->editable_state.lfo_amount = 0.0f;
-         ch->editable_state.sampling_rate = 1.0f;
-         ch->editble_state_write_mask = 0x00;
+static audio_channel *audio_get_first_idle_channel(void)
+{
+   for( int i=0; i<AUDIO_CHANNELS; i++ ){
+      audio_channel *ch = &vg_audio.channels[i];
+
+      if( !ch->allocated ){
          return ch;
       }
    }
@@ -392,6 +386,42 @@ static audio_channel *audio_request_channel( audio_clip *clip, u32 flags )
    return NULL;
 }
 
+static audio_channel *audio_get_group_idle_channel( u32 group, u32 max_count )
+{
+   u32 count = 0;
+   audio_channel *dest;
+
+   for( int i=0; i<AUDIO_CHANNELS; i++ ){
+      audio_channel *ch = &vg_audio.channels[i];
+
+      if( ch->allocated ){
+         if( ch->group == group ){
+            count ++;
+         }
+      }
+      else{
+         if( !dest )
+            dest = ch;
+      }
+   }
+
+   if( dest && (count < max_count) ){
+      return dest;
+   }
+
+   return NULL;
+}
+
+static audio_channel *audio_get_group_first_active_channel( u32 group )
+{
+   for( int i=0; i<AUDIO_CHANNELS; i++ ){
+      audio_channel *ch = &vg_audio.channels[i];
+      if( ch->allocated && (ch->group == group) )
+         return ch;
+   }
+   return NULL;
+}
+
 static int audio_channel_finished( audio_channel *ch )
 {
    if( ch->readable_activity == k_channel_activity_end )
@@ -424,13 +454,11 @@ static void audio_channel_set_sampling_rate( audio_channel *ch, float rate )
 static void audio_channel_edit_volume( audio_channel *ch,
                                        float new_volume, int instant )
 {
-   if( instant )
-   {
+   if( instant ){
       ch->editable_state.volume = new_volume;
       ch->editble_state_write_mask |= AUDIO_EDIT_VOLUME;
    }
-   else
-   {
+   else{
       audio_channel_slope_volume( ch, 0.05f, new_volume );
    }
 }
@@ -456,10 +484,12 @@ static audio_channel *audio_channel_crossfade( audio_channel *ch,
    if( ch )
       ch = audio_channel_fadeout( ch, length );
 
-   audio_channel *replacement = audio_request_channel( new_clip, flags );
+   audio_channel *replacement = audio_get_first_idle_channel();
 
-   if( replacement )
+   if( replacement ){
+      audio_channel_init( replacement, new_clip, flags );
       audio_channel_fadein( replacement, length );
+   }
 
    return replacement;
 }
@@ -474,8 +504,7 @@ static void audio_channel_sidechain_lfo( audio_channel *ch, int lfo_id,
 
 static void audio_channel_set_spacial( audio_channel *ch, v3f co, float range )
 {
-   if( ch->flags & AUDIO_FLAG_SPACIAL_3D )
-   {
+   if( ch->flags & AUDIO_FLAG_SPACIAL_3D ){
       v3_copy( co, ch->editable_state.spacial_falloff );
 
       if( range == 0.0f )
@@ -485,8 +514,7 @@ static void audio_channel_set_spacial( audio_channel *ch, v3f co, float range )
 
       ch->editble_state_write_mask |= AUDIO_EDIT_SPACIAL;
    }
-   else
-   {
+   else{
       vg_warn( "Tried to set spacialization paramaters for 2D channel (%s)\n",
                ch->name );
    }
@@ -495,10 +523,10 @@ static void audio_channel_set_spacial( audio_channel *ch, v3f co, float range )
 static int audio_oneshot_3d( audio_clip *clip, v3f position, 
                              float range, float volume )
 {
-   audio_channel *ch = audio_request_channel( clip, AUDIO_FLAG_SPACIAL_3D );
+   audio_channel *ch = audio_get_first_idle_channel();
 
-   if( ch )
-   {
+   if( ch ){
+      audio_channel_init( ch, clip, AUDIO_FLAG_SPACIAL_3D );
       audio_channel_set_spacial( ch, position, range );
       audio_channel_edit_volume( ch, volume, 1 );
       ch = audio_relinquish_channel( ch );
@@ -511,10 +539,10 @@ static int audio_oneshot_3d( audio_clip *clip, v3f position,
 
 static int audio_oneshot( audio_clip *clip, float volume, float pan )
 {
-   audio_channel *ch = audio_request_channel( clip, 0x00 );
+   audio_channel *ch = audio_get_first_idle_channel();
 
-   if( ch )
-   {
+   if( ch ){
+      audio_channel_init( ch, clip, 0x00 );
       audio_channel_edit_volume( ch, volume, 1 );
       ch = audio_relinquish_channel( ch );
 
@@ -542,7 +570,6 @@ static void audio_set_lfo_frequency( int id, float freq )
 }
 
 
-
 /* 
  * Committers
  * -----------------------------------------------------------------------------
@@ -551,8 +578,7 @@ static int audio_channel_load_source( audio_channel *ch )
 {
    u32 format = ch->source->flags & AUDIO_FLAG_FORMAT;
 
-   if( format == k_audio_format_vorbis )
-   {
+   if( format == k_audio_format_vorbis ){
       /* Setup vorbis decoder */
       u32 index = ch - vg_audio.channels;
 
@@ -569,20 +595,17 @@ static int audio_channel_load_source( audio_channel *ch )
             ch->source->data,
             ch->source->size, &err, &alloc );
 
-      if( !decoder )
-      {
+      if( !decoder ){
          vg_error( "stb_vorbis_open_memory failed on '%s' (%d)\n", 
                      ch->source->path, err );
          return 0;
       }
-      else
-      {
+      else{
          ch->source_length = stb_vorbis_stream_length_in_samples( decoder );
          ch->vorbis_handle = decoder;
       }
    }
-   else if( format == k_audio_format_bird )
-   {
+   else if( format == k_audio_format_bird ){
       u32 index = ch - vg_audio.channels;
 
       u8 *buf = (u8*)vg_audio.decode_buffer;
@@ -594,12 +617,10 @@ static int audio_channel_load_source( audio_channel *ch )
       ch->bird_handle = loc;
       ch->source_length = synth_bird_get_length_in_samples( loc );
    }
-   else if( format == k_audio_format_stereo )
-   {
+   else if( format == k_audio_format_stereo ){
       ch->source_length = ch->source->size / 2;
    }
-   else
-   {
+   else{
       ch->source_length = ch->source->size;
    }
 
@@ -608,8 +629,7 @@ static int audio_channel_load_source( audio_channel *ch )
 
 VG_STATIC void audio_decode_uncompressed_mono( i16 *src, u32 count, float *dst )
 {
-   for( u32 i=0; i<count; i++ )
-   {
+   for( u32 i=0; i<count; i++ ){
       dst[ i*2 + 0 ] = ((float)src[i]) * (1.0f/32767.0f);
       dst[ i*2 + 1 ] = ((float)src[i]) * (1.0f/32767.0f);
    }
@@ -625,15 +645,13 @@ stb_vorbis_get_samples_float_interleaved_stereo( stb_vorbis *f, float *buffer,
    int n = 0,
        c = VG_MIN( 1, f->channels - 1 );
 
-   while( n < len ) 
-   {
+   while( n < len ) {
       int k = f->channel_buffer_end - f->channel_buffer_start;
 
       if( n+k >= len ) 
          k = len - n;
 
-      for( int j=0; j < k; ++j ) 
-      {
+      for( int j=0; j < k; ++j ) {
          *buffer++ = f->channel_buffers[ 0 ][f->channel_buffer_start+j];
          *buffer++ = f->channel_buffers[ c ][f->channel_buffer_start+j];
       }
@@ -660,15 +678,13 @@ stb_vorbis_get_samples_i16_downmixed( stb_vorbis *f, i16 *buffer, int len )
    int n = 0,
        c = VG_MIN( 1, f->channels - 1 );
 
-   while( n < len ) 
-   {
+   while( n < len ) {
       int k = f->channel_buffer_end - f->channel_buffer_start;
 
       if( n+k >= len ) 
          k = len - n;
 
-      for( int j=0; j < k; ++j ) 
-      {
+      for( int j=0; j < k; ++j ) {
          float sl = f->channel_buffers[ 0 ][f->channel_buffer_start+j],
                sr = f->channel_buffers[ c ][f->channel_buffer_start+j];
 
@@ -699,8 +715,7 @@ static inline float audio_lfo_pull_sample( audio_lfo *lfo )
    float t  = lfo->time;
          t /= (float)lfo->_.period;
 
-   if( lfo->_.wave_type == k_lfo_polynomial_bipolar )
-   {
+   if( lfo->_.wave_type == k_lfo_polynomial_bipolar ){
       /*
        *           #
        *          # #
@@ -722,8 +737,7 @@ static inline float audio_lfo_pull_sample( audio_lfo *lfo )
               
              ) * (1.0f-fabsf(t));
    }
-   else
-   {
+   else{
       return 0.0f;
    }
 }
@@ -738,45 +752,37 @@ static void audio_channel_get_samples( audio_channel *ch,
 
    u32 format = ch->source->flags & AUDIO_FLAG_FORMAT;
 
-   while( remaining )
-   {
+   while( remaining ){
       u32 samples_this_run = VG_MIN(remaining, ch->source_length - ch->cursor);
       remaining -= samples_this_run;
 
       float *dst = &buf[ buffer_pos * 2 ]; 
       
-      if( format == k_audio_format_stereo )
-      {
-         for( int i=0;i<samples_this_run; i++ )
-         {
+      if( format == k_audio_format_stereo ){
+         for( int i=0;i<samples_this_run; i++ ){
             dst[i*2+0] = 0.0f;
             dst[i*2+1] = 0.0f;
          }
       }
-      else if( format == k_audio_format_vorbis )
-      {
+      else if( format == k_audio_format_vorbis ){
          int read_samples = stb_vorbis_get_samples_float_interleaved_stereo( 
                ch->vorbis_handle,
                dst,
                samples_this_run );
 
-         if( read_samples != samples_this_run )
-         {
+         if( read_samples != samples_this_run ){
             vg_warn( "Invalid samples read (%s)\n", ch->source->path );
 
-            for( int i=0; i<samples_this_run; i++ )
-            {
+            for( int i=0; i<samples_this_run; i++ ){
                dst[i*2+0] = 0.0f;
                dst[i*2+1] = 0.0f;
             }
          }
       }
-      else if( format == k_audio_format_bird )
-      {
+      else if( format == k_audio_format_bird ){
          synth_bird_generate_samples( ch->bird_handle, dst, samples_this_run );
       }
-      else
-      {
+      else{
          i16 *src_buffer = ch->source->data,
              *src        = &src_buffer[ch->cursor];
 
@@ -786,8 +792,7 @@ static void audio_channel_get_samples( audio_channel *ch,
       ch->cursor += samples_this_run;
       buffer_pos += samples_this_run;
       
-      if( (ch->flags & AUDIO_FLAG_LOOP) && remaining )
-      {
+      if( (ch->flags & AUDIO_FLAG_LOOP) && remaining ){
          if( format == k_audio_format_vorbis )
             stb_vorbis_seek_start( ch->vorbis_handle );
          else if( format == k_audio_format_bird )
@@ -800,8 +805,7 @@ static void audio_channel_get_samples( audio_channel *ch,
          break;
    }
 
-   while( remaining )
-   {
+   while( remaining ){
       buf[ buffer_pos*2 + 0 ] = 0.0f;
       buf[ buffer_pos*2 + 1 ] = 0.0f;
       buffer_pos ++;
@@ -819,51 +823,40 @@ static void audio_channel_mix( audio_channel *ch, float *buffer )
 
    float frame_samplerate = ch->_.sampling_rate;
 
-   if( ch->flags & AUDIO_FLAG_SPACIAL_3D )
-   {
-      if( !vg_validf(vg_audio.listener_pos[0]) ||
-          !vg_validf(vg_audio.listener_pos[1]) ||
-          !vg_validf(vg_audio.listener_pos[2]) ||
-          !vg_validf(vg_audio.listener_velocity[0]) ||
-          !vg_validf(vg_audio.listener_velocity[1]) ||
-          !vg_validf(vg_audio.listener_velocity[2]) ||
-          !vg_validf(ch->_.spacial_falloff[0]) ||
-          !vg_validf(ch->_.spacial_falloff[1]) ||
-          !vg_validf(ch->_.spacial_falloff[2]) )
-      {
-         vg_error( "NaN listener/world position (%s)\n", ch->name );
+   if( ch->flags & AUDIO_FLAG_SPACIAL_3D ){
+      v3f delta;
+      v3_sub( ch->_.spacial_falloff, vg_audio.listener_pos, delta );
+
+      float dist = v3_length( delta ),
+            vol  = vg_maxf( 0.0f, 1.0f - ch->_.spacial_falloff[3]*dist );
 
-         framevol_l = 0.0f;
-         framevol_r = 0.0f;
+      if( dist <= 0.01f ){
+         
       }
       else{
-         v3f delta;
-         v3_sub( ch->_.spacial_falloff, vg_audio.listener_pos, delta );
-
-         float dist = v3_length( delta ),
-               vol  = vg_maxf( 0.0f, 1.0f - ch->_.spacial_falloff[3]*dist );
-
          v3_muls( delta, 1.0f/dist, delta );
-         float pan  = v3_dot( vg_audio.listener_ears, delta );
+         float pan = v3_dot( vg_audio.listener_ears, delta );
          vol = powf( vol, 5.0f );
 
          framevol_l *= (vol * 0.5f) * (1.0f - pan);
          framevol_r *= (vol * 0.5f) * (1.0f + pan);
 
-         v3_muls( delta, 1.0f/dist, delta );
-
-         const float vs = 100.0f;
+         const float vs = 323.0f;
          float doppler = (vs+v3_dot(delta,vg_audio.listener_velocity))/vs;
                doppler = vg_clampf( doppler, 0.6f, 1.4f );
                
          if( fabsf(doppler-1.0f) > 0.01f )
             frame_samplerate *= doppler;
       }
+
+      if( !vg_validf( framevol_l ) ) vg_fatal_exit_loop( "NaN left channel" );
+      if( !vg_validf( framevol_r ) ) vg_fatal_exit_loop( "NaN right channel" );
+      if( !vg_validf( frame_samplerate ) ) 
+         vg_fatal_exit_loop( "NaN sample rate" );
    }
 
    u32 buffer_length = AUDIO_MIX_FRAME_SIZE;
-   if( frame_samplerate != 1.0f )
-   {
+   if( frame_samplerate != 1.0f ){
       float l = ceilf( (float)(AUDIO_MIX_FRAME_SIZE) * frame_samplerate );
       buffer_length = l+1;
    }
@@ -882,8 +875,7 @@ static void audio_channel_mix( audio_channel *ch, float *buffer )
    const float volume_start  = ch->volume_movement_start;
    const float volume_target = ch->_.volume_target;
 
-   for( u32 j=0; j<AUDIO_MIX_FRAME_SIZE; j++ )
-   {
+   for( u32 j=0; j<AUDIO_MIX_FRAME_SIZE; j++ ){
       /*
        * there is some REALLY weird behaviour with minss,
        * i cannot begin to guess what the cause is, but the bahaviour when
@@ -912,8 +904,7 @@ static void audio_channel_mix( audio_channel *ch, float *buffer )
             sample_l,
             sample_r;
       
-      if( frame_samplerate != 1.0f )
-      {
+      if( frame_samplerate != 1.0f ){
          /* absolutely garbage resampling, but it will do
           */
 
@@ -926,8 +917,7 @@ static void audio_channel_mix( audio_channel *ch, float *buffer )
          sample_l = pcf[ i0*2+0 ]*(1.0f-t) + pcf[ i1*2+0 ]*t;
          sample_r = pcf[ i0*2+1 ]*(1.0f-t) + pcf[ i1*2+1 ]*t;
       }
-      else
-      {
+      else{
          sample_l = pcf[ j*2+0 ];
          sample_r = pcf[ j*2+1 ];
       }
@@ -949,15 +939,13 @@ VG_STATIC void audio_mixer_callback( void *user, u8 *stream, int byte_count )
     * Copy data and move edit flags to commit flags
     * ------------------------------------------------------------- */
    audio_lock();
-   for( int i=0; i<AUDIO_CHANNELS; i++ )
-   {
+   for( int i=0; i<AUDIO_CHANNELS; i++ ){
       audio_channel *ch = &vg_audio.channels[i];
 
       if( !ch->allocated )
          continue;
 
-      if( ch->activity == k_channel_activity_alive )
-      {
+      if( ch->activity == k_channel_activity_alive ){
          if( (ch->cursor >= ch->source_length) && 
                !(ch->flags & AUDIO_FLAG_LOOP) )
          {
@@ -966,8 +954,7 @@ VG_STATIC void audio_mixer_callback( void *user, u8 *stream, int byte_count )
       }
 
       /* process relinquishments */
-      if( (ch->activity != k_channel_activity_reset) && ch->_.relinquished )
-      {
+      if( (ch->activity != k_channel_activity_reset) && ch->_.relinquished ){
          if(   (ch->activity == k_channel_activity_end)
             || (ch->_.volume == 0.0f)
             || (ch->activity == k_channel_activity_error) )
@@ -980,8 +967,7 @@ VG_STATIC void audio_mixer_callback( void *user, u8 *stream, int byte_count )
       }
 
       /* process new channels */
-      if( ch->activity == k_channel_activity_reset )
-      {
+      if( ch->activity == k_channel_activity_reset ){
          ch->_ = ch->editable_state;
          ch->cursor = 0;
          ch->source_length = 0;
@@ -994,27 +980,23 @@ VG_STATIC void audio_mixer_callback( void *user, u8 *stream, int byte_count )
          ch->editable_state.relinquished = ch->_.relinquished;
 
 
-      if( ch->editble_state_write_mask & AUDIO_EDIT_VOLUME )
-      {
+      if( ch->editble_state_write_mask & AUDIO_EDIT_VOLUME ){
          ch->_.volume = ch->editable_state.volume;
          ch->_.volume_target = ch->editable_state.volume;
       }
-      else
-      {
+      else{
          ch->editable_state.volume = ch->_.volume;
       }
       
 
-      if( ch->editble_state_write_mask & AUDIO_EDIT_VOLUME_SLOPE )
-      {
+      if( ch->editble_state_write_mask & AUDIO_EDIT_VOLUME_SLOPE ){
          ch->volume_movement_start = ch->_.volume;
          ch->volume_movement = 0;
          
          ch->_.volume_target = ch->editable_state.volume_target;
          ch->_.volume_rate   = ch->editable_state.volume_rate;
       }
-      else
-      {
+      else{
          ch->editable_state.volume_target = ch->_.volume_target;
          ch->editable_state.volume_rate   = ch->_.volume_rate;
       }
@@ -1026,13 +1008,11 @@ VG_STATIC void audio_mixer_callback( void *user, u8 *stream, int byte_count )
          ch->editable_state.sampling_rate = ch->_.sampling_rate;
 
 
-      if( ch->editble_state_write_mask & AUDIO_EDIT_LFO_ATTACHMENT )
-      {
+      if( ch->editble_state_write_mask & AUDIO_EDIT_LFO_ATTACHMENT ){
          ch->_.lfo        = ch->editable_state.lfo;
          ch->_.lfo_amount = ch->editable_state.lfo_amount;
       }
-      else
-      {
+      else{
          ch->editable_state.lfo        = ch->_.lfo;
          ch->editable_state.lfo_amount = ch->_.lfo_amount;
       }
@@ -1050,16 +1030,13 @@ VG_STATIC void audio_mixer_callback( void *user, u8 *stream, int byte_count )
       ch->editble_state_write_mask  = 0x00;
    }
 
-   for( int i=0; i<AUDIO_LFOS; i++ )
-   {
+   for( int i=0; i<AUDIO_LFOS; i++ ){
       audio_lfo *lfo = &vg_audio.oscillators[ i ];
 
-      if( lfo->editble_state_write_mask & AUDIO_EDIT_LFO_WAVE )
-      {
+      if( lfo->editble_state_write_mask & AUDIO_EDIT_LFO_WAVE ){
          lfo->_.wave_type = lfo->editable_state.wave_type;
 
-         if( lfo->_.wave_type == k_lfo_polynomial_bipolar )
-         {
+         if( lfo->_.wave_type == k_lfo_polynomial_bipolar ){
             lfo->_.polynomial_coefficient = 
                lfo->editable_state.polynomial_coefficient;
             lfo->sqrt_polynomial_coefficient = 
@@ -1067,18 +1044,15 @@ VG_STATIC void audio_mixer_callback( void *user, u8 *stream, int byte_count )
          }
       }
 
-      if( lfo->editble_state_write_mask & AUDIO_EDIT_LFO_PERIOD )
-      {
-         if( lfo->_.period )
-         {
+      if( lfo->editble_state_write_mask & AUDIO_EDIT_LFO_PERIOD ){
+         if( lfo->_.period ){
             float t = lfo->time;
                   t/= (float)lfo->_.period;
 
             lfo->_.period = lfo->editable_state.period;
             lfo->time = lfo->_.period * t;
          }
-         else
-         {
+         else{
             lfo->time = 0;
             lfo->_.period = lfo->editable_state.period;
          }
@@ -1093,12 +1067,10 @@ VG_STATIC void audio_mixer_callback( void *user, u8 *stream, int byte_count )
    /*
     * Process spawns
     * ------------------------------------------------------------- */
-   for( int i=0; i<AUDIO_CHANNELS; i++ )
-   {
+   for( int i=0; i<AUDIO_CHANNELS; i++ ){
       audio_channel *ch = &vg_audio.channels[i];
 
-      if( ch->activity == k_channel_activity_wake )
-      {
+      if( ch->activity == k_channel_activity_wake ){
          if( audio_channel_load_source( ch ) )
             ch->activity = k_channel_activity_alive;
          else
@@ -1116,26 +1088,22 @@ VG_STATIC void audio_mixer_callback( void *user, u8 *stream, int byte_count )
    for( int i=0; i<frame_count*2; i ++ )
       pOut32F[i] = 0.0f;
 
-   for( int i=0; i<AUDIO_LFOS; i++ )
-   {
+   for( int i=0; i<AUDIO_LFOS; i++ ){
       audio_lfo *lfo = &vg_audio.oscillators[i];
       lfo->time_startframe = lfo->time;
    }
 
-   for( int i=0; i<AUDIO_CHANNELS; i ++ )
-   {
+   for( int i=0; i<AUDIO_CHANNELS; i ++ ){
       audio_channel *ch = &vg_audio.channels[i];
 
-      if( ch->activity == k_channel_activity_alive )
-      {
+      if( ch->activity == k_channel_activity_alive ){
          if( ch->_.lfo )
             ch->_.lfo->time = ch->_.lfo->time_startframe;
 
          u32 remaining = frame_count,
              subpos    = 0;
 
-         while( remaining )
-         {
+         while( remaining ){
             audio_channel_mix( ch, pOut32F+subpos );
             remaining -= AUDIO_MIX_FRAME_SIZE;
             subpos += AUDIO_MIX_FRAME_SIZE*2;
@@ -1152,8 +1120,7 @@ VG_STATIC void audio_mixer_callback( void *user, u8 *stream, int byte_count )
 
    audio_lock();
 
-   for( int i=0; i<AUDIO_CHANNELS; i ++ )
-   {
+   for( int i=0; i<AUDIO_CHANNELS; i ++ ){
       audio_channel *ch = &vg_audio.channels[i];
       ch->readable_activity = ch->activity;
    }
@@ -1170,8 +1137,7 @@ VG_STATIC void audio_mixer_callback( void *user, u8 *stream, int byte_count )
 
    vg_audio.samples_last = frame_count;
 
-   if( vg_audio.debug_ui )
-   {
+   if( vg_audio.debug_ui ){
       vg_dsp_update_texture();
    }
 
@@ -1189,8 +1155,15 @@ VG_STATIC void audio_clip_load( audio_clip *clip, void *lin_alloc )
    /* TODO: This contains audio_lock() and unlock, but i don't know why
     *       can probably remove them. Low priority to check this */
 
-   if( format == k_audio_format_vorbis )
-   {
+   /* TODO: packed files for vorbis etc, should take from data if its not not 
+    *       NULL when we get the clip
+    */
+
+   if( format == k_audio_format_vorbis ){
+      if( !clip->path ){
+         vg_fatal_exit_loop( "No path specified, embeded vorbis unsupported" );
+      }
+
       audio_lock();
       clip->data = vg_file_read( lin_alloc, clip->path, &clip->size );
       audio_unlock();
@@ -1201,25 +1174,34 @@ VG_STATIC void audio_clip_load( audio_clip *clip, void *lin_alloc )
       float mb = (float)(clip->size) / (1024.0f*1024.0f);
       vg_info( "Loaded audio clip '%s' (%.1fmb)\n", clip->path, mb );
    }
-   else if( format == k_audio_format_stereo )
-   {
+   else if( format == k_audio_format_stereo ){
       vg_fatal_exit_loop( "Unsupported format (Stereo uncompressed)" );
    }
-   else if( format == k_audio_format_bird )
-   {
-      u32 len = strlen( clip->path ),
-          size = synth_bird_memory_requirement( len );
+   else if( format == k_audio_format_bird ){
+      if( !clip->data ){
+         vg_fatal_exit_loop( "No data, external birdsynth unsupported" );
+      }
 
-      if( size > AUDIO_DECODE_SIZE )
-         vg_fatal_exit_loop( "Bird code too long\n" );
+      u32 total_size  = clip->size + sizeof(struct synth_bird);
+          total_size -= sizeof(struct synth_bird_settings);
+          total_size  = vg_align8( total_size );
 
-      clip->size = size;
-      clip->data = vg_linear_alloc( lin_alloc, size );
+      if( total_size > AUDIO_DECODE_SIZE )
+         vg_fatal_exit_loop( "Bird coding too long\n" );
 
-      synth_bird_load( clip->data, clip->path, len );
+      struct synth_bird *bird = vg_linear_alloc( lin_alloc, total_size );
+      memcpy( &bird->settings, clip->data, clip->size );
+
+      clip->data = bird;
+      clip->size = total_size;
+
+      vg_info( "Loaded bird synthesis pattern (%u bytes)\n", total_size );
    }
-   else
-   {
+   else{
+      if( !clip->path ){
+         vg_fatal_exit_loop( "No path specified, embeded mono unsupported" );
+      }
+
       vg_linear_clear( vg_mem.scratch );
       u32 fsize;
 
@@ -1234,8 +1216,7 @@ VG_STATIC void audio_clip_load( audio_clip *clip, void *lin_alloc )
       stb_vorbis *decoder = stb_vorbis_open_memory( 
                             filedata, fsize, &err, &alloc );
 
-      if( !decoder )
-      {
+      if( !decoder ){
          vg_error( "stb_vorbis_open_memory failed on '%s' (%d)\n", 
                      clip->path, err );
          vg_fatal_exit_loop( "Vorbis decode error" );
@@ -1331,8 +1312,7 @@ VG_STATIC void audio_debug_ui( m4x4f mtx_pv )
    u32 overlap_length = 0;
 
        /* Draw audio stack */
-       for( int i=0; i<AUDIO_CHANNELS; i ++ )
-       {
+       for( int i=0; i<AUDIO_CHANNELS; i ++ ){
       audio_channel *ch = &vg_audio.channels[i];
 
                vg_uictx.cursor[2] = 400;
@@ -1340,8 +1320,7 @@ VG_STATIC void audio_debug_ui( m4x4f mtx_pv )
                
                ui_new_node();
 
-      if( !ch->allocated )
-      {
+      if( !ch->allocated ){
          ui_fill_rect( vg_uictx.cursor, 0x50333333 );
 
          ui_end_down();
@@ -1399,16 +1378,14 @@ VG_STATIC void audio_debug_ui( m4x4f mtx_pv )
                ui_end_down();
                vg_uictx.cursor[1] += 1;
       
-      if( AUDIO_FLAG_SPACIAL_3D )
-      {
+      if( AUDIO_FLAG_SPACIAL_3D ){
          v4f wpos;
          v3_copy( ch->editable_state.spacial_falloff, wpos );
 
          wpos[3] = 1.0f;
          m4x4_mulv( mtx_pv, wpos, wpos );
 
-         if( wpos[3] > 0.0f )
-         {
+         if( wpos[3] > 0.0f ){
             v2_muls( wpos, (1.0f/wpos[3]) * 0.5f, wpos );
             v2_add( wpos, (v2f){ 0.5f, 0.5f }, wpos );
             
@@ -1418,11 +1395,9 @@ VG_STATIC void audio_debug_ui( m4x4f mtx_pv )
             wr[2] = 100;
             wr[3] = 17;
             
-            for( int j=0; j<12; j++ )
-            {
+            for( int j=0; j<12; j++ ){
                int collide = 0;
-               for( int k=0; k<overlap_length; k++ )
-               {
+               for( int k=0; k<overlap_length; k++ ){
                   ui_px *wk = overlap_buffer[k];
                   if( ((wr[0] <= wk[0]+wk[2]) && (wr[0]+wr[2] >= wk[0])) &&
                       ((wr[1] <= wk[1]+wk[3]) && (wr[1]+wr[3] >= wk[1])) )