The Trunk: Morphic-eem.1597.mcz

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

The Trunk: Morphic-eem.1597.mcz

commits-2
Eliot Miranda uploaded a new version of Morphic to project The Trunk:
http://source.squeak.org/trunk/Morphic-eem.1597.mcz

==================== Summary ====================

Name: Morphic-eem.1597
Author: eem
Time: 30 November 2019, 7:05:24.872005 pm
UUID: 068e0786-10ca-433e-9ec1-7c5cc2a15942
Ancestors: Morphic-eem.1596

Fix regression from hurried programming. Apologies!

=============== Diff against Morphic-eem.1596 ===============

Item was added:
+ ----- Method: TextEditor>>selectedLiteral (in category 'menu messages') -----
+ selectedLiteral
+ "Try to make a literal out of the current text selection"
+ | tokens |
+ tokens := Scanner new typedScanTokens: self selection string.
+ ^tokens isEmpty ifFalse: [tokens first]!

Item was changed:
  ----- Method: TextEditor>>selectedSelector (in category 'menu messages') -----
  selectedSelector
  "Try to make a selector out of the current text selection"
+ ^self selection string findSelector!
- | tokens |
- tokens := Scanner new typedScanTokens: self selection string.
- ^tokens isEmpty ifFalse: [tokens first]!


Reply | Threaded
Open this post in threaded view
|

Re: The Trunk: Morphic-eem.1597.mcz

marcel.taeumel
Hi Eliot,

can you give an example string that would match #selectedLiteral but not #selectedSelector? Once I can understand this change, I can write some tests for it. :-)

Best,
Marcel

Am 01.12.2019 04:05:43 schrieb [hidden email] <[hidden email]>:

Eliot Miranda uploaded a new version of Morphic to project The Trunk:
http://source.squeak.org/trunk/Morphic-eem.1597.mcz

==================== Summary ====================

Name: Morphic-eem.1597
Author: eem
Time: 30 November 2019, 7:05:24.872005 pm
UUID: 068e0786-10ca-433e-9ec1-7c5cc2a15942
Ancestors: Morphic-eem.1596

Fix regression from hurried programming. Apologies!

=============== Diff against Morphic-eem.1596 ===============

Item was added:
+ ----- Method: TextEditor>>selectedLiteral (in category 'menu messages') -----
+ selectedLiteral
+ "Try to make a literal out of the current text selection"
+ | tokens |
+ tokens := Scanner new typedScanTokens: self selection string.
+ ^tokens isEmpty ifFalse: [tokens first]!

Item was changed:
----- Method: TextEditor>>selectedSelector (in category 'menu messages') -----
selectedSelector
"Try to make a selector out of the current text selection"
+ ^self selection string findSelector!
- | tokens |
- tokens := Scanner new typedScanTokens: self selection string.
- ^tokens isEmpty ifFalse: [tokens first]!




Reply | Threaded
Open this post in threaded view
|

Re: The Trunk: Morphic-eem.1597.mcz

marcel.taeumel
Got it! :-o) I added tests for it (CollectionsTests-mt.328).

Best,
Marcel

Am 02.12.2019 09:31:32 schrieb Marcel Taeumel <[hidden email]>:

Hi Eliot,

can you give an example string that would match #selectedLiteral but not #selectedSelector? Once I can understand this change, I can write some tests for it. :-)

Best,
Marcel

Am 01.12.2019 04:05:43 schrieb [hidden email] <[hidden email]>:

Eliot Miranda uploaded a new version of Morphic to project The Trunk:
http://source.squeak.org/trunk/Morphic-eem.1597.mcz

==================== Summary ====================

Name: Morphic-eem.1597
Author: eem
Time: 30 November 2019, 7:05:24.872005 pm
UUID: 068e0786-10ca-433e-9ec1-7c5cc2a15942
Ancestors: Morphic-eem.1596

Fix regression from hurried programming. Apologies!

=============== Diff against Morphic-eem.1596 ===============

Item was added:
+ ----- Method: TextEditor>>selectedLiteral (in category 'menu messages') -----
+ selectedLiteral
+ "Try to make a literal out of the current text selection"
+ | tokens |
+ tokens := Scanner new typedScanTokens: self selection string.
+ ^tokens isEmpty ifFalse: [tokens first]!

Item was changed:
----- Method: TextEditor>>selectedSelector (in category 'menu messages') -----
selectedSelector
"Try to make a selector out of the current text selection"
+ ^self selection string findSelector!
- | tokens |
- tokens := Scanner new typedScanTokens: self selection string.
- ^tokens isEmpty ifFalse: [tokens first]!