No Mark, it seems you missed what I said.
I showed what the expression *does* as written.
And I explained why it's *wrong* as written.

To clear up confusion, here is what the expression *should* be, in order for
it to work as the author really intended:

isMemoFindKey =
    event.eType == keyDownEvent
    &&
    (
        (
            TxtCharIsHardKey(event.data.keyDown.modifiers,
                    .data.keyDown.chr)
            &&
            event.data.keyDown.chr == vchrHard4
        )
        ||
        event.data.keyDown.chr == vchrFind
    )

See the problem now?


"Mark Wilden" <[EMAIL PROTECTED]> wrote in message
news:[EMAIL PROTECTED]
>
> ----- Original Message -----
> From: "Chris Antos" <[EMAIL PROTECTED]>
>
>
> > First of all, your operator precedence is messed up -- read up on the
> > precedence of && vs || and add more parentheses to make it explicit what
> > conditions you wanted to test.
> > [...]
> > isMemoFindKey =
> >     (
> >         (event.eType == keyDownEvent) &&
> >         (TxtCharIsHardKey(event.data.keyDown.modifiers,
> >                 event.data.keyDown.chr)) &&
> >         (event.data.keyDown.chr == vchrHard4)
> >     )
> >     ||
> >     (
> >         event.data.keyDown.chr == vchrFind
> >     )
>
> Actually, this doesn't need any additional parentheses.
>
>  isMemoFindKey =
>          event.eType == keyDownEvent &&
>          TxtCharIsHardKey(event.data.keyDown.modifiers,
>                  event.data.keyDown.chr) &&
>          event.data.keyDown.chr == vchrHard4
>      ||   event.data.keyDown.chr == vchrFind
>
>
>



-- 
For information on using the Palm Developer Forums, or to unsubscribe, please see 
http://www.palmos.com/dev/support/forums/

Reply via email to