Hi Stef, Object>>assert: also works with Boolean args, just like TestCase>>assert: Does it make any difference? I cannot find the testSplit method you mention. I am looking at RubyShards-on.6.mcz Am I looking in the wrong place? I think Keith has written some more tests (according to Mantis). Are they part of Keith's tests? I think Damien Pollet (cdlm) write splitBy: I suggest we rename the package SplitJoin. But I want to know who is charge first -- is it Keith, or Damien, or Stef (or me)? I do not want to step on anyone's toes. Cheers, - on On Apr 5, 2009, at 10:33, Stéphane Ducasse wrote:
I checked the code and it looks nice
Now I found
split: regexString self assert: regexString size > 0 description: 'Cannot split on empty regex'. ^regexString asRegex split: self
and assert: expects a block. for the test
testSplit self assert: (eg split: 'the') size = 4. self assert: (eg split: 't\w +e') size = 5. self assert: (eg split: 'hello') size = 1. it would be nice to have also an assert showing the result and not just its size
then I have a question what splitBy: returns? does it return a collection that is the same kind of the receiver? Why do you hard code an OrderedCollection in splitBySubCollection: aSplitter. self species would do it too? Syef
On Apr 5, 2009, at 9:51 AM, Oscar Nierstrasz wrote:
Hi Folks,
What's the status of split and join?
http://bugs.squeak.org/view.php?id=4874
I need them for a project and since they are not there yet, I have gone back to RubyShards.
http://squeaksource.com/RubyShards/
Keith, do you think this can be integrated into Pharo already, or does it need some work still?
(I think our implementation is not particularly efficient, but it should be serviceable.)
Cheers, - on
_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project