[Cuis-dev] Refactoring fixes and enhancements
Juan Vuletich
juan at cuis.st
Wed Sep 30 11:49:04 PDT 2026
They are all now at GitHub.
Thanks!
On 2026-09-29 10:27 AM, Hernán Wilkinson via Cuis-dev wrote:
> Hi,
> Attached is a series of cs and tests for the following refactoring
> fixes and enhancements:
>
>
> Inline Method
>
> * *Two sends in one statement.* Inlining failed with overlapping
> replacements. Now only the send itself is replaced.
> * *Evaluation order.*A method with several statements is inlined as
> `|[ ... ] value`| when moving its statements before the sender's
> statement would change the order: several sends in the statement,
> or something other than literals evaluated before the send. The
> block keeps original method source format.
> * *Temporaries of a block.* When inlining a method as block, they
> are declared inside the block and keep their name, unless the
> containing scope already has it.
>
>
> Extract to Temporary and Add Parameter
>
> * Both added names to |Undeclared| when parsing code. They now parse
> ignoring undeclared names.
>
>
> Tests
>
> * *Flaky tests.* Five tests asserted on the global |Undeclared|.
> They now use |RefactoringTest>>should:notLeaveUndeclared:|, which
> removes the name first.
> * *Tests that need an undeclared name *were fixed
> * *New tests.* for |InlineMethod|, |ExtractToTemporary and|
> |AddParameter|
>
>
> Renames
>
> * |isReceiverOfPieceTemporary:| and the other names with "piece"
> that I had written now say |CodeToExtract|.
>
>
> They are small changes (some not so small :) ), but I decided to send
> them all together to avoid bothering with this :-)
>
> Cheers!
> Hernan.
>
> --
> *Hernán Wilkinson
> Agile 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
>
--
Juan Vuletich
www.cuis.st
github.com/jvuletich
researchgate.net/profile/Juan-Vuletich
independent.academia.edu/JuanVuletich
patents.justia.com/inventor/juan-manuel-vuletich
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.cuis.st/mailman/archives/cuis-dev/attachments/20260930/941edab0/attachment.htm>
More information about the Cuis-dev
mailing list