Restricting input to an edit control

I can't believe it, but I'm having trouble restricting input to an edit control. I only want the user to be able to enter in certain keys, but placing the restriction in WM_KEYDOWN didn't work:

Code:

---

        case WM_KEYDOWN:
                if (wParam!='I')
                        if (wParam!='V')
                                if (wParam!='X')
                                        if (wParam!='L')
                                                if (wParam!='C')
                                                        if (wParam!='M')
                                                                if (wParam!='~')
                                                                        return 0;
                break;

---

The user can still enter anything into the edit control. How can I fix this problem?

:p Don't worry, I put the restriction code in WM_CHAR and now it works fine. I feel so.... n00bish.

Code:

---

        case WM_CHAR:
                if (wParam!='I')
                        if (wParam!='V')
                                if (wParam!='X')
                                        if (wParam!='L')
                                                if (wParam!='C')
                                                        if (wParam!='M')
                                                                if (wParam!='~')
                                                                        return 0;

                break;

---

Dude, use &&

Probably works out to the same assembly commands confuted. But anyway, is this what you mean?

Code:

---

        case WM_CHAR:
                if (wParam!='I' &&
                wParam!='V' &&
                wParam!='X' &&
                wParam!='L' &&
                wParam!='C' &&
                wParam!='M' &&
                wParam!='~')
                    return 0;

                break;

---

Yes, that's what I meant. I hadn't thought about it working out to the same ASM... I wonder if it does.

Well, let's consider what happens:

My code:
1 MOV wParam to register
2 MOV 'I' to register
3 JE 8
4 MOV wParam to register
5 MOV 'V' to register
6 JE 8
7 return 0; //whatever that would be in ASM
8 break;

Your code:
Can't be bothered trying it but maybe it would contain a counter to check how many expressions come out TRUE or something. It might depend on the compiler, ie, your code may end up exactly the same as mine in the end.

Who really cares though? I'll just do whichever one takes the least time to type out.

Pointless post follows.

Code:

---

switch (wParam) {
        case  'I':
        case  'V':
        case  'X':
        case  'L':
        case  'C':
        case  'M':
        case  '~':
                break;
        default:
                return 0;
}

---

Yeah I guess that'd work too.