DelphiAST does not support complex compiler directives about delphiast HOT 4 CLOSED

The following are also not supported:
{$IF SOME_CONSTANT >= SOME_INTEGER}
{$IF SOME_CONSTANT > SOME_INTEGER}
{$IF SOME_CONSTANT = SOME_INTEGER}
{$IF SOME_CONSTANT < SOME_INTEGER}
{$IF SOME_CONSTANT <= SOME_INTEGER}

(Basically, any expression other than "DEFINED" or "NOT DEFINED")

from delphiast.

Re my last comment, I suggest we make "complex Conditional Expression evaluation" a separate issue as this would be best implemented AFTER resolving issue #30.

A working solution to the original context specified in this issue has been committed to my fork, and a pull request issued. It has been tested with every combination of expression, and appears to work very well indeed.

from delphiast.

Now it's much better! But we also have to remember that brackets is also not supported yet. Like:

{$IF DEFINED(A) AND (DEFINED(B) OR DEFINED(C))}

I'll create a new issue later to better describe the current state.

from delphiast.

I did think about that before I implemented my intermediary solution.

The only thing there is I'd have to define and build an "expression tree" before evaluating it. It's not exactly the most difficult thing in the world, and I'm going to get that done hopefully this coming weekend when I have some time free.

The reason I did the intermediary solution is that certain units in a particular codebase were using multiple conditional expressions and wouldn't parse. Thus far, I haven't encountered any "conditional expression trees" in our codebase, so I didn't put the extra time in (at this point).

Beauty of outsourcing the solution into its own method (rather than all rolled into that massive CASE OF statement) is that we need only implement it once, and it applies to both {$IF} and {$ELSEIF} statements identically.

from delphiast.