The Trunk: Kernel-dtl.1124.mcz

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

The Trunk: Kernel-dtl.1124.mcz

commits-2
David T. Lewis uploaded a new version of Kernel to project The Trunk:
http://source.squeak.org/trunk/Kernel-dtl.1124.mcz

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

Name: Kernel-dtl.1124
Author: dtl
Time: 18 November 2017, 5:44:53.215977 pm
UUID: 290e3cd7-ffe5-442e-9095-0217cc0b8be3
Ancestors: Kernel-dtl.1123

UndefinedObject>>suspend was presumably implemented for debugging process scheduling issues a long time ago. It serves no useful purpose now, and is dangerous because it will hang the image if evaluated. Remove it.

=============== Diff against Kernel-dtl.1123 ===============

Item was removed:
- ----- Method: UndefinedObject>>suspend (in category 'dependents access') -----
- suspend
- "Kills off processes that didn't terminate properly"
- "Display reverse; reverse."  "<-- So we can catch the suspend bug"
- Processor terminateActive!


Reply | Threaded
Open this post in threaded view
|

Re: The Trunk: Kernel-dtl.1124.mcz

marcel.taeumel
Could we just deprecate it instead of deleting it? :-)

Best,
Marcel

Am 18.11.2017 23:45:23 schrieb [hidden email] <[hidden email]>:

David T. Lewis uploaded a new version of Kernel to project The Trunk:
http://source.squeak.org/trunk/Kernel-dtl.1124.mcz

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

Name: Kernel-dtl.1124
Author: dtl
Time: 18 November 2017, 5:44:53.215977 pm
UUID: 290e3cd7-ffe5-442e-9095-0217cc0b8be3
Ancestors: Kernel-dtl.1123

UndefinedObject>>suspend was presumably implemented for debugging process scheduling issues a long time ago. It serves no useful purpose now, and is dangerous because it will hang the image if evaluated. Remove it.

=============== Diff against Kernel-dtl.1123 ===============

Item was removed:
- ----- Method: UndefinedObject>>suspend (in category 'dependents access') -----
- suspend
- "Kills off processes that didn't terminate properly"
- "Display reverse; reverse." "<-- so="" we="" can="" catch="" the="" suspend="">
- Processor terminateActive!




Reply | Threaded
Open this post in threaded view
|

Re: The Trunk: Kernel-dtl.1124.mcz

David T. Lewis
We could deprecate it, but then how would we know when it was safe to remove
it?  After all, it hangs the image so it is unlikely that there have been any
senders for a very long time.

Granted, it might have some use for debugging something in MVC, so I guess that
we could reactivate the "Display reverse; reverse" line and then move the
method to 60Deprecated on the theory that this would do something noticeable
in MVC. But I suspect that just dropping into an emergency evaluator might be
better at that point.

Dave

On Sun, Nov 19, 2017 at 08:47:53AM +0100, Marcel Taeumel wrote:

> Could we just deprecate it instead of deleting it? :-)
>
> Best,
> Marcel
> Am 18.11.2017 23:45:23 schrieb [hidden email] <[hidden email]>:
> David T. Lewis uploaded a new version of Kernel to project The Trunk:
> http://source.squeak.org/trunk/Kernel-dtl.1124.mcz
>
> ==================== Summary ====================
>
> Name: Kernel-dtl.1124
> Author: dtl
> Time: 18 November 2017, 5:44:53.215977 pm
> UUID: 290e3cd7-ffe5-442e-9095-0217cc0b8be3
> Ancestors: Kernel-dtl.1123
>
> UndefinedObject>>suspend was presumably implemented for debugging process scheduling issues a long time ago. It serves no useful purpose now, and is dangerous because it will hang the image if evaluated. Remove it.
>
> =============== Diff against Kernel-dtl.1123 ===============
>
> Item was removed:
> - ----- Method: UndefinedObject>>suspend (in category 'dependents access') -----
> - suspend
> - "Kills off processes that didn't terminate properly"
> - "Display reverse; reverse." "
> - Processor terminateActive!
>
>


Reply | Threaded
Open this post in threaded view
|

Re: The Trunk: Kernel-dtl.1124.mcz

marcel.taeumel
Let's not mix up the discussions/issues here. :-) This one here should be a deprecation. Then we should start unloading older depracation packages from the Trunk.

