And now the question for the monday morning development effort is: what more semantic warnings of the compiler can we reveal like this? :-)

On Nov 28, 2015, at 19:06, Cyril Ferlicot <cyril.ferlicot@gmail.com> wrote:

This is really cool ! :)

One more step to get a nice IDE!

Thank you for your efforts :)

On Saturday, November 28, 2015, Marcus Denker <marcus.denker@inria.fr> wrote:

Hi,

One result of Friday���s Pharo Sprint in Chile: Use the nice icons of the new text editor

to display some of the semantic errors nicely (instead of breaking flow with a intrusive dialog):

This was done by Johan based on the prototype Miguel did in September:

1) unused temps:

<unused.png>

(the ���drag the crosshair���r is an artefact of the snapshot tool)

2) temps used that are not initialised:

<uninit.png>

In a next step, we need to integrate that with the QA tool��� no need to have this reported

twice.

What is nice is that this simplified the Compiler: instead of having an Exception and raising

it, we now just annotate the AST:

unusedVariable: variableNode

variableNode propertyAt: #semanticWarning put: 'unused variable���

(this is very simple for now, with a node just being able to have one kind of semanticWarning).

SemanticWarningIconStyler is what adds the icon. It has 4 methods with one line each.

Marcus

--

Cheers

Cyril Ferlicot

---> Save our in-boxes!��http://emailcharter.org <---

Johan Fabry �� - ����http://pleiad.cl/~jfabry
PLEIAD and RyCh labs ��- ��Computer Science��Department (DCC) ��- ��University of Chile