[Cuis-dev] [ChangeSet] source ranges for cascade parse nodes

Hernan Wilkinson hernan.wilkinson at 10pines.com
Fri Apr 17 19:19:40 PDT 2020


Thank you!

It is on github now.

Cheers!
Hernan.

On Tue, Apr 14, 2020 at 9:05 PM Nahuel Garbezza via Cuis-dev <
cuis-dev at lists.cuis.st> wrote:

> Hi!
>
> Attached are two changesets to add support for source range tracking of
> cascade nodes. This is particularly useful for refactorings like the
> extract method or the extract temporary variable.
>
> Example: if you have this expression:
>
> 3 factorial; yourself
>
> now we are not reporting source range for the cascade, just for each
> individual message. From now on, we'll be able to get:
>
> * "raw" source range: 3 [factorial; yourself] (original source range
> definition, used for debugging)
> * "complete" source range: [3 factorial; yourself] (definition added for
> refactoring purposes)
>
> The reason I made two changesets is because I wanted to extract a method
> in the Parser but I cannot do it in the same changeset, because it could
> break the image.
>
> * The first changeset includes the new feature itself
> * The second changeset is just cleanup to make the parser #cascade method
> more readable.
>
> This change allows us to automatically support a new extract method case,
> which is extracting expressions enclosing cascade message sends (yay!).
> Also attached a test for it.
>
> Thanks,
> Nahuel
> --
> Cuis-dev mailing list
> Cuis-dev at lists.cuis.st
> https://lists.cuis.st/mailman/listinfo/cuis-dev
>


-- 

*Hernán WilkinsonAgile Software Development, Teaching & Coaching*
*Phone: +54-011*-4893-2057
*Twitter: @HernanWilkinson*
*site: http://www.10Pines.com <http://www.10pines.com/>*
Address: Alem 896, Floor 6, Buenos Aires, Argentina
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.cuis.st/mailman/archives/cuis-dev/attachments/20200417/a7bc74a5/attachment.htm>


More information about the Cuis-dev mailing list