Злой код Опции

[uragan90]

Здравствуйте уважаемые!
Написал такой вот код в keil 4.73
А он не компилируется не в какую, где может быть косяк???

Код

#include <math.h>

unsigned int Massine[8000];

typedef union{
int ch1_ch2;
char ch[2];    
}chn;

void  sinetest(void)
{    
    float ampvol_ch1  = 65;  
    float period_ch1   = 20;
//-----------------------------------------
    float ampvol_ch2  = 45;  
    float period_ch2   = 30;  
//-----------------------------------------  
    float pi   = 3.14;
    int index  = 8000;
    int i      = 0;
//-----------------------------------------
    chn channel;
    
      for(i=0; i<index; i++)
    {
            
      channel.ch[0] = 128 + (ampvol_ch1*sin(i*pi*period_ch1));  
      channel.ch[1] = 128 + (ampvol_ch2*sin(i*pi*period_ch2));
      Massine[i]      = channel.ch1_ch2;             
    }
}

Интересует правильность синтексиса!
А то я в майне вызываю эту функцию чтоб она заполнила массив, а код не компилируется!
Но ведь с виду ошибок не вижу

[uragan90]

От этой ошибки удалось иавится!
Теперь ругается на обьявление
chn channel;
пишет что ранее обьявлен мог быть...
Да что за фигня то?

union
в кейле так и не удалось попробовать!

[winipuh]

Теперь ругается на обьявление chn channel; пишет что ранее обьявлен мог быть...

А Вы приведите тут варнинг как он есть, на английском.

Попутно

1. А Вы там с union не перемудрили? Надо то и всего Massine[i] = ch[0] | ((unsigned int)ch[1] << 8) ...
2. Зачем используете int для Massine, если на каждый сэмпл используется только 2 байта (по байту на канал).
   Оставшиеся 4000 байт врагу подарите?
3. float pi = 3.14 — это не наш метод! Про дефайн "M_PI" в хедере math.h знаете?
4. Как насчет такого кода (ver.2):
   
   Код
   #include <stdio.h>
   #include <stdlib.h>
   #define _USE_MATH_DEFINES  // это нужно только для VisualStudio
   #include <math.h>
   
   typedef union {
       unsigned short data;
       struct {
           unsigned char left;
           unsigned char right;
       } chan;
   } stereo_sample_t;
   
   struct sine_param {
       float amp_vol;
       float period;
   };
   
   static unsigned char get_sine_sample(const struct sine_param *p, int i) {
       return 128u + (unsigned char)(p->amp_vol * sin(M_PI * p->period * i));
   }
   
   static void sine_test(stereo_sample_t buf[], int num_samples) {
       struct sine_param left_ch = {65, 20};
       struct sine_param right_ch = {45, 30};
       int i;
   
       for(i = 0; i < num_samples; i++) {
           buf[i].chan.left = get_sine_sample(&left_ch, i);
           buf[i].chan.right = get_sine_sample(&right_ch, i);
       }
   }
   
   
   int main(void) {
       stereo_sample_t Massine[8000];
   
       sine_test(Massine, sizeof(Massine) / sizeof(Massine[0]));
       return 0;
   }
   Так вот немного приятнее...

Сообщение отредактировал winipuh - Dec 2 2013, 20:19