Modification on CollectionsExtensions
Hello, Fixing some bugs in Traits we realized that it would be good idea to use CollectionsExtensions package (it has some useful functions), also it would be great to include CollectionsExtensions in Pharo 3.0. One "problem" we found is that CollectionsExtensions is depending on a Nile Package (that's not a desired feature). In the method Collection>>flatCollect: there is a line refering to a nile function "nsWriteStream". If i change "nsWriteStream" for "writeStream" the dependency disapears. I also run all the tests and they are working, so i was wondering if it would be possible to commit the change. Thanks.
On 11 June 2013 14:18, Sebastian Tleye <stleye@gmail.com> wrote:
Hello,
Fixing some bugs in Traits we realized that it would be good idea to use CollectionsExtensions package (it has some useful functions), also it would be great to include CollectionsExtensions in Pharo 3.0.
Is this the HPI library? I seem to recall TraitClasses [1] using it. frank [1] https://github.com/lauritzthamsen/TraitClasses
One "problem" we found is that CollectionsExtensions is depending on a Nile Package (that's not a desired feature). In the method Collection>>flatCollect: there is a line refering to a nile function "nsWriteStream".
If i change "nsWriteStream" for "writeStream" the dependency disapears.
I also run all the tests and they are working, so i was wondering if it would be possible to commit the change.
Thanks.
On Jun 11, 2013, at 4:24 PM, Frank Shearar <frank.shearar@gmail.com> wrote:
On 11 June 2013 14:18, Sebastian Tleye <stleye@gmail.com> wrote:
Hello,
Fixing some bugs in Traits we realized that it would be good idea to use CollectionsExtensions package (it has some useful functions), also it would be great to include CollectionsExtensions in Pharo 3.0.
Is this the HPI library? I seem to recall TraitClasses [1] using it.
CollectionExtension is from Moose. the Trait based Stream refactoring was done some years ago but has not been used or maintained. Marcus
If they are not different so a better way would be replace #flatenCollect: by #gather: but anyway, moose people should decide and commit the change since i am not a moose developer. 2013/6/11 Marcus Denker <marcus.denker@inria.fr>
On Jun 11, 2013, at 4:24 PM, Frank Shearar <frank.shearar@gmail.com> wrote:
On 11 June 2013 14:18, Sebastian Tleye <stleye@gmail.com> wrote:
Hello,
Fixing some bugs in Traits we realized that it would be good idea to use CollectionsExtensions package (it has some useful functions), also it would be great to include CollectionsExtensions in Pharo 3.0.
Is this the HPI library? I seem to recall TraitClasses [1] using it.
CollectionExtension is from Moose.
the Trait based Stream refactoring was done some years ago but has not been used or maintained.
Marcus
On Jun 11, 2013, at 4:54 PM, Sebastian Tleye <stleye@gmail.com> wrote:
If they are not different so a better way would be replace #flatenCollect: by #gather: but anyway, moose people should decide and commit the change since i am not a moose developer.
The moose people should have taken the lead to get the extension integrated a *long* time ago⦠nothing happens by itself. Marcus
2013/6/11 Marcus Denker <marcus.denker@inria.fr>
On Jun 11, 2013, at 4:24 PM, Frank Shearar <frank.shearar@gmail.com> wrote:
On 11 June 2013 14:18, Sebastian Tleye <stleye@gmail.com> wrote:
Hello,
Fixing some bugs in Traits we realized that it would be good idea to use CollectionsExtensions package (it has some useful functions), also it would be great to include CollectionsExtensions in Pharo 3.0.
Is this the HPI library? I seem to recall TraitClasses [1] using it.
CollectionExtension is from Moose.
the Trait based Stream refactoring was done some years ago but has not been used or maintained.
Marcus
gather: sucks as a name. flatCollect: is much better so do not remove it. On Jun 11, 2013, at 4:53 PM, Sebastian Tleye <stleye@gmail.com> wrote:
If they are not different so a better way would be replace #flatenCollect: by #gather: but anyway, moose people should decide and commit the change since i am not a moose developer.
2013/6/11 Marcus Denker <marcus.denker@inria.fr>
On Jun 11, 2013, at 4:24 PM, Frank Shearar <frank.shearar@gmail.com> wrote:
On 11 June 2013 14:18, Sebastian Tleye <stleye@gmail.com> wrote:
Hello,
Fixing some bugs in Traits we realized that it would be good idea to use CollectionsExtensions package (it has some useful functions), also it would be great to include CollectionsExtensions in Pharo 3.0.
Is this the HPI library? I seem to recall TraitClasses [1] using it.
CollectionExtension is from Moose.
the Trait based Stream refactoring was done some years ago but has not been used or maintained.
Marcus
I think the simplest solution is to keep both and that flatCollect directly call gather: If you agree with that, I will correct the moose extension package 2013/6/11 Stéphane Ducasse <stephane.ducasse@inria.fr>
gather: sucks as a name. flatCollect: is much better so do not remove it.
On Jun 11, 2013, at 4:53 PM, Sebastian Tleye <stleye@gmail.com> wrote:
If they are not different so a better way would be replace #flatenCollect: by #gather: but anyway, moose people should decide and commit the change since i am not a moose developer.
2013/6/11 Marcus Denker <marcus.denker@inria.fr>
On Jun 11, 2013, at 4:24 PM, Frank Shearar <frank.shearar@gmail.com> wrote:
On 11 June 2013 14:18, Sebastian Tleye <stleye@gmail.com> wrote:
Hello,
Fixing some bugs in Traits we realized that it would be good idea to use CollectionsExtensions package (it has some useful functions), also it would be great to include CollectionsExtensions in Pharo 3.0.
Is this the HPI library? I seem to recall TraitClasses [1] using it.
CollectionExtension is from Moose.
the Trait based Stream refactoring was done some years ago but has not been used or maintained.
Marcus
-- *Guillaume Larcheveque*
+1 2013/6/12 Guillaume Larcheveque <guillaume.larcheveque@gmail.com>
I think the simplest solution is to keep both and that flatCollect directly call gather:
If you agree with that, I will correct the moose extension package
2013/6/11 Stéphane Ducasse <stephane.ducasse@inria.fr>
gather: sucks as a name. flatCollect: is much better so do not remove it.
On Jun 11, 2013, at 4:53 PM, Sebastian Tleye <stleye@gmail.com> wrote:
If they are not different so a better way would be replace #flatenCollect: by #gather: but anyway, moose people should decide and commit the change since i am not a moose developer.
2013/6/11 Marcus Denker <marcus.denker@inria.fr>
On Jun 11, 2013, at 4:24 PM, Frank Shearar <frank.shearar@gmail.com> wrote:
On 11 June 2013 14:18, Sebastian Tleye <stleye@gmail.com> wrote:
Hello,
Fixing some bugs in Traits we realized that it would be good idea to use CollectionsExtensions package (it has some useful functions), also it would be great to include CollectionsExtensions in Pharo 3.0.
Is this the HPI library? I seem to recall TraitClasses [1] using it.
CollectionExtension is from Moose.
the Trait based Stream refactoring was done some years ago but has not been used or maintained.
Marcus
-- *Guillaume Larcheveque*
Why not the inverse. They are much more users of flatCollect: then gather: and again gather: sucks. It does not convey that this is a mapcan ie happening the results of each iteration into the So flatCollect: is 10 times more explicit and better. you flatten the results of the collect: Stef
I think the simplest solution is to keep both and that flatCollect directly call gather:
If you agree with that, I will correct the moose extension package
2013/6/11 Stéphane Ducasse <stephane.ducasse@inria.fr> gather: sucks as a name. flatCollect: is much better so do not remove it.
On Jun 11, 2013, at 4:53 PM, Sebastian Tleye <stleye@gmail.com> wrote:
If they are not different so a better way would be replace #flatenCollect: by #gather: but anyway, moose people should decide and commit the change since i am not a moose developer.
2013/6/11 Marcus Denker <marcus.denker@inria.fr>
On Jun 11, 2013, at 4:24 PM, Frank Shearar <frank.shearar@gmail.com> wrote:
On 11 June 2013 14:18, Sebastian Tleye <stleye@gmail.com> wrote:
Hello,
Fixing some bugs in Traits we realized that it would be good idea to use CollectionsExtensions package (it has some useful functions), also it would be great to include CollectionsExtensions in Pharo 3.0.
Is this the HPI library? I seem to recall TraitClasses [1] using it.
CollectionExtension is from Moose.
the Trait based Stream refactoring was done some years ago but has not been used or maintained.
Marcus
-- Guillaume Larcheveque
As far as i know the implementation of gather: is faster than the implementation of flatCollect: 2013/6/12 Stéphane Ducasse <stephane.ducasse@inria.fr>
Why not the inverse. They are much more users of flatCollect: then gather:
and again gather: sucks. It does not convey that this is a mapcan ie happening the results of each iteration into the
So flatCollect: is 10 times more explicit and better. you flatten the results of the collect:
Stef
I think the simplest solution is to keep both and that flatCollect directly call gather:
If you agree with that, I will correct the moose extension package
2013/6/11 Stéphane Ducasse <stephane.ducasse@inria.fr>
gather: sucks as a name. flatCollect: is much better so do not remove it.
On Jun 11, 2013, at 4:53 PM, Sebastian Tleye <stleye@gmail.com> wrote:
If they are not different so a better way would be replace #flatenCollect: by #gather: but anyway, moose people should decide and commit the change since i am not a moose developer.
2013/6/11 Marcus Denker <marcus.denker@inria.fr>
On Jun 11, 2013, at 4:24 PM, Frank Shearar <frank.shearar@gmail.com> wrote:
On 11 June 2013 14:18, Sebastian Tleye <stleye@gmail.com> wrote:
Hello,
Fixing some bugs in Traits we realized that it would be good idea to use CollectionsExtensions package (it has some useful functions), also it would be great to include CollectionsExtensions in Pharo 3.0.
Is this the HPI library? I seem to recall TraitClasses [1] using it.
CollectionExtension is from Moose.
the Trait based Stream refactoring was done some years ago but has not been used or maintained.
Marcus
-- *Guillaume Larcheveque*
#gather: directly applies the stream fusion technique as its implementation, so is faster than the current implementation of #flatCollect: Stef's saying "why not move #gather:'s implemenation to #flatCollect:, and make the old #gather: call the new #flatCollect:" frank On 12 June 2013 10:33, Sebastian Tleye <stleye@gmail.com> wrote:
As far as i know the implementation of gather: is faster than the implementation of flatCollect:
2013/6/12 Stéphane Ducasse <stephane.ducasse@inria.fr>
Why not the inverse. They are much more users of flatCollect: then gather:
and again gather: sucks. It does not convey that this is a mapcan ie happening the results of each iteration into the
So flatCollect: is 10 times more explicit and better. you flatten the results of the collect:
Stef
I think the simplest solution is to keep both and that flatCollect directly call gather:
If you agree with that, I will correct the moose extension package
2013/6/11 Stéphane Ducasse <stephane.ducasse@inria.fr>
gather: sucks as a name. flatCollect: is much better so do not remove it.
On Jun 11, 2013, at 4:53 PM, Sebastian Tleye <stleye@gmail.com> wrote:
If they are not different so a better way would be replace #flatenCollect: by #gather: but anyway, moose people should decide and commit the change since i am not a moose developer.
2013/6/11 Marcus Denker <marcus.denker@inria.fr>
On Jun 11, 2013, at 4:24 PM, Frank Shearar <frank.shearar@gmail.com> wrote:
On 11 June 2013 14:18, Sebastian Tleye <stleye@gmail.com> wrote:
Hello,
Fixing some bugs in Traits we realized that it would be good idea to use CollectionsExtensions package (it has some useful functions), also it would be great to include CollectionsExtensions in Pharo 3.0.
Is this the HPI library? I seem to recall TraitClasses [1] using it.
CollectionExtension is from Moose.
the Trait based Stream refactoring was done some years ago but has not been used or maintained.
Marcus
-- Guillaume Larcheveque
On 12 June 2013 10:54, Frank Shearar <frank.shearar@gmail.com> wrote:
#gather: directly applies the stream fusion technique as its implementation, so is faster than the current implementation of #flatCollect:
Stef's saying "why not move #gather:'s implemenation to #flatCollect:, and make the old #gather: call the new #flatCollect:"
Er, and that would mean moving #flatCollect: out of CollectionsExtensions into the base image.
frank
On 12 June 2013 10:33, Sebastian Tleye <stleye@gmail.com> wrote:
As far as i know the implementation of gather: is faster than the implementation of flatCollect:
2013/6/12 Stéphane Ducasse <stephane.ducasse@inria.fr>
Why not the inverse. They are much more users of flatCollect: then gather:
and again gather: sucks. It does not convey that this is a mapcan ie happening the results of each iteration into the
So flatCollect: is 10 times more explicit and better. you flatten the results of the collect:
Stef
I think the simplest solution is to keep both and that flatCollect directly call gather:
If you agree with that, I will correct the moose extension package
2013/6/11 Stéphane Ducasse <stephane.ducasse@inria.fr>
gather: sucks as a name. flatCollect: is much better so do not remove it.
On Jun 11, 2013, at 4:53 PM, Sebastian Tleye <stleye@gmail.com> wrote:
If they are not different so a better way would be replace #flatenCollect: by #gather: but anyway, moose people should decide and commit the change since i am not a moose developer.
2013/6/11 Marcus Denker <marcus.denker@inria.fr>
On Jun 11, 2013, at 4:24 PM, Frank Shearar <frank.shearar@gmail.com> wrote:
On 11 June 2013 14:18, Sebastian Tleye <stleye@gmail.com> wrote:
Hello,
Fixing some bugs in Traits we realized that it would be good idea to use CollectionsExtensions package (it has some useful functions), also it would be great to include CollectionsExtensions in Pharo 3.0.
Is this the HPI library? I seem to recall TraitClasses [1] using it.
CollectionExtension is from Moose.
the Trait based Stream refactoring was done some years ago but has not been used or maintained.
Marcus
-- Guillaume Larcheveque
+ 1 guys a name and an implementation body are two separate entities.
#gather: directly applies the stream fusion technique as its implementation, so is faster than the current implementation of #flatCollect:
Stef's saying "why not move #gather:'s implemenation to #flatCollect:, and make the old #gather: call the new #flatCollect:"
Er, and that would mean moving #flatCollect: out of CollectionsExtensions into the base image.
frank
On 12 June 2013 10:33, Sebastian Tleye <stleye@gmail.com> wrote:
As far as i know the implementation of gather: is faster than the implementation of flatCollect:
2013/6/12 Stéphane Ducasse <stephane.ducasse@inria.fr>
Why not the inverse. They are much more users of flatCollect: then gather:
and again gather: sucks. It does not convey that this is a mapcan ie happening the results of each iteration into the
So flatCollect: is 10 times more explicit and better. you flatten the results of the collect:
Stef
I think the simplest solution is to keep both and that flatCollect directly call gather:
If you agree with that, I will correct the moose extension package
2013/6/11 Stéphane Ducasse <stephane.ducasse@inria.fr>
gather: sucks as a name. flatCollect: is much better so do not remove it.
On Jun 11, 2013, at 4:53 PM, Sebastian Tleye <stleye@gmail.com> wrote:
If they are not different so a better way would be replace #flatenCollect: by #gather: but anyway, moose people should decide and commit the change since i am not a moose developer.
2013/6/11 Marcus Denker <marcus.denker@inria.fr>
On Jun 11, 2013, at 4:24 PM, Frank Shearar <frank.shearar@gmail.com> wrote:
On 11 June 2013 14:18, Sebastian Tleye <stleye@gmail.com> wrote:
Hello,
Fixing some bugs in Traits we realized that it would be good idea to use CollectionsExtensions package (it has some useful functions), also it would be great to include CollectionsExtensions in Pharo 3.0.
Is this the HPI library? I seem to recall TraitClasses [1] using it.
CollectionExtension is from Moose.
the Trait based Stream refactoring was done some years ago but has not been used or maintained.
Marcus
-- Guillaume Larcheveque
ok, i understood, Discussing with Marcus we believe that it would be better to keep Collecition>>gather: and add Collection>>flatCollect: (with the same implementation) and put a comment in gather: saying that is kept for compatibility purposes. flatCollect: will be included in Pharo and will be removed from CollectionExtensions. 2013/6/12 Frank Shearar <frank.shearar@gmail.com>
On 12 June 2013 10:54, Frank Shearar <frank.shearar@gmail.com> wrote:
#gather: directly applies the stream fusion technique as its implementation, so is faster than the current implementation of #flatCollect:
Stef's saying "why not move #gather:'s implemenation to #flatCollect:, and make the old #gather: call the new #flatCollect:"
Er, and that would mean moving #flatCollect: out of CollectionsExtensions into the base image.
frank
On 12 June 2013 10:33, Sebastian Tleye <stleye@gmail.com> wrote:
As far as i know the implementation of gather: is faster than the implementation of flatCollect:
2013/6/12 Stéphane Ducasse <stephane.ducasse@inria.fr>
Why not the inverse. They are much more users of flatCollect: then gather:
and again gather: sucks. It does not convey that this is a mapcan ie happening the results of each iteration into the
So flatCollect: is 10 times more explicit and better. you flatten the results of the collect:
Stef
I think the simplest solution is to keep both and that flatCollect directly call gather:
If you agree with that, I will correct the moose extension package
2013/6/11 Stéphane Ducasse <stephane.ducasse@inria.fr>
gather: sucks as a name. flatCollect: is much better so do not remove it.
On Jun 11, 2013, at 4:53 PM, Sebastian Tleye <stleye@gmail.com>
wrote:
If they are not different so a better way would be replace #flatenCollect: by #gather: but anyway, moose people should decide and commit the change since i
am
not a moose developer.
2013/6/11 Marcus Denker <marcus.denker@inria.fr>
On Jun 11, 2013, at 4:24 PM, Frank Shearar <frank.shearar@gmail.com> wrote:
On 11 June 2013 14:18, Sebastian Tleye <stleye@gmail.com> wrote:
Hello,
Fixing some bugs in Traits we realized that it would be good idea
to
use CollectionsExtensions package (it has some useful functions), also it would be great to include CollectionsExtensions in Pharo 3.0.
Is this the HPI library? I seem to recall TraitClasses [1] using it.
CollectionExtension is from Moose.
the Trait based Stream refactoring was done some years ago but has not been used or maintained.
Marcus
-- Guillaume Larcheveque
sorry, gather will call flatCollect: is not the same code. 2013/6/12 Sebastian Tleye <stleye@gmail.com>
ok, i understood,
Discussing with Marcus we believe that it would be better to keep Collecition>>gather: and add Collection>>flatCollect: (with the same implementation) and put a comment in gather: saying that is kept for compatibility purposes.
flatCollect: will be included in Pharo and will be removed from CollectionExtensions.
2013/6/12 Frank Shearar <frank.shearar@gmail.com>
On 12 June 2013 10:54, Frank Shearar <frank.shearar@gmail.com> wrote:
#gather: directly applies the stream fusion technique as its implementation, so is faster than the current implementation of #flatCollect:
Stef's saying "why not move #gather:'s implemenation to #flatCollect:, and make the old #gather: call the new #flatCollect:"
Er, and that would mean moving #flatCollect: out of CollectionsExtensions into the base image.
frank
On 12 June 2013 10:33, Sebastian Tleye <stleye@gmail.com> wrote:
As far as i know the implementation of gather: is faster than the implementation of flatCollect:
2013/6/12 Stéphane Ducasse <stephane.ducasse@inria.fr>
Why not the inverse. They are much more users of flatCollect: then gather:
and again gather: sucks. It does not convey that this is a mapcan ie happening the results of each iteration into the
So flatCollect: is 10 times more explicit and better. you flatten the results of the collect:
Stef
I think the simplest solution is to keep both and that flatCollect directly call gather:
If you agree with that, I will correct the moose extension package
2013/6/11 Stéphane Ducasse <stephane.ducasse@inria.fr>
gather: sucks as a name. flatCollect: is much better so do not remove it.
On Jun 11, 2013, at 4:53 PM, Sebastian Tleye <stleye@gmail.com>
wrote:
If they are not different so a better way would be replace #flatenCollect: by #gather: but anyway, moose people should decide and commit the change since i
am
not a moose developer.
2013/6/11 Marcus Denker <marcus.denker@inria.fr>
On Jun 11, 2013, at 4:24 PM, Frank Shearar <frank.shearar@gmail.com
wrote:
On 11 June 2013 14:18, Sebastian Tleye <stleye@gmail.com> wrote:
Hello,
Fixing some bugs in Traits we realized that it would be good idea to use CollectionsExtensions package (it has some useful functions), also it would be great to include CollectionsExtensions in Pharo 3.0.
Is this the HPI library? I seem to recall TraitClasses [1] using it.
CollectionExtension is from Moose.
the Trait based Stream refactoring was done some years ago but has not been used or maintained.
Marcus
-- Guillaume Larcheveque
On Jun 12, 2013, at 1:02 PM, Sebastian Tleye <stleye@gmail.com> wrote:
ok, i understood,
Discussing with Marcus we believe that it would be better to keep Collecition>>gather: and add Collection>>flatCollect: (with the same implementation) and put a comment in gather: saying that is kept for compatibility purposes.
flatCollect: will be included in Pharo and will be removed from CollectionExtensions.
+ 1
Ok so if everyone agree I will remove the flatCollect: method from CollectionExtensions since Sebastian made the correction in Pharo 3.0. 2013/6/12 Stéphane Ducasse <stephane.ducasse@inria.fr>
On Jun 12, 2013, at 1:02 PM, Sebastian Tleye <stleye@gmail.com> wrote:
ok, i understood,
Discussing with Marcus we believe that it would be better to keep Collecition>>gather: and add Collection>>flatCollect: (with the same implementation) and put a comment in gather: saying that is kept for compatibility purposes.
flatCollect: will be included in Pharo and will be removed from CollectionExtensions.
+ 1
-- *Guillaume Larcheveque*
but pay attention because in 2.0 you will need it. :) On Jun 12, 2013, at 4:13 PM, Guillaume Larcheveque <guillaume.larcheveque@gmail.com> wrote:
Ok so if everyone agree I will remove the flatCollect: method from CollectionExtensions since Sebastian made the correction in Pharo 3.0.
2013/6/12 Stéphane Ducasse <stephane.ducasse@inria.fr>
On Jun 12, 2013, at 1:02 PM, Sebastian Tleye <stleye@gmail.com> wrote:
ok, i understood,
Discussing with Marcus we believe that it would be better to keep Collecition>>gather: and add Collection>>flatCollect: (with the same implementation) and put a comment in gather: saying that is kept for compatibility purposes.
flatCollect: will be included in Pharo and will be removed from CollectionExtensions.
+ 1
-- Guillaume Larcheveque
First, thanks for looking at CollectionsExtensions. But, please do not remove flatCollect: yet! This is used everywhere in Moose. We will release Moose at the end of this month and we will move to Pharo 3.0, and then we can do what we want. Doru -- www.tudorgirba.com "Every thing has its own flow." On 12.06.2013, at 17:41, Stéphane Ducasse <stephane.ducasse@inria.fr> wrote:
but pay attention because in 2.0 you will need it. :)
On Jun 12, 2013, at 4:13 PM, Guillaume Larcheveque <guillaume.larcheveque@gmail.com> wrote:
Ok so if everyone agree I will remove the flatCollect: method from CollectionExtensions since Sebastian made the correction in Pharo 3.0.
2013/6/12 Stéphane Ducasse <stephane.ducasse@inria.fr>
On Jun 12, 2013, at 1:02 PM, Sebastian Tleye <stleye@gmail.com> wrote:
ok, i understood,
Discussing with Marcus we believe that it would be better to keep Collecition>>gather: and add Collection>>flatCollect: (with the same implementation) and put a comment in gather: saying that is kept for compatibility purposes.
flatCollect: will be included in Pharo and will be removed from CollectionExtensions.
+ 1
-- Guillaume Larcheveque
Tudor Girba-2 wrote
We will release Moose at the end of this month and we will move to Pharo 3.0
I'm impressed! Did you move Moose to Pharo 2.0 before it was released? I have been hesitant to move my projects to the alpha/beta versions because we are making so much progress (i.e. breaking things, lol)... ----- Cheers, Sean -- View this message in context: http://forum.world.st/Pharo-dev-Modification-on-CollectionsExtensions-tp4692... Sent from the Pharo Smalltalk Developers mailing list archive at Nabble.com.
Moose is running (smoothly :)) on Pharo 2.0 since several months now. The idea is to merge the efforts to have more people working on top of the latest Pharo. There are two reasons for it: - first, we are too small of a community to split our efforts - second, often the Moose projects are directly relevant for Pharo and the other way around Cheers, Doru -- www.tudorgirba.com "Every thing has its own flow." On 13.06.2013, at 23:24, "Sean P. DeNigris" <sean@clipperadams.com> wrote:
Tudor Girba-2 wrote
We will release Moose at the end of this month and we will move to Pharo 3.0
I'm impressed! Did you move Moose to Pharo 2.0 before it was released? I have been hesitant to move my projects to the alpha/beta versions because we are making so much progress (i.e. breaking things, lol)...
----- Cheers, Sean -- View this message in context: http://forum.world.st/Pharo-dev-Modification-on-CollectionsExtensions-tp4692... Sent from the Pharo Smalltalk Developers mailing list archive at Nabble.com.
On 11 juin 2013, at 15:18, Sebastian Tleye wrote:
Hello,
Fixing some bugs in Traits we realized that it would be good idea to use CollectionsExtensions package (it has some useful functions), also it would be great to include CollectionsExtensions in Pharo 3.0.
One "problem" we found is that CollectionsExtensions is depending on a Nile Package (that's not a desired feature). In the method Collection>>flatCollect: there is a line refering to a nile function "nsWriteStream".
How this #flatCollect: is different from #gather: anyway?
If i change "nsWriteStream" for "writeStream" the dependency disapears.
I also run all the tests and they are working, so i was wondering if it would be possible to commit the change.
Thanks.
On 11 June 2013 15:37, Camille Teruel <camille.teruel@gmail.com> wrote:
On 11 juin 2013, at 15:18, Sebastian Tleye wrote:
Hello,
Fixing some bugs in Traits we realized that it would be good idea to use CollectionsExtensions package (it has some useful functions), also it would be great to include CollectionsExtensions in Pharo 3.0.
One "problem" we found is that CollectionsExtensions is depending on a Nile Package (that's not a desired feature). In the method Collection>>flatCollect: there is a line refering to a nile function "nsWriteStream".
How this #flatCollect: is different from #gather: anyway?
Functionally, it's no different. frank
If i change "nsWriteStream" for "writeStream" the dependency disapears.
I also run all the tests and they are working, so i was wondering if it would be possible to commit the change.
Thanks.
I don't know, i should see the implementation, i just want to remove the dependency between CollectionsExtensions and Nile and one way to do it is modifying #flatCollect: implementation. if they are the same #flatCollect: could be remove safely replaced. 2013/6/11 Camille Teruel <camille.teruel@gmail.com>
On 11 juin 2013, at 15:18, Sebastian Tleye wrote:
Hello,
Fixing some bugs in Traits we realized that it would be good idea to use CollectionsExtensions package (it has some useful functions), also it would be great to include CollectionsExtensions in Pharo 3.0.
One "problem" we found is that CollectionsExtensions is depending on a Nile Package (that's not a desired feature). In the method Collection>>flatCollect: there is a line refering to a nile function "nsWriteStream".
How this #flatCollect: is different from #gather: anyway?
If i change "nsWriteStream" for "writeStream" the dependency disapears.
I also run all the tests and they are working, so i was wondering if it
would be possible to commit the change.
Thanks.
Yes please go ahead. Stef On Jun 11, 2013, at 4:49 PM, Sebastian Tleye <stleye@gmail.com> wrote:
I don't know, i should see the implementation, i just want to remove the dependency between CollectionsExtensions and Nile and one way to do it is modifying #flatCollect: implementation.
if they are the same #flatCollect: could be remove safely replaced.
2013/6/11 Camille Teruel <camille.teruel@gmail.com>
On 11 juin 2013, at 15:18, Sebastian Tleye wrote:
Hello,
Fixing some bugs in Traits we realized that it would be good idea to use CollectionsExtensions package (it has some useful functions), also it would be great to include CollectionsExtensions in Pharo 3.0.
One "problem" we found is that CollectionsExtensions is depending on a Nile Package (that's not a desired feature). In the method Collection>>flatCollect: there is a line refering to a nile function "nsWriteStream".
How this #flatCollect: is different from #gather: anyway?
If i change "nsWriteStream" for "writeStream" the dependency disapears.
I also run all the tests and they are working, so i was wondering if it would be possible to commit the change.
Thanks.
Thanks again. I committed the change you proposed. So, now we have flatCollect: using writeStream. Cheers, Doru On Tue, Jun 11, 2013 at 3:18 PM, Sebastian Tleye <stleye@gmail.com> wrote:
Hello,
Fixing some bugs in Traits we realized that it would be good idea to use CollectionsExtensions package (it has some useful functions), also it would be great to include CollectionsExtensions in Pharo 3.0.
One "problem" we found is that CollectionsExtensions is depending on a Nile Package (that's not a desired feature). In the method Collection>>flatCollect: there is a line refering to a nile function "nsWriteStream".
If i change "nsWriteStream" for "writeStream" the dependency disapears.
I also run all the tests and they are working, so i was wondering if it would be possible to commit the change.
Thanks.
-- www.tudorgirba.com "Every thing has its own flow"
Great! Thank you! 2013/6/21 Tudor Girba <tudor@tudorgirba.com>
Thanks again.
I committed the change you proposed. So, now we have flatCollect: using writeStream.
Cheers, Doru
On Tue, Jun 11, 2013 at 3:18 PM, Sebastian Tleye <stleye@gmail.com> wrote:
Hello,
Fixing some bugs in Traits we realized that it would be good idea to use CollectionsExtensions package (it has some useful functions), also it would be great to include CollectionsExtensions in Pharo 3.0.
One "problem" we found is that CollectionsExtensions is depending on a Nile Package (that's not a desired feature). In the method Collection>>flatCollect: there is a line refering to a nile function "nsWriteStream".
If i change "nsWriteStream" for "writeStream" the dependency disapears.
I also run all the tests and they are working, so i was wondering if it would be possible to commit the change.
Thanks.
-- www.tudorgirba.com
"Every thing has its own flow"
participants (8)
-
Camille Teruel -
Frank Shearar -
Guillaume Larcheveque -
Marcus Denker -
Sean P. DeNigris -
Sebastian Tleye -
Stéphane Ducasse -
Tudor Girba