[pharo-project/pharo-core] 2787bd: 50151

  Branch: refs/heads/5.0
  Home:   https://github.com/pharo-project/pharo-core
  Commit: 2787bdf9b46b65b1c9b833f2a44bc898cfb6983d
      https://github.com/pharo-project/pharo-core/commit/2787bdf9b46b65b1c9b833f2a44bc898cfb6983d
  Author: Jenkins Build Server <[hidden email]>
  Date:   2015-07-03 (Fri, 03 Jul 2015)

  Changed paths:
    R Reflectivity-Tests.package/HookGeneratorTest.class/README.md
    R Reflectivity-Tests.package/HookGeneratorTest.class/definition.st
    R Reflectivity-Tests.package/HookGeneratorTest.class/instance/tests - hooks/testConditionWithArguments.st
    R Reflectivity-Tests.package/HookGeneratorTest.class/instance/tests - hooks/testHaltHook.st
    R Reflectivity-Tests.package/HookGeneratorTest.class/instance/tests - hooks/testHookWithArguments.st
    R Reflectivity-Tests.package/HookGeneratorTest.class/instance/tests - hooks/testHookWithVirtualMeta.st
    R Reflectivity.package/HookGenerator.class/README.md
    R Reflectivity.package/HookGenerator.class/class/instance creation/node_link_.st
    R Reflectivity.package/HookGenerator.class/definition.st
    R Reflectivity.package/HookGenerator.class/instance/accessing/link.st
    R Reflectivity.package/HookGenerator.class/instance/accessing/link_.st
    R Reflectivity.package/HookGenerator.class/instance/accessing/plugins.st
    R Reflectivity.package/HookGenerator.class/instance/accessing/table.st
    R Reflectivity.package/HookGenerator.class/instance/ast manipulation/encloseInBlock_.st
    R Reflectivity.package/HookGenerator.class/instance/ast manipulation/selectorForNode.st
    R Reflectivity.package/HookGenerator.class/instance/ast manipulation/wrapCondition_.st
    R Reflectivity.package/HookGenerator.class/instance/ast manipulation/wrapInContext_.st
    R Reflectivity.package/HookGenerator.class/instance/initialize/node_link_.st
    R Reflectivity.package/HookGenerator.class/instance/initialize/setupTable.st
    R Reflectivity.package/HookGenerator.class/instance/result/hook.st
    R Reflectivity.package/HookGenerator.class/instance/result/preamble.st
    A Reflectivity.package/HookGenerator2.class/README.md
    A Reflectivity.package/HookGenerator2.class/class/instance creation/node_.st
    A Reflectivity.package/HookGenerator2.class/definition.st
    A Reflectivity.package/HookGenerator2.class/instance/accessing/links.st
    A Reflectivity.package/HookGenerator2.class/instance/accessing/plugins.st
    A Reflectivity.package/HookGenerator2.class/instance/ast manipulation/encloseInBlock_.st
    A Reflectivity.package/HookGenerator2.class/instance/ast manipulation/selectorForNode.st
    A Reflectivity.package/HookGenerator2.class/instance/ast manipulation/wrapCondition_link_.st
    A Reflectivity.package/HookGenerator2.class/instance/ast manipulation/wrapInContext_link_.st
    A Reflectivity.package/HookGenerator2.class/instance/initialize/node_links_.st
    A Reflectivity.package/HookGenerator2.class/instance/private/genReification_for_.st
    A Reflectivity.package/HookGenerator2.class/instance/private/hookFor_.st
    A Reflectivity.package/HookGenerator2.class/instance/results/afterHooks.st
    A Reflectivity.package/HookGenerator2.class/instance/results/beforeHooks.st
    A Reflectivity.package/HookGenerator2.class/instance/results/hook.st
    A Reflectivity.package/HookGenerator2.class/instance/results/insteadHooks.st
    A Reflectivity.package/HookGenerator2.class/instance/results/preamble.st
    R Reflectivity.package/MetaLink.class/instance/ast/hookOn_.st
    R Reflectivity.package/MetaLink.class/instance/ast/preambleOn_.st
    M Reflectivity.package/RFASTTranslator.class/instance/reflectivity/emitMetaLinkInstead_.st
    M Reflectivity.package/RFSemanticAnalyzer.class/instance/visiting/analyseForLinks_.st
    M Reflectivity.package/extension/RBProgramNode/instance/afterHooks.st
    M Reflectivity.package/extension/RBProgramNode/instance/beforeHooks.st
    M Reflectivity.package/extension/RBProgramNode/instance/preambles.st
    R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50150.st
    A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50151.st
    R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50150.st
    A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50151.st
    M ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st

  Log Message:
  -----------
  50151
15891 Reflectivity: New HookGenerator
        https://pharo.fogbugz.com/f/cases/15891

15871 enable ProcessList
        https://pharo.fogbugz.com/f/cases/15871

http://files.pharo.org/image/50/50151.zip

> 15871 enable ProcessList
> https://pharo.fogbugz.com/f/cases/15871
>
> http://files.pharo.org/image/50/50151.zip

Hi Marcus,

there must be a reason to switch from a LinkedList to a special
ProcessList class that is bound to VM - still the bug tracker and the class
comments do not tell much about the "why".

Anyone able to ellaborate why this is necessary/what's the plan behind that?

Thanks
T.

The reason is that the ProcessList is very special: some methods are carefully written to
not have any intererrupt point, for example (not using messages sends).

In the past we had serious side effects from people improving LinkedList… in the end
to the point that nobody dares to change it anymore.

Having two solved that:

        -> ProcessList will be minmal, unchanging, for the VM
        -> LinkedList will be a linked list for people to use and that we can improve and
             change.


        Marcus