On 16 Apr 2018, at 11:18, Marcus Denker <marcus.denker@inria.fr> wrote:



On 12 Apr 2018, at 17:18, Marcus Denker <marcus.denker@inria.fr> wrote:



On 12 Apr 2018, at 17:06, Hilaire <hilaire@drgeo.eu> wrote:

Hi Marcus, Benoit,

Nice to know. I was testing in a set up with both .changes and source files. So I was suspecting something like what your described. I can imagine in production code your feature to speed up compilation will be appreciated. For Dr. Geo and Smalltalk sketch, it is better to have variable names.


I should have a fix ready tomorrow.


I will for now revert the change:
https://github.com/pharo-project/pharo/pull/1199

This way it behaves as people expect, I have an idea how to provide an option for the faster way.
This will come next.

1) added compiler option to compile with source embedded:
https://github.com/pharo-project/pharo/pull/1200
2) then we can use this when evaluating, which avoids the double recompile that we did:
https://github.com/pharo-project/pharo/pull/1201

We regain now a lot of speed by avoiding the double recompile, which had been the real source of the speed problem.

The pretty-print of course is still done and costs��� so this could be restructured further. e.g. we could add an explicit option to not pretty print. 

Marcus