Vincent Blondeau

Calypso should use not deprecated methods in Pharo 8

1

``` ClyClassWithUndeclares>>method2WithUndeclares (undeclaredStubInstVar2 is Undeclared) ClyClassWithUndeclares>>method1WithUndeclares (undeclaredStubInstVar1 is Undeclared) The method FTTableMorph>>#basicHighlightRowIndexes: called from ClyHighlightingSelection>>#setUpSelectedRows:in:silently: has been deprecated. Use #basicHighlightIndexes: instead The method FTTableMorph>>#basicSelectRowIndexes: called from ClyDataSourceSelection>>#setUpSelectedRows:in:silently: has been deprecated....

Comment text cut

8

It should be better to see the diff now

See tests in commit 3ccfeb3890e220be683249950de7d5cf7e8f23ac

It seems that the accesses from the methods in innerclasses are not fully created and the one created has its "to" link pointing to nil. Method example in a class:...

E.g.: package com.foo; import org.junit.Test; public class FooTest { @Test void test(){ String foo = new String(); } } Does not create a reference from FooTest::test() to String

Because the IVariableBinding on the field access is null. See createAccessFromVariableBinding(IVariableBinding binding)

See InJavaImporter>>ensureAttributeForFragment(VariableDeclarationFragment fragment, FieldDeclaration field) -> if (binding == null) Test to come in PR

See #13