David T. Lewis uploaded a new version of 39Deprecated to project The Trunk: http://source.squeak.org/trunk/39Deprecated-fbs.20.mcz
==================== Summary ====================
Name: 39Deprecated-fbs.20 Author: fbs Time: 25 July 2012, 9:42:16.56 pm UUID: b22c5338-3200-4545-89bc-3f75c07e39af Ancestors: 39Deprecated-ar.19
These methods are deprecated, and I can find no uses in the base image.
=============== Diff against 39Deprecated-ar.19 ===============
Item was removed: - ----- Method: SharedQueue2>>flush (in category '*39Deprecated') ----- - flush - self deprecated: 'use removeAll'. - ^self removeAll!
Item was removed: - ----- Method: SharedQueue2>>flushAllSuchThat: (in category '*39Deprecated') ----- - flushAllSuchThat: aBlock - self deprecated: 'use removeAllSuchThat:'. - - ^self removeAllSuchThat: aBlock!
Moved from inbox to trunk. These two methods have been deprecated for five years, so we can let them go now.
Dave
On Wed, Sep 20, 2017 at 01:21:42PM +0000, commits@source.squeak.org wrote:
David T. Lewis uploaded a new version of 39Deprecated to project The Trunk: http://source.squeak.org/trunk/39Deprecated-fbs.20.mcz
==================== Summary ====================
Name: 39Deprecated-fbs.20 Author: fbs Time: 25 July 2012, 9:42:16.56 pm UUID: b22c5338-3200-4545-89bc-3f75c07e39af Ancestors: 39Deprecated-ar.19
These methods are deprecated, and I can find no uses in the base image.
=============== Diff against 39Deprecated-ar.19 ===============
Item was removed:
- ----- Method: SharedQueue2>>flush (in category '*39Deprecated') -----
- flush
- self deprecated: 'use removeAll'.
- ^self removeAll!
Item was removed:
- ----- Method: SharedQueue2>>flushAllSuchThat: (in category '*39Deprecated') -----
- flushAllSuchThat: aBlock
- self deprecated: 'use removeAllSuchThat:'.
- ^self removeAllSuchThat: aBlock!
Hi Dave,
why would we ever want to remove things from the "deprecated"-packages? Wouldn't be the only thing to do to not load that respective package at all?
We could start unloading some older "deprecated"-packages for the next release. If we are unsure about certain methods, we can always move them from an older "deprecated"-package to a newer, loaded "deprecated" package. :)
Best, Marcel Am 20.09.2017 15:23:53 schrieb David T. Lewis lewis@mail.msen.com: Moved from inbox to trunk. These two methods have been deprecated for five years, so we can let them go now.
Dave
On Wed, Sep 20, 2017 at 01:21:42PM +0000, commits@source.squeak.org wrote:
David T. Lewis uploaded a new version of 39Deprecated to project The Trunk: http://source.squeak.org/trunk/39Deprecated-fbs.20.mcz
==================== Summary ====================
Name: 39Deprecated-fbs.20 Author: fbs Time: 25 July 2012, 9:42:16.56 pm UUID: b22c5338-3200-4545-89bc-3f75c07e39af Ancestors: 39Deprecated-ar.19
These methods are deprecated, and I can find no uses in the base image.
=============== Diff against 39Deprecated-ar.19 ===============
Item was removed:
- ----- Method: SharedQueue2>>flush (in category '*39Deprecated') -----
- flush
- self deprecated: 'use removeAll'.
- ^self removeAll!
Item was removed:
- ----- Method: SharedQueue2>>flushAllSuchThat: (in category '*39Deprecated') -----
- flushAllSuchThat: aBlock
- self deprecated: 'use removeAllSuchThat:'.
- ^self removeAllSuchThat: aBlock!
On 9/20/17, Marcel Taeumel marcel.taeumel@hpi.de wrote:
Hi Dave,
why would we ever want to remove things from the "deprecated"-packages? Wouldn't be the only thing to do to not load that respective package at all?
+1
We could start unloading some older "deprecated"-packages for the next release. If we are unsure about certain methods, we can always move them from an older "deprecated"-package to a newer, loaded "deprecated" package. :)
+1
Best, Marcel Am 20.09.2017 15:23:53 schrieb David T. Lewis lewis@mail.msen.com: Moved from inbox to trunk. These two methods have been deprecated for five years, so we can let them go now.
Dave
On Wed, Sep 20, 2017 at 01:21:42PM +0000, commits@source.squeak.org wrote:
David T. Lewis uploaded a new version of 39Deprecated to project The Trunk: http://source.squeak.org/trunk/39Deprecated-fbs.20.mcz
==================== Summary ====================
Name: 39Deprecated-fbs.20 Author: fbs Time: 25 July 2012, 9:42:16.56 pm UUID: b22c5338-3200-4545-89bc-3f75c07e39af Ancestors: 39Deprecated-ar.19
These methods are deprecated, and I can find no uses in the base image.
=============== Diff against 39Deprecated-ar.19 ===============
Item was removed:
- ----- Method: SharedQueue2>>flush (in category '*39Deprecated') -----
- flush
- self deprecated: 'use removeAll'.
- ^self removeAll!
Item was removed:
- ----- Method: SharedQueue2>>flushAllSuchThat: (in category
'*39Deprecated') -----
- flushAllSuchThat: aBlock
- self deprecated: 'use removeAllSuchThat:'.
- ^self removeAllSuchThat: aBlock!
You are right.
Actually, I was just picking a random package out of the inbox to clean up, and that was the first one one I picked. I figured that after five years it was probably time it was time to get it out of the inbox ;-)
I should have just moved that one to the treated inbox instead. I will revert.
Dave
On Wed, Sep 20, 2017 at 04:40:26PM +0200, Marcel Taeumel wrote:
Hi Dave,
why would we ever want to remove things from the "deprecated"-packages???Wouldn't be the only thing to do to not load that respective package at all?
We could start unloading some older "deprecated"-packages for the next release. If we are unsure about certain methods, we can always move them from an older "deprecated"-package to a newer, loaded "deprecated" package. :)
Best, Marcel Am 20.09.2017 15:23:53 schrieb David T. Lewis lewis@mail.msen.com: Moved from inbox to trunk. These two methods have been deprecated for five years, so we can let them go now.
Dave
On Wed, Sep 20, 2017 at 01:21:42PM +0000, commits@source.squeak.org wrote:
David T. Lewis uploaded a new version of 39Deprecated to project The Trunk: http://source.squeak.org/trunk/39Deprecated-fbs.20.mcz
==================== Summary ====================
Name: 39Deprecated-fbs.20 Author: fbs Time: 25 July 2012, 9:42:16.56 pm UUID: b22c5338-3200-4545-89bc-3f75c07e39af Ancestors: 39Deprecated-ar.19
These methods are deprecated, and I can find no uses in the base image.
=============== Diff against 39Deprecated-ar.19 ===============
Item was removed:
- ----- Method: SharedQueue2>>flush (in category '*39Deprecated') -----
- flush
- self deprecated: 'use removeAll'.
- ^self removeAll!
Item was removed:
- ----- Method: SharedQueue2>>flushAllSuchThat: (in category '*39Deprecated') -----
- flushAllSuchThat: aBlock
- self deprecated: 'use removeAllSuchThat:'.
- ^self removeAllSuchThat: aBlock!
These methods were there to make it possible to migrate from SharedQueue to SharedQueue2 in a live system. That migration has not happened yet, so removing them means SharedQueue2 will not replace SharedQueue.
Levente
On Wed, 20 Sep 2017, David T. Lewis wrote:
Moved from inbox to trunk. These two methods have been deprecated for five years, so we can let them go now.
Dave
On Wed, Sep 20, 2017 at 01:21:42PM +0000, commits@source.squeak.org wrote:
David T. Lewis uploaded a new version of 39Deprecated to project The Trunk: http://source.squeak.org/trunk/39Deprecated-fbs.20.mcz
==================== Summary ====================
Name: 39Deprecated-fbs.20 Author: fbs Time: 25 July 2012, 9:42:16.56 pm UUID: b22c5338-3200-4545-89bc-3f75c07e39af Ancestors: 39Deprecated-ar.19
These methods are deprecated, and I can find no uses in the base image.
=============== Diff against 39Deprecated-ar.19 ===============
Item was removed:
- ----- Method: SharedQueue2>>flush (in category '*39Deprecated') -----
- flush
- self deprecated: 'use removeAll'.
- ^self removeAll!
Item was removed:
- ----- Method: SharedQueue2>>flushAllSuchThat: (in category '*39Deprecated') -----
- flushAllSuchThat: aBlock
- self deprecated: 'use removeAllSuchThat:'.
- ^self removeAllSuchThat: aBlock!
On 9/20/17, Levente Uzonyi leves@caesar.elte.hu wrote:
These methods were there to make it possible to migrate from SharedQueue to SharedQueue2 in a live system.
Could you please give some more background information about this migration?
That migration has not happened yet,
Is there a reason not to do it after 5 years?
so removing them means SharedQueue2 will not replace SharedQueue.
Levente
On Wed, 20 Sep 2017, David T. Lewis wrote:
Moved from inbox to trunk. These two methods have been deprecated for five years, so we can let them go now.
Dave
On Wed, Sep 20, 2017 at 01:21:42PM +0000, commits@source.squeak.org wrote:
David T. Lewis uploaded a new version of 39Deprecated to project The Trunk: http://source.squeak.org/trunk/39Deprecated-fbs.20.mcz
==================== Summary ====================
Name: 39Deprecated-fbs.20 Author: fbs Time: 25 July 2012, 9:42:16.56 pm UUID: b22c5338-3200-4545-89bc-3f75c07e39af Ancestors: 39Deprecated-ar.19
These methods are deprecated, and I can find no uses in the base image.
=============== Diff against 39Deprecated-ar.19 ===============
Item was removed:
- ----- Method: SharedQueue2>>flush (in category '*39Deprecated') -----
- flush
- self deprecated: 'use removeAll'.
- ^self removeAll!
Item was removed:
- ----- Method: SharedQueue2>>flushAllSuchThat: (in category
'*39Deprecated') -----
- flushAllSuchThat: aBlock
- self deprecated: 'use removeAllSuchThat:'.
- ^self removeAllSuchThat: aBlock!
On Wed, 20 Sep 2017, H. Hirzel wrote:
On 9/20/17, Levente Uzonyi leves@caesar.elte.hu wrote:
These methods were there to make it possible to migrate from SharedQueue to SharedQueue2 in a live system.
Could you please give some more background information about this migration?
http://bugs.squeak.org/view.php?id=1375
That migration has not happened yet,
Is there a reason not to do it after 5 years?
The mantis issue above has a few potential reasons. Another one may be that now there are other potential shared queue implementations to replace SharedQueue.
Levente
so removing them means SharedQueue2 will not replace SharedQueue.
Levente
On Wed, 20 Sep 2017, David T. Lewis wrote:
Moved from inbox to trunk. These two methods have been deprecated for five years, so we can let them go now.
Dave
On Wed, Sep 20, 2017 at 01:21:42PM +0000, commits@source.squeak.org wrote:
David T. Lewis uploaded a new version of 39Deprecated to project The Trunk: http://source.squeak.org/trunk/39Deprecated-fbs.20.mcz
==================== Summary ====================
Name: 39Deprecated-fbs.20 Author: fbs Time: 25 July 2012, 9:42:16.56 pm UUID: b22c5338-3200-4545-89bc-3f75c07e39af Ancestors: 39Deprecated-ar.19
These methods are deprecated, and I can find no uses in the base image.
=============== Diff against 39Deprecated-ar.19 ===============
Item was removed:
- ----- Method: SharedQueue2>>flush (in category '*39Deprecated') -----
- flush
- self deprecated: 'use removeAll'.
- ^self removeAll!
Item was removed:
- ----- Method: SharedQueue2>>flushAllSuchThat: (in category
'*39Deprecated') -----
- flushAllSuchThat: aBlock
- self deprecated: 'use removeAllSuchThat:'.
- ^self removeAllSuchThat: aBlock!
squeak-dev@lists.squeakfoundation.org