I vote for keeping only 50Deprecated, 51Deprecated, and (obvioulsy) 60Depcrecated in the next release. So, let's start removing those other *Deprecated packages right now. :-)

Best,
Marcel

Am 19.11.2017 12:16:28 schrieb David T. Lewis <[hidden email]>:

We could deprecate it, but then how would we know when it was safe to remove
it? After all, it hangs the image so it is unlikely that there have been any
senders for a very long time.

Granted, it might have some use for debugging something in MVC, so I guess that
we could reactivate the "Display reverse; reverse" line and then move the
method to 60Deprecated on the theory that this would do something noticeable
in MVC. But I suspect that just dropping into an emergency evaluator might be
better at that point.

Dave

On Sun, Nov 19, 2017 at 08:47:53AM +0100, Marcel Taeumel wrote:
> Could we just deprecate it instead of deleting it? :-)
>
> Best,
> Marcel
> Am 18.11.2017 23:45:23 schrieb [hidden email] :
> David T. Lewis uploaded a new version of Kernel to project The Trunk:
> http://source.squeak.org/trunk/Kernel-dtl.1124.mcz
>
> ==================== Summary ====================
>
> Name: Kernel-dtl.1124
> Author: dtl
> Time: 18 November 2017, 5:44:53.215977 pm
> UUID: 290e3cd7-ffe5-442e-9095-0217cc0b8be3
> Ancestors: Kernel-dtl.1123
>
> UndefinedObject>>suspend was presumably implemented for debugging process scheduling issues a long time ago. It serves no useful purpose now, and is dangerous because it will hang the image if evaluated. Remove it.
>
> =============== Diff against Kernel-dtl.1123 ===============
>
> Item was removed:
> - ----- Method: UndefinedObject>>suspend (in category 'dependents access') -----
> - suspend
> - "Kills off processes that didn't terminate properly"
> - "Display reverse; reverse." "
> - Processor terminateActive!
>
>




Reply | Threaded
Open this post in threaded view
|

Re: The Trunk: Kernel-dtl.1124.mcz

David T. Lewis
On Mon, Nov 20, 2017 at 02:51:42PM +0100, Marcel Taeumel wrote:
>
> Let's not mix up the discussions/issues here. :-) This one here should be a deprecation.

OK,I put UndefinedObject>>suspend in the 60Deprecated package.

Dave


> Then we should start unloading older depracation packages from the Trunk.
>
> I vote for keeping only 50Deprecated, 51Deprecated, and (obvioulsy) 60Depcrecated in the next release. So, let's start removing those other *Deprecated packages right now. :-)
>
> Best,
> Marcel
> Am 19.11.2017 12:16:28 schrieb David T. Lewis <[hidden email]>:
> We could deprecate it, but then how would we know when it was safe to remove
> it? After all, it hangs the image so it is unlikely that there have been any
> senders for a very long time.
>
> Granted, it might have some use for debugging something in MVC, so I guess that
> we could reactivate the "Display reverse; reverse" line and then move the
> method to 60Deprecated on the theory that this would do something noticeable
> in MVC. But I suspect that just dropping into an emergency evaluator might be
> better at that point.
>
> Dave
>
> On Sun, Nov 19, 2017 at 08:47:53AM +0100, Marcel Taeumel wrote:
> > Could we just deprecate it instead of deleting it? :-)
> >
> > Best,
> > Marcel
> > Am 18.11.2017 23:45:23 schrieb [hidden email] :
> > David T. Lewis uploaded a new version of Kernel to project The Trunk:
> > http://source.squeak.org/trunk/Kernel-dtl.1124.mcz
> >
> > ==================== Summary ====================
> >
> > Name: Kernel-dtl.1124
> > Author: dtl
> > Time: 18 November 2017, 5:44:53.215977 pm
> > UUID: 290e3cd7-ffe5-442e-9095-0217cc0b8be3
> > Ancestors: Kernel-dtl.1123
> >
> > UndefinedObject>>suspend was presumably implemented for debugging process scheduling issues a long time ago. It serves no useful purpose now, and is dangerous because it will hang the image if evaluated. Remove it.
> >
> > =============== Diff against Kernel-dtl.1123 ===============
> >
> > Item was removed:
> > - ----- Method: UndefinedObject>>suspend (in category 'dependents access') -----
> > - suspend
> > - "Kills off processes that didn't terminate properly"
> > - "Display reverse; reverse." "
> > - Processor terminateActive!
> >
> >
>
>

>