Fixed PPCSetUpBefore...Resource to work on Pharo. Few othr minor Pharo fixes.

Portability fix: do not use "/ style comments in PetitParser code.

Added smoke tests for LRPParser

Added LRPParser from Live Robots Prograaming project

Tests refactoring - use generated test cases to make sure all posibilities are tested. Do not generate resource for all combinations, use PPCSetUpBeforeTearDownAfterResource instead that delegates parser compilation to the testcase itself (it calls it's #setUpBefore method).

Inline child of an action node only is its unnamed sequence node. Named nodes should not be inlined as they should make a method. There's little point in inlining non-sequence nodes, so don't enforce inlining on those. Some (JK :-) may prefer them non-inlined (for debugging purposes)

Temporarily disabled #acceptsEpsilon test in PPCTokenizingCodeGenerator>>visitChoiceNode: JK should have a look and come up with proper solution.

Workaround for stc bug.

Added static analysis of blocks when inlining. Allow inlining only when block is functional ...i.e., does not access any shared state (in instance or class variables). If the block does a self-send, the sent method has to be (transitively) functional too. To allow for self-sends in action blocks, copy (transitively) self-sent methods to target parser. This is safe as these self-sent methods are guarnateed to be functional.

Post-merge fixes. PPCCodeBlock refactoring.

Portability fix: override #new for class that implements #initialize #initialize is not sent by default.

Merge

Updated to PetitCompiler-JanVrany.135, PetitCompiler-Tests-JanKurs.93, PetitCompiler-Extras-Tests-JanVrany.16, PetitCompiler-Benchmarks-JanKurs.12 Name: PetitCompiler-JanVrany.135 Author: JanVrany Time: 22-07-2015, 06:53:29.127 PM UUID: 890178b5-275d-46af-a2ad-1738998f07cb Ancestors: PetitCompiler-JanVrany.134 Name: PetitCompiler-Tests-JanKurs.93 Author: JanKurs Time: 20-07-2015, 11:30:10.283 PM UUID: 6473e671-ad70-42ca-b6c3-654b78edc531 Ancestors: PetitCompiler-Tests-JanKurs.92 Name: PetitCompiler-Extras-Tests-JanVrany.16 Author: JanVrany Time: 22-07-2015, 05:18:22.387 PM UUID: 8f6f9129-dbba-49b1-9402-038470742f98 Ancestors: PetitCompiler-Extras-Tests-JanKurs.15 Name: PetitCompiler-Benchmarks-JanKurs.12 Author: JanKurs Time: 06-07-2015, 02:10:06.901 PM UUID: cb24f1ac-46a4-494d-9780-64576f0f0dba Ancestors: PetitCompiler-Benchmarks-JanKurs.11, PetitCompiler-Benchmarks-JanVrany.e29bd90f388e.20150619081300

Moved PPMappedActionParser>>asCompilerNode to protocol 'converting' If in '*petitparser' it makes some problems with Monticello.

Use RBVariableNode>>name: to change variable's name. The old way - `node token value:` is working in Pharo - there's no token in RBVariableNode. Sigh.

Portability fix: do not use #? ...use `notNil ifTrue:[] ifFalse:[]` instead. #? is not supported in Squeak/Pharo

Moved PPCMappedActionNode to category PetitCompiler-Nodes

Do not use RBProgramNode>>variableNodesDo: as this method is not present in Pharo. Use local helper method instead.

Reverted fc3dbe5654c5: sending #copy should be OK (copy set's parent properly)

Send #sourceNode instead of #ast as there's no #ast in Pharo ...I was mistaken.

Oops, removed EOL comments as they;re not supported in Squeak / Pharo

Fixed target mcz - exit Smalltalk after all packages are exported.

Use #deepCopy instead of #copy when copying RB parse tree ...because #copy make a copy if child nodes but does not set their parents properly. Therefore node replacing does not work because it replaces it in wrong tree (original, not the copy).

Codegen: added support for smart action node compiling. Avoid creation of intermediate result collection for action nodes if all references to action block's argument (i.e., the nodes collection) is in form of: * <nodes> at: <numeric constant> * <nodes> first (second, third...

Oops, renamed testMappedNode3/4/5 to testActionNode3/4/5 which is the correct name!

Fix in codegen for inlined sequence nodes. For inlined sequence nodes, generate nested ifs rather than sequential code which does not work when inlined. The reason is that #codeReturn: in inline generates instvar assignment, not method return, so in sequential code the next child of a sequence will be probed even if previous failed. If that happends, the whole sequence fail and therefore we must generate nested ifs to correctly handle this w.r.t. inlining.

Avoid creation of intermediate collection for mapped action nodes.

Fix in codegen: fixed compilation of unknown node: do not hardcode return variable name ...use `self retvalVar` instead

Renamed #codeStoreValueOf:intoVariable: to #codeAssignParsedValueOf:to: to use same naming as #codeAssign:to:

Fixed codegen for empty (mapped) action block. Although empty blocks are useless in practice, they may occur (and do occur in tests)