Creating a new class using `Class new superclass:` blocks the image in Pharo 7

Hi,

Executed in the latest Pharo 7 image (Pharo-7.0.0+alpha.build.1310.sha.a454977bb3d55bc4fcca3a57ac2fafcf137c0f30 (64 Bit)) the code below blocks the image (both with latest and stable vm on MacOs HighSierra). The vm does not crash but gets into a "Not Responding" state and does not get out:

Class new        

     superclass: GLMTransmission;        

     setFormat: GLMTransmission format;

     classLayout: GLMTransmission classLayout copy;        

     yourself 

Code like this is used in Moose in some tests and in SmaCC in the rewrite engine:

Class new    

    superclass: SmaCCRewriteMatchContext;    

    setFormat: SmaCCRewriteMatchContext format;     

    classLayout: SmaCCRewriteMatchContext classLayout copy;    

    yourself. 

This worked in Pharo 6. Bug or here should be a different way to do this?

Cheers,

Andrei

Now I found the comment from https://github.com/pharo-project/pharo/pull/1618.

Will add a comment there.

On Fri, Oct 5, 2018 at 7:48 PM Andrei Chis <[hidden email]> wrote:

Hi,

Executed in the latest Pharo 7 image (Pharo-7.0.0+alpha.build.1310.sha.a454977bb3d55bc4fcca3a57ac2fafcf137c0f30 (64 Bit)) the code below blocks the image (both with latest and stable vm on MacOs HighSierra). The vm does not crash but gets into a "Not Responding" state and does not get out:

Class new        

     superclass: GLMTransmission;        

     setFormat: GLMTransmission format;

     classLayout: GLMTransmission classLayout copy;        

     yourself 

Code like this is used in Moose in some tests and in SmaCC in the rewrite engine:

Class new    

    superclass: SmaCCRewriteMatchContext;    

    setFormat: SmaCCRewriteMatchContext format;     

    classLayout: SmaCCRewriteMatchContext classLayout copy;    

    yourself. 

This worked in Pharo 6. Bug or here should be a different way to do this?

Cheers,

Andrei

Hi Andrei, would you mind opening an issue for this, making a comment in a merged PR does not help to see it.

Thanks 

On Fri, 5 Oct 2018, 20:09 Andrei Chis, <[hidden email]> wrote:

Now I found the comment from https://github.com/pharo-project/pharo/pull/1618.

Will add a comment there.

On Fri, Oct 5, 2018 at 7:48 PM Andrei Chis <[hidden email]> wrote:

Hi,

Executed in the latest Pharo 7 image (Pharo-7.0.0+alpha.build.1310.sha.a454977bb3d55bc4fcca3a57ac2fafcf137c0f30 (64 Bit)) the code below blocks the image (both with latest and stable vm on MacOs HighSierra). The vm does not crash but gets into a "Not Responding" state and does not get out:

Class new        

     superclass: GLMTransmission;        

     setFormat: GLMTransmission format;

     classLayout: GLMTransmission classLayout copy;        

     yourself 

Code like this is used in Moose in some tests and in SmaCC in the rewrite engine:

Class new    

    superclass: SmaCCRewriteMatchContext;    

    setFormat: SmaCCRewriteMatchContext format;     

    classLayout: SmaCCRewriteMatchContext classLayout copy;    

    yourself. 

This worked in Pharo 6. Bug or here should be a different way to do this?

Cheers,

Andrei

Done: https://pharo.fogbugz.com/f/cases/22547/Creating-a-new-class-using-Class-new-superclass-blocks-the-image-in-Pharo-7

Cheers,

Andrei

On Fri, Oct 5, 2018 at 8:15 PM [hidden email] <[hidden email]> wrote:

Hi Andrei, would you mind opening an issue for this, making a comment in a merged PR does not help to see it.

Thanks 

On Fri, 5 Oct 2018, 20:09 Andrei Chis, <[hidden email]> wrote:

Now I found the comment from https://github.com/pharo-project/pharo/pull/1618.

Will add a comment there.

On Fri, Oct 5, 2018 at 7:48 PM Andrei Chis <[hidden email]> wrote:

Hi,

Executed in the latest Pharo 7 image (Pharo-7.0.0+alpha.build.1310.sha.a454977bb3d55bc4fcca3a57ac2fafcf137c0f30 (64 Bit)) the code below blocks the image (both with latest and stable vm on MacOs HighSierra). The vm does not crash but gets into a "Not Responding" state and does not get out:

Class new        

     superclass: GLMTransmission;        

     setFormat: GLMTransmission format;

     classLayout: GLMTransmission classLayout copy;        

     yourself 

Code like this is used in Moose in some tests and in SmaCC in the rewrite engine:

Class new    

    superclass: SmaCCRewriteMatchContext;    

    setFormat: SmaCCRewriteMatchContext format;     

    classLayout: SmaCCRewriteMatchContext classLayout copy;    

    yourself. 

This worked in Pharo 6. Bug or here should be a different way to do this?

Cheers,

Andrei