c error: case label does not reduce to an integer constant

Question

This is the switch case concerned by the error:

switch(event.type)
        {
        case SDL_Quit:
            for (int i=0;i<NUMMENU;i++)
                SDL_FreeSurface(menus[i]);
            return 1;

        case SDL_MOUSEMOTION:
            x=event.motion.x;
            y=event.motion.y;
            for(int i=0;i<NUMMENU;i++)
            {
                if(x>=position[i].x && x<=position[i].x+position[i].w && y>=position[i].y && y<=position[i].y+position[i].h)
                    {
                    if(!selected[i])
                    {
                        selected[i]=1;
                        SDL_FreeSurface(menus[i]);
                        menus[i]=TTF_RenderText_Solid(font,labels[i],color[1]);
                    }
                }else{
                        if(selected[i])
                        {
                            selected[i]=0;
                            SDL_FreeSurface(menus[i]);
                            menus[i]=TTF_RenderText_Solid(font,labels[i],color[0]);
                        }
                      }
            }

        case SDL_MOUSEBUTTONDOWN:
            x=event.button.x;
            y=event.button.y;
            for(int i=0;i<NUMMENU;i++)
                if(x>=position[1].x && x<=position[1].x+position[i].w && y>=position[i].y && y<=position[i].y+position[i].h)
                {
                    for(int j=0;j<NUMMENU;j++)
                        SDL_FreeSurface(menus[j]);
                    return i;
                }
                break;
        case SDL_KEYDOWN:
            if(event.key.keysym.sym==SDLK_ESCAPE)
            {
                for(int i=0;i<NUMMENU;i++)
                    SDL_FreeSurface(menus[i]);
                return 0;
            }

        }

I have this error and I haven't any idea of why and what should I do to resolve this. I have tried to add break; at the end of each case but no results.

If you need more code or further information, I can give precisions.

Regards,

have you tried moving the declaration of variable `i` outside the case statements — cmidi, Jan 21 '15 at 15:17
`SDL_xxx` are probably const variables not macros, and those are not allowed to be used as case labels. — Marlon, Jan 21 '15 at 15:18
http://stackoverflow.com/questions/92396/why-cant-variables-be-declared-in-a-switch-statement — cmidi, Jan 21 '15 at 15:19
possible duplicate of [switch case: error: case label does not reduce to an integer constant](http://stackoverflow.com/questions/14069737/switch-case-error-case-label-does-not-reduce-to-an-integer-constant) — Marlon, Jan 21 '15 at 15:19
@Marlon: Huh? I would assume `SDL_Quit` is not an integer constant but maybe a function. But if it is an integer constant then of course it's valid as a case label. — R.. GitHub STOP HELPING ICE, Jan 21 '15 at 15:19
Indeed, it was only a fail on the SDL_Quit, SDL_QUIT resolved the problem.. Could you make an answer, to uprate it and agree as answer? — Flo, Jan 21 '15 at 15:22

score 5 · Accepted Answer · answered Jan 21 '15 at 15:20

5

You are using the function void SDL_Quit(void) rather than the SDL_QUIT event type at line 3.

case SDL_Quit:
     ^^^^^^^^
        for (int i=0;i<NUMMENU;i++)
            SDL_FreeSurface(menus[i]);
        return 1;

answered Jan 21 '15 at 15:20

JBarberU

999
8
16

1

This is what I suspected (see my comments) but I'm not familiar with SDL. Thanks for getting the right info for the answer. – R.. GitHub STOP HELPING ICE Jan 21 '15 at 15:21

score 0 · Answer 2 · answered Jan 21 '15 at 15:20

0

Your case labels such as SDL_Quit/SDL_MOUSEMOTION... could not be integer constants. For example its valid if they are characters labels but not something like const char * string literals which are not integer constants.

answered Jan 21 '15 at 15:20

Sunil Bojanapally

12,528
4
33
46

c error: case label does not reduce to an integer constant

2 Answers2