Проблема определения макросов в C - элемент Initializer не является константой - PullRequest
0 голосов
/ 23 декабря 2018

Я программирую для Game Boy Advance, и мне нужно иметь список областей памяти для каждой области (куча ОЗУ и ПЗУ).

Однако при определении макросов в заголовочном файле, компилятор указал, что в одном макросе error: initializer element is not constant.

enter image description here

Вот мой полный заголовочный файл (около 90 строк), который я позаимствовалот Tonc:

#ifndef TOOLBOX_H
#define TOOLBOX_H

// === (from tonc_types.h) ============================================

typedef unsigned char   u8;
typedef unsigned short  u16;
typedef unsigned int    u32;

typedef u16 COLOR;

#define INLINE static inline

// === (from tonc_memmap.h) ===========================================

#define MEM_IO      0x04000000
#define MEM_VRAM    0x06000000
#define GAMEPAK_RAM 0x0E000000

#define REG_DISPCNT     *((volatile u32*)(MEM_IO+0x0000))
#define REG_VCOUNT      *(volatile u16*)(MEM_IO+0x0006) // vertical count

// === (from tonc_memdef.h) ===========================================

// --- REG_DISPCNT defines ---
#define DCNT_MODE0      0x0000
#define DCNT_MODE1      0x0001
#define DCNT_MODE2      0x0002
#define DCNT_MODE3      0x0003
#define DCNT_MODE4      0x0004
#define DCNT_MODE5      0x0005
// layers
#define DCNT_BG0        0x0100
#define DCNT_BG1        0x0200
#define DCNT_BG2        0x0400
#define DCNT_BG3        0x0800
#define DCNT_OBJ        0x1000

#define save_mem        ((u8*)GAMEPAK_RAM) //<- Error here

// === (from tonc_video.h) ============================================

#define SCREEN_WIDTH   240
#define SCREEN_HEIGHT  160

#define vid_mem        ((u16*)MEM_VRAM) //But not here

INLINE void m3_plot(int x, int y, COLOR clr)
{   vid_mem[y*SCREEN_WIDTH+x]= clr;    }

#define CLR_BLACK   0x0000
#define CLR_RED     0x001F
#define CLR_LIME    0x03E0
#define CLR_YELLOW  0x03FF
#define CLR_BLUE    0x7C00
#define CLR_MAG     0x7C1F
#define CLR_CYAN    0x7FE0
#define CLR_WHITE   0x7FFF

INLINE int CLAMP(int val, int min, int max)
{
    if(val >= max)
    {
        val = max - 1;
    }
    else if(val < min)
    {
        val = min;
    }
    else
    {
        return val;
    }
    return val;
}

INLINE COLOR RGB15(u32 red, u32 green, u32 blue)
{   return red | (green<<5) | (blue<<10);   }

INLINE u16 * get_RGB(COLOR clr)
{
    u16 red, green, blue;
    red = clr & 31;
    green = (clr >> 5) & 31;
    blue = clr >> 10;
    static u16 rgb[3];
    rgb[0] = red; rgb[1] = green; rgb[2] = blue;
    return rgb;
}

INLINE void vid_vsync()
{
    while(REG_VCOUNT >= 160);   // wait till VDraw
    while(REG_VCOUNT < 160);    // wait till VBlank
}

#endif // TOOLBOX_H

Однако в моем коде изначально не было этих двух строк:

#define GAMEPAK_RAM 0x0E000000

и

#define save_mem ((u8*)GAMEPAK_RAM)

Если вы заметили, я намеренно определил save_mem почти так же, как vid_mem, за исключением того, что мне нужно было преобразовать его в u8, потому что эта память могла только читать / записывать 8 бит за раз.*

Тогда я подумал, что если будут ошибки, то в обеих строках должны быть ошибки.Однако только save_mem был пойман с error: initializer element is not constant.(Я лично прокомментировал строку save_mem, чтобы увидеть, будет ли у vid_mem такая проблема.)

Кроме того, я уже занимался программированием своей игры для GBA и уже использовал vid_mem для многихраз, так что это озадачивает меня еще больше.

Мне интересно, почему это происходит и как это исправить. (Уверен, мой компилятор в порядке)

Заранее спасибо.

РЕДАКТИРОВАТЬ:

Вот код, где я использую макрос save_mem:

#include "toolbox.h"

