[Cuis-dev] findSelector fixes
Hernan Wilkinson
hernan.wilkinson at 10pines.com
Tue Dec 29 08:34:42 PST 2020
Hi,
String>>#findSelector failed when the string has an unmatched double
quote, for example:
'"self m:' findSelector --> signals error
Also, it did not support when looking inside a comment, which is very
common to do when a method has an example as a comment.
Attached is a fix for that case and also a test for it.
@Juan Vuletich <juan at jvuletich.org> Do you know why findSelector was doing
this check?
"and: [ (self at: colonIndex - 1) isLetter ]"
Why is it checking for isLetter? and btw, the check was wrong because it
was sending #at: to self instead of to sel.
I removed that check because I could not find a reason for it
Cheers!
Hernan.
--
*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/20201229/4845541c/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: UnsavedChangesTo-BaseImageTests-HAW.003.cs.st
Type: application/octet-stream
Size: 419 bytes
Desc: not available
URL: <http://lists.cuis.st/mailman/archives/cuis-dev/attachments/20201229/4845541c/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 4505-findSelectorFix-HernanWilkinson-2020Dec29-12h51m-HAW.001.cs.st
Type: application/octet-stream
Size: 2489 bytes
Desc: not available
URL: <http://lists.cuis.st/mailman/archives/cuis-dev/attachments/20201229/4845541c/attachment-0001.obj>
More information about the Cuis-dev
mailing list