search

rdsteed / la-pe

Automatically exported from code.google.com/p/la-pe
0 stars 0 forks source link

All compiler defines are parsed at all times #36

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
What will reproduce the problem?
  If you compile the test program with a handler for OnHandleDirective.
  Then have any {$IFDEF SomethingNoDefined}{$MyCustomDirective}{$ENDIF}

What is the expected output? What do you see instead?
  It shouldn't call handleDirective (or at least have a param stating were in a section of code that shouldn't be there...

Which version are you using?
  Latest

Please provide any additional information below.
  This is an issue for custom compiler defines being used inside of a IFDEF.

Original issue reported on code.google.com by Dgby...@gmail.com on 7 Oct 2014 at 4:02

GoogleCodeExporter commented 8 years ago 
Not the greatest title.

I think we should either pass InIgnore to the handler or not call the handler 
if InIgnore.

Original comment by Dgby...@gmail.com on 7 Oct 2014 at 4:07

GoogleCodeExporter commented 8 years ago 
Here is a patch I've wrote to do the former of my previous commit.

Original comment by Dgby...@gmail.com on 7 Oct 2014 at 4:15

Attachments:

GoogleCodeExporter commented 8 years ago 
Thanks, applied your patch!

Original comment by niels....@gmail.com on 28 Nov 2014 at 8:10