u8 played = save_mem[0];
u8 coins = save_mem[1] | (save_mem[2] << 8) | (save_mem[3] << 16) | 
(save_mem[4] < 24);

void get_coins()
{
    coins = save_mem[1] | (save_mem[2] << 8) | (save_mem[3] << 16) | 
(save_mem[4] < 24);
}

void save_coins(int amount)
{
    save_mem[1] = (amount & 255);
    save_mem[2] = ((amount >> 8) & 255);
    save_mem[3] = ((amount >> 16) & 255);
    save_mem[4] = ((amount >> 24) & 255);
}

void set_played()
{
    save_mem[0] = 1;
}

1 Ответ

0 голосов
/ 23 декабря 2018

Я мог воспроизвести вашу проблему, выполнив

u8 played = save_mem[0];

вне какой-либо функции.Это не ошибка в вашем define, а ошибка в приведенной выше строке.Способ представления ошибки действительно вводит в заблуждение.В любом случае, ошибка возникает из-за того, что в приведенной выше строке программа будет загружать содержимое из памяти, и, как вы ожидаете, компилятор не сможет узнать, что будет в этой памяти во время компиляции, а инициализация глобальных переменных принимает только константы.,Вы должны переместить эту строку кода внутри функции.

Код, который я использовал для проверки:

#ifndef TOOLBOX_H
#define TOOLBOX_H

// === (from tonc_types.h) ============================================

typedef unsigned char   u8;
typedef unsigned short  u16;
typedef unsigned int    u32;

typedef u16 COLOR;

#define INLINE static inline

// === (from tonc_memmap.h) ===========================================

#define MEM_IO      0x04000000
#define MEM_VRAM    0x06000000
#define GAMEPAK_RAM 0x0E000000

#define REG_DISPCNT     *((volatile u32*)(MEM_IO+0x0000))
#define REG_VCOUNT      *(volatile u16*)(MEM_IO+0x0006) // vertical count

// === (from tonc_memdef.h) ===========================================

// --- REG_DISPCNT defines ---
#define DCNT_MODE0      0x0000
#define DCNT_MODE1      0x0001
#define DCNT_MODE2      0x0002
#define DCNT_MODE3      0x0003
#define DCNT_MODE4      0x0004
#define DCNT_MODE5      0x0005
// layers
#define DCNT_BG0        0x0100
#define DCNT_BG1        0x0200
#define DCNT_BG2        0x0400
#define DCNT_BG3        0x0800
#define DCNT_OBJ        0x1000

#define save_mem        ((u8*)GAMEPAK_RAM) //<- Error here

// === (from tonc_video.h) ============================================

#define SCREEN_WIDTH   240
#define SCREEN_HEIGHT  160

#define vid_mem        ((u16*)MEM_VRAM) //But not here

INLINE void m3_plot(int x, int y, COLOR clr)
{   vid_mem[y*SCREEN_WIDTH+x]= clr;    }

#define CLR_BLACK   0x0000
#define CLR_RED     0x001F
#define CLR_LIME    0x03E0
#define CLR_YELLOW  0x03FF
#define CLR_BLUE    0x7C00
#define CLR_MAG     0x7C1F
#define CLR_CYAN    0x7FE0
#define CLR_WHITE   0x7FFF

INLINE int CLAMP(int val, int min, int max)
{
    if(val >= max)
    {
        val = max - 1;
    }
    else if(val < min)
    {
        val = min;
    }
    else
    {
        return val;
    }
    return val;
}

INLINE COLOR RGB15(u32 red, u32 green, u32 blue)
{   return red | (green<<5) | (blue<<10);   }

INLINE u16 * get_RGB(COLOR clr)
{
    u16 red, green, blue;
    red = clr & 31;
    green = (clr >> 5) & 31;
    blue = clr >> 10;
    static u16 rgb[3];
    rgb[0] = red; rgb[1] = green; rgb[2] = blue;
    return rgb;
}

INLINE void vid_vsync()
{
    while(REG_VCOUNT >= 160);   // wait till VDraw
    while(REG_VCOUNT < 160);    // wait till VBlank
}

#endif // TOOLBOX_H

/*void bob(){*/
    u8 played = save_mem[0];
/*}*/

Для компиляции я скопировал со скриншота:

arm-none-eabi-gcc -mthumb-interwork  -mthumb -O2 test.c -c

но добавлено -c, чтобы избежать ссылки (так как нет основной функции).